fixup

nodejs · Jun 28, 2024 · e270e58 · e270e58
1 parent c583049
commit e270e58
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/lib/internal/test_runner/runner.js b/lib/internal/test_runner/runner.js
@@ -425,19 +425,18 @@ function watchFiles(testFiles, opts) {
       const newFileName = ArrayPrototypeFind(updatedTestFiles, (x) => !ArrayPrototypeIncludes(testFiles, x));
       const previousFileName = ArrayPrototypeFind(testFiles, (x) => !ArrayPrototypeIncludes(updatedTestFiles, x));
 
+      testFiles = updatedTestFiles;
+
       // When file renamed
       if (newFileName && previousFileName) {
         owners = new SafeSet().add(newFileName);
         watcher.filterFile(resolve(newFileName), owners);
       }
 
-      // When file deleted
       if (!newFileName && previousFileName) {
-        testFiles = updatedTestFiles;
-        return;
+        return; // Avoid rerunning files when file deleted
       }
 
-      testFiles = updatedTestFiles;
     }
 
     watcher.unfilterFilesOwnedBy(owners);

diff --git a/test/parallel/test-runner-watch-mode-complex.mjs b/test/parallel/test-runner-watch-mode-complex.mjs
@@ -41,8 +41,11 @@ import 'data:text/javascript,';
 test('test to delete has ran');`,
 };
 
-const fixturePaths = Object.keys(fixtureContent)
-  .reduce((acc, file) => ({ ...acc, [file]: tmpdir.resolve(file) }), {});
+const fixturePaths = Object.fromEntries(Object.keys(fixtureContent)
+  .map((file) => [file, tmpdir.resolve(file)]));
+
+  console.log(fixturePaths)
+
 Object.entries(fixtureContent)
   .forEach(([file, content]) => writeFileSync(fixturePaths[file], content));