also validate in haste-map

jestjs · Nov 2, 2020 · d0bccc4 · d0bccc4
1 parent 5c365ab
commit d0bccc4
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 0 deletions.
diff --git a/e2e/__tests__/crawlSymlinks.test.ts b/e2e/__tests__/crawlSymlinks.test.ts
@@ -6,6 +6,7 @@
  */
 import {tmpdir} from 'os';
 import * as path from 'path';
+import {wrap} from 'jest-snapshot-serializer-raw';
 import {cleanup, writeFiles, writeSymlinks} from '../Utils';
 import runJest from '../runJest';
 
@@ -69,3 +70,22 @@ test('Node crawler does not pick up symlinked files by default', () => {
   expect(stderr).toEqual('');
   expect(exitCode).toEqual(1);
 });
+
+test('Should throw if watchman used with haste.enableSymlinks', () => {
+  init({'.watchmanconfig': JSON.stringify({})});
+
+  const {stdout, stderr, exitCode} = runJest(DIR, [
+    '--haste={"enableSymlinks": true}',
+    '--watchman',
+  ]);
+
+  expect(stdout).toEqual('');
+  expect(wrap(stderr)).toMatchInlineSnapshot(`
+    Validation Error:
+
+    haste.enableSymlinks is incompatible with watchman
+
+    Either set haste.enableSymlinks to false or do not use watchman
+  `);
+  expect(exitCode).toEqual(1);
+});
diff --git a/packages/jest-haste-map/src/index.ts b/packages/jest-haste-map/src/index.ts
@@ -278,6 +278,14 @@ class HasteMap extends EventEmitter {
       this._options.ignorePattern = new RegExp(VCS_DIRECTORIES);
     }
 
+    if (this._options.enableSymlinks && this._options.useWatchman) {
+      throw new Error(
+        'jest-haste-map: enableSymlinks config option was set, but ' +
+          'is incompatible with watchman.\n' +
+          'Set either `enableSymlinks` to false or `watchman` to false ',
+      );
+    }
+
     const rootDirHash = createHash('md5').update(options.rootDir).digest('hex');
     let hasteImplHash = '';
     let dependencyExtractorHash = '';