test: fix flaky unrefed timers test

Remove unnecessary `setImmediate()` that causes a minor race condition. Stop the test after 3 occurrences rather than 5 to allow for slower hosts running the test in parallel with other tests. Fixes: nodejs#4559 PR-URL: nodejs#4599 Reviewed-By: Johan Bergström <[email protected]>
MylesBorins · Feb 15, 2016 · 556703d · 556703d
1 parent 15fa555
commit 556703d
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/test/parallel/test-timers-unrefd-interval-still-fires.js b/test/parallel/test-timers-unrefd-interval-still-fires.js
@@ -5,7 +5,7 @@
 const common = require('../common');
 
 const TEST_DURATION = common.platformTimeout(100);
-const N = 5;
+const N = 3;
 var nbIntervalFired = 0;
 
 const keepOpen = setTimeout(() => {
@@ -20,7 +20,7 @@ const timer = setInterval(() => {
     timer._onTimeout = () => {
       throw new Error('Unrefd interval fired after being cleared.');
     };
-    setImmediate(() => clearTimeout(keepOpen));
+    clearTimeout(keepOpen);
   }
 }, 1);