dd400b9f12Previously, 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>
19 lines
499 B
JavaScript
19 lines
499 B
JavaScript
'use strict';
|
|
|
|
// This tests that require.resolve() falls through to default resolution
|
|
// when resolve hooks registered via module.registerHooks() don't override.
|
|
|
|
require('../common');
|
|
const assert = require('assert');
|
|
const { registerHooks } = require('module');
|
|
|
|
const hook = registerHooks({
|
|
resolve(specifier, context, nextResolve) {
|
|
return nextResolve(specifier, context);
|
|
},
|
|
});
|
|
|
|
const resolved = require.resolve('assert');
|
|
assert.strictEqual(resolved, 'assert');
|
|
|
|
hook.deregister();
|