mirror of
https://github.com/nodejs/node.git
synced 2025-04-29 14:25:18 +00:00

This ensures that microtasks scheduled by native immediates are run
after the tasks are done. In particular, this affects the inspector
integration since 6f9f546406
.
Fixes: https://github.com/nodejs/node/issues/33002
Refs: https://github.com/nodejs/node/pull/32523
PR-URL: https://github.com/nodejs/node/pull/34366
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
52 lines
1.2 KiB
JavaScript
52 lines
1.2 KiB
JavaScript
'use strict';
|
|
const common = require('../common');
|
|
const assert = require('assert');
|
|
const async_hooks = require('async_hooks');
|
|
|
|
// Checks that enabling async hooks in a callback actually
|
|
// triggers after & destroy as expected.
|
|
|
|
const fnsToTest = [setTimeout, (cb) => {
|
|
setImmediate(() => {
|
|
cb();
|
|
|
|
// We need to keep the event loop open for this to actually work
|
|
// since destroy hooks are triggered in unrefed Immediates
|
|
setImmediate(() => {
|
|
hook.disable();
|
|
});
|
|
});
|
|
}, (cb) => {
|
|
setImmediate(() => {
|
|
process.nextTick(() => {
|
|
cb();
|
|
|
|
// We need to keep the event loop open for this to actually work
|
|
// since destroy hooks are triggered in unrefed Immediates
|
|
setImmediate(() => {
|
|
hook.disable();
|
|
assert.strictEqual(fnsToTest.length, 0);
|
|
});
|
|
});
|
|
});
|
|
}];
|
|
|
|
const hook = async_hooks.createHook({
|
|
before: common.mustNotCall(),
|
|
after: common.mustCall(() => {}, 3),
|
|
destroy: common.mustCall(() => {
|
|
hook.disable();
|
|
nextTest();
|
|
}, 3)
|
|
});
|
|
|
|
nextTest();
|
|
|
|
function nextTest() {
|
|
if (fnsToTest.length > 0) {
|
|
fnsToTest.shift()(common.mustCall(() => {
|
|
hook.enable();
|
|
}));
|
|
}
|
|
}
|