CR

nodejs · Sep 12, 2022 · 60a33fa · 60a33fa
1 parent cf98295
commit 60a33fa
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/test/sequential/test-watch-mode.mjs b/test/sequential/test-watch-mode.mjs
@@ -15,6 +15,7 @@ if (common.isIBMi)
   common.skip('IBMi does not support `fs.watch()`');
 
 function restart(file) {
+  // To avoid flakiness, we save the file repeatedly until test is done
   writeFileSync(file, readFileSync(file));
   const timer = setInterval(() => writeFileSync(file, readFileSync(file)), 100);
   return () => clearInterval(timer);
@@ -48,6 +49,7 @@ async function spawnWithRestarts({
     if (restartsCount >= restarts) {
       cancelRestarts?.();
       if (!child.kill()) {
+        // If failed to kill, kill forcefully
         setTimeout(() => child.kill('SIGKILL'), 1);
       }
       return;