Browse Source

test: increase timeouts on some unref timers tests

Fix: https://github.com/nodejs/node/issues/5351
Refs: https://github.com/nodejs/node/pull/4561
PR-URL: https://github.com/nodejs/node/pull/5352

Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Minwoo Jung <jmwsoft@gmail.com>
v5.x
Jeremiah Senkpiel 9 years ago
committed by Rod Vagg
parent
commit
0f54553a99
  1. 2
      test/parallel/test-timers-unref-active-unenrolled-disposed.js
  2. 2
      test/parallel/test-timers-unref-active.js
  3. 2
      test/parallel/test-timers-unrefd-interval-still-fires.js

2
test/parallel/test-timers-unref-active-unenrolled-disposed.js

@ -9,7 +9,7 @@ const domain = require('domain');
// Crazy stuff to keep the process open, // Crazy stuff to keep the process open,
// then close it when we are actually done. // then close it when we are actually done.
const TEST_DURATION = common.platformTimeout(100); const TEST_DURATION = common.platformTimeout(1000);
const keepOpen = setTimeout(function() { const keepOpen = setTimeout(function() {
throw new Error('Test timed out. keepOpen was not canceled.'); throw new Error('Test timed out. keepOpen was not canceled.');
}, TEST_DURATION); }, TEST_DURATION);

2
test/parallel/test-timers-unref-active.js

@ -31,7 +31,7 @@ var nbTimeouts = 0;
* and thus expect 5 timers to be able to fire in under 100 ms. * and thus expect 5 timers to be able to fire in under 100 ms.
*/ */
const N = 5; const N = 5;
const TEST_DURATION = 100; const TEST_DURATION = 1000;
timers.unenroll(someObject); timers.unenroll(someObject);
timers.enroll(someObject, 1); timers.enroll(someObject, 1);

2
test/parallel/test-timers-unrefd-interval-still-fires.js

@ -4,7 +4,7 @@
*/ */
const common = require('../common'); const common = require('../common');
const TEST_DURATION = common.platformTimeout(100); const TEST_DURATION = common.platformTimeout(1000);
const N = 3; const N = 3;
var nbIntervalFired = 0; var nbIntervalFired = 0;

Loading…
Cancel
Save