Previously, require.resolve() called Module._resolveFilename() directly, bypassing any resolve hooks registered via module.registerHooks(). This patch fixes that. PR-URL: https://github.com/nodejs/node/pull/62028 Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
29 lines
929 B
JavaScript
29 lines
929 B
JavaScript
'use strict';
|
|
|
|
// This tests that require.resolve() in an imported CJS file goes through
|
|
// resolve hooks registered via module.registerHooks().
|
|
|
|
const common = require('../common');
|
|
const assert = require('assert');
|
|
const { registerHooks } = require('module');
|
|
const fixtures = require('../common/fixtures');
|
|
const { pathToFileURL } = require('url');
|
|
|
|
const redirectedPath = fixtures.path('module-hooks', 'redirected-assert.js');
|
|
|
|
const hook = registerHooks({
|
|
resolve: common.mustCall((specifier, context, nextResolve) => {
|
|
if (specifier === 'test-require-resolve-hook-target') {
|
|
return {
|
|
url: pathToFileURL(redirectedPath).href,
|
|
shortCircuit: true,
|
|
};
|
|
}
|
|
return nextResolve(specifier, context);
|
|
}, 2),
|
|
});
|
|
|
|
import('../fixtures/module-hooks/require-resolve-caller.js').then(common.mustCall((ns) => {
|
|
assert.strictEqual(ns.default.resolved, redirectedPath);
|
|
hook.deregister();
|
|
}));
|