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>
29 lines
928 B
JavaScript
29 lines
928 B
JavaScript
'use strict';
|
|
|
|
// This tests that require.resolve() with the paths option work transparently
|
|
// when resolve hooks are registered via module.registerHooks().
|
|
|
|
require('../common');
|
|
const assert = require('assert');
|
|
const path = require('path');
|
|
const { registerHooks } = require('module');
|
|
const fixtures = require('../common/fixtures');
|
|
|
|
const nodeModules = path.join(fixtures.path(), 'node_modules');
|
|
const resolveCallCount = [];
|
|
|
|
const hook = registerHooks({
|
|
resolve(specifier, context, nextResolve) {
|
|
if (specifier === 'bar') {
|
|
resolveCallCount.push(specifier);
|
|
}
|
|
return nextResolve(specifier, context);
|
|
},
|
|
});
|
|
|
|
// require.resolve with paths option should go through hooks and resolve correctly.
|
|
const resolved = require.resolve('bar', { paths: [fixtures.path()] });
|
|
assert.strictEqual(resolved, path.join(nodeModules, 'bar.js'));
|
|
assert.deepStrictEqual(resolveCallCount, ['bar']);
|
|
|
|
hook.deregister();
|