Fixes things

jestjs · Oct 2, 2018 · b512ae4 · b512ae4
1 parent 60f37bf
commit b512ae4
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/packages/jest-haste-map/src/crawlers/__tests__/watchman.test.js b/packages/jest-haste-map/src/crawlers/__tests__/watchman.test.js
@@ -201,6 +201,7 @@ describe('watchman watch', () => {
         n.endsWith('.zip')
           ? [path.join(n, 'foo.1.js'), path.join(n, 'foo.2.js')]
           : null,
+      rootDir: ROOT_MOCK,
       roots: ROOTS,
     }).then(data => {
       expect(data.files).toEqual({

diff --git a/packages/jest-haste-map/src/crawlers/watchman.js b/packages/jest-haste-map/src/crawlers/watchman.js
@@ -184,13 +184,13 @@ module.exports = async function watchmanCrawl(
         const existingFileData = data.files.get(relativeFilePath);
         let nextData;
 
-        const isOld = existingFileData && existingFileData[H.MTIME] === mtime;
-        const hasChanged =
-          existingFileData && sha1hex && existingFileData[H.SHA1] === sha1hex;
-
-        if (existingFileData && isOld) {
+        if (existingFileData && existingFileData[H.MTIME] === mtime) {
           nextData = existingFileData;
-        } else if (existingFileData && !hasChanged) {
+        } else if (
+          existingFileData &&
+          sha1hex &&
+          existingFileData[H.SHA1] === sha1hex
+        ) {
           nextData = [...existingFileData];
           nextData[1] = mtime;
         } else {