mirror of
https://github.com/nodejs/node.git
synced 2024-12-01 16:10:02 +01:00
7dd82dd1c3
This commit adds a mustNotCall() helper for testing. This provides an alternative to using common.fail() as a callback, or creating a callback function for the sole purpose of calling common.fail(). PR-URL: https://github.com/nodejs/node/pull/11152 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
71 lines
2.7 KiB
JavaScript
71 lines
2.7 KiB
JavaScript
'use strict';
|
|
|
|
/*
|
|
* This is a regression test for https://github.com/nodejs/node/issues/7722.
|
|
*
|
|
* When nested timers have the same timeout, calling clearTimeout on the
|
|
* older timer after it has fired causes the list the newer timer is in
|
|
* to be deleted. Since the newer timer was not cleared, it still blocks
|
|
* the event loop completing for the duration of its timeout, however, since
|
|
* no reference exists to it in its list, it cannot be canceled and its
|
|
* callback is not called when the timeout elapses.
|
|
*/
|
|
|
|
const common = require('../common');
|
|
const assert = require('assert');
|
|
const Timer = process.binding('timer_wrap').Timer;
|
|
|
|
const TIMEOUT = common.platformTimeout(100);
|
|
|
|
const handle1 = setTimeout(common.mustCall(function() {
|
|
// Cause the old TIMEOUT list to be deleted
|
|
clearTimeout(handle1);
|
|
|
|
// Cause a new list with the same key (TIMEOUT) to be created for this timer
|
|
const handle2 = setTimeout(common.mustNotCall(), TIMEOUT);
|
|
|
|
setTimeout(common.mustCall(function() {
|
|
// Attempt to cancel the second timer. Fix for this bug will keep the
|
|
// newer timer from being dereferenced by keeping its list from being
|
|
// erroneously deleted. If we are able to cancel the timer successfully,
|
|
// the bug is fixed.
|
|
clearTimeout(handle2);
|
|
|
|
setImmediate(common.mustCall(function() {
|
|
setImmediate(common.mustCall(function() {
|
|
const activeTimers = getActiveTimers();
|
|
|
|
// Make sure our clearTimeout succeeded. One timer finished and
|
|
// the other was canceled, so none should be active.
|
|
assert.strictEqual(activeTimers.length, 0, 'Timers remain.');
|
|
}));
|
|
}));
|
|
}), 1);
|
|
|
|
// Make sure our timers got added to the list.
|
|
const activeTimers = getActiveTimers();
|
|
const shortTimer = activeTimers.find(function(handle) {
|
|
return handle._list.msecs === 1;
|
|
});
|
|
const longTimers = activeTimers.filter(function(handle) {
|
|
return handle._list.msecs === TIMEOUT;
|
|
});
|
|
|
|
// Make sure our clearTimeout succeeded. One timer finished and
|
|
// the other was canceled, so none should be active.
|
|
assert.strictEqual(activeTimers.length, 3,
|
|
'There should be 3 timers in the list.');
|
|
assert(shortTimer instanceof Timer, 'The shorter timer is not in the list.');
|
|
assert.strictEqual(longTimers.length, 2,
|
|
'Both longer timers should be in the list.');
|
|
|
|
// When this callback completes, `listOnTimeout` should now look at the
|
|
// correct list and refrain from removing the new TIMEOUT list which
|
|
// contains the reference to the newer timer.
|
|
}), TIMEOUT);
|
|
|
|
function getActiveTimers() {
|
|
const activeHandles = process._getActiveHandles();
|
|
return activeHandles.filter((handle) => handle instanceof Timer);
|
|
}
|