module: replace default paths in require.resolve()

Prior to this commit, the default search paths would be included in the require.resolve() process, even if user specified paths were provided. This commit causes the default paths to be omitted by using a fake parent module. Refs: #5963 PR-URL: #17113 Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: cjihrig

lib/module.js

@@ -518,11 +518,14 @@ Module._resolveFilename = function(request, parent, isMain, options){
 
 if(typeofoptions==='object'&&options!==null&&
 Array.isArray(options.paths)){
+constfakeParent=newModule('',null);
+
 paths=[];
 
 for(vari=0;i<options.paths.length;i++){
 constpath=options.paths[i];
-constlookupPaths=Module._resolveLookupPaths(path,parent,true);
+fakeParent.paths=Module._nodeModulePaths(path);
+constlookupPaths=Module._resolveLookupPaths(request,fakeParent,true);
 
 if(!paths.includes(path))
 paths.push(path);

test/fixtures/resolve-paths/default/node_modules/dep/index.js

@@ -0,0 +1,21 @@
+'use strict';
+require('../../../common');
+constassert=require('assert');
+constpath=require('path');
+
+// By default, resolving 'dep' should return
+// fixturesDir/resolve-paths/default/node_modules/dep/index.js. By setting
+// the path to fixturesDir/resolve-paths/default, the 'default' directory
+// structure should be ignored.
+
+assert.strictEqual(
+require.resolve('dep'),
+path.join(__dirname,'node_modules','dep','index.js')
+);
+
+constpaths=[path.resolve(__dirname,'..','defined')];
+
+assert.strictEqual(
+require.resolve('dep',{ paths }),
+path.join(paths[0],'node_modules','dep','index.js')
+);

test/fixtures/resolve-paths/default/verify-paths.js

@@ -37,3 +37,4 @@ assert.strictEqual('path', require.resolve('path'));
 
 // Test configurable resolve() paths.
 require(fixtures.path('require-resolve.js'));
+require(fixtures.path('resolve-paths','default','verify-paths.js'));