ea2004a2ac
This aligns the behavior of synchronous hooks with asynchronous hooks by allowing omission of the context parameter in the invocation of next hooks. The contexts are merged along the chain. PR-URL: https://github.com/nodejs/node/pull/57056 Fixes: https://github.com/nodejs/node/issues/57030 Reviewed-By: Jacob Smith <jacob@frende.me> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
32 lines
967 B
JavaScript
32 lines
967 B
JavaScript
// Test that the context parameter will be merged in multiple load hooks.
|
|
|
|
import * as common from '../common/index.mjs';
|
|
import assert from 'node:assert';
|
|
import { registerHooks } from 'node:module';
|
|
|
|
const hook1 = registerHooks({
|
|
load: common.mustCall(function(url, context, nextLoad) {
|
|
assert.strictEqual(context.testProp, 'custom'); // It should be merged from hook 2 and 3.
|
|
return nextLoad(url, context);
|
|
}, 1),
|
|
});
|
|
|
|
const hook2 = registerHooks({
|
|
load: common.mustCall(function(url, context, nextLoad) {
|
|
assert.strictEqual(context.testProp, 'custom'); // It should be merged from hook 3.
|
|
return nextLoad(url); // Omit the context.
|
|
}, 1),
|
|
});
|
|
|
|
const hook3 = registerHooks({
|
|
load: common.mustCall(function(url, context, nextLoad) {
|
|
return nextLoad(url, { testProp: 'custom' }); // Add a custom property
|
|
}, 1),
|
|
});
|
|
|
|
await import('../fixtures/es-modules/message.mjs');
|
|
|
|
hook3.deregister();
|
|
hook2.deregister();
|
|
hook1.deregister();
|