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>
33 lines
1.0 KiB
JavaScript
33 lines
1.0 KiB
JavaScript
// Test that the context parameter will be merged in multiple resolve hooks.
|
|
|
|
import * as common from '../common/index.mjs';
|
|
import assert from 'node:assert';
|
|
import { registerHooks } from 'node:module';
|
|
|
|
const hook1 = registerHooks({
|
|
resolve: common.mustCall(function(specifier, context, nextResolve) {
|
|
assert.strictEqual(context.testProp, 'custom'); // It should be merged from hook 2 and 3.
|
|
const result = nextResolve(specifier, context);
|
|
return result;
|
|
}, 1),
|
|
});
|
|
|
|
const hook2 = registerHooks({
|
|
resolve: common.mustCall(function(specifier, context, nextResolve) {
|
|
assert.strictEqual(context.testProp, 'custom'); // It should be merged from hook 3.
|
|
return nextResolve(specifier); // Omit the context.
|
|
}, 1),
|
|
});
|
|
|
|
const hook3 = registerHooks({
|
|
resolve: common.mustCall(function(specifier, context, nextResolve) {
|
|
return nextResolve(specifier, { testProp: 'custom' }); // Add a custom property
|
|
}, 1),
|
|
});
|
|
|
|
await import('../fixtures/es-modules/message.mjs');
|
|
|
|
hook3.deregister();
|
|
hook2.deregister();
|
|
hook1.deregister();
|