fix(jest-haste-map): Add computeDependencies to cache path

jestjs · Sep 30, 2021 · 83705af · 83705af
1 parent da7c4a4
commit 83705af
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,8 @@
 
 ### Fixes
 
+- `[jest-haste-map]` Use distinct cache paths for different values of `computeDependencies`.
+
 ### Chore & Maintenance
 
 ### Performance

diff --git a/packages/jest-haste-map/src/__tests__/index.test.js b/packages/jest-haste-map/src/__tests__/index.test.js
@@ -265,6 +265,20 @@ describe('HasteMap', () => {
     expect(hasteMap1.getCacheFilePath()).not.toBe(hasteMap2.getCacheFilePath());
   });
 
+  it('creates different cache file paths for different values of computeDependencies', () => {
+    jest.resetModules();
+    const HasteMap = require('../').default;
+    const hasteMap1 = new HasteMap({
+      ...defaultConfig,
+      computeDependencies: true,
+    });
+    const hasteMap2 = new HasteMap({
+      ...defaultConfig,
+      computeDependencies: false,
+    });
+    expect(hasteMap1.getCacheFilePath()).not.toBe(hasteMap2.getCacheFilePath());
+  });
+
   it('creates different cache file paths for different hasteImplModulePath cache keys', () => {
     jest.resetModules();
     const HasteMap = require('../').default;

diff --git a/packages/jest-haste-map/src/index.ts b/packages/jest-haste-map/src/index.ts
@@ -325,6 +325,7 @@ export default class HasteMap extends EventEmitter {
       (options.ignorePattern || '').toString(),
       hasteImplHash,
       dependencyExtractorHash,
+      this._options.computeDependencies.toString(),
     );
     this._buildPromise = null;
     this._watchers = [];