test: add real world case for node_modules paths

Add a real world global node_modules path test case come from npm's dependency to make test more effective.

Author: hefangshi

lib/module.js

@@ -231,8 +231,11 @@ if (process.platform === 'win32'){
 varlast=from.length;
 for(vari=from.length-1;i>=0;--i){
 constcode=from.charCodeAt(i);
-// use colon as a split to add driver root node_modules
-// it's a little more tricky than posix version to avoid parse drive name.
+// The path segment separator check ('\' and '/') was used to get
+// node_modules path for every path segment.
+// Use colon as an extra condition since we can get node_modules
+// path for dirver root like 'C:\node_modules' and don't need to
+// parse driver name.
 if(code===92/*\*/||code===47/*/*/||code===58/*:*/){
 if(p!==nmLen)
 paths.push(from.slice(0,last)+'\\node_modules');
@@ -280,7 +283,8 @@ if (process.platform === 'win32'){
 }
 }
 }
-// append /node_modules since this approach didn't handle root path
+
+// Append /node_modules to handle root paths.
 paths.push('/node_modules');
 
 returnpaths;

test/parallel/test-module-nodemodulepaths.js

@@ -6,6 +6,21 @@ const _module = require('module');
 
 constcases={
 'WIN': [{
+file: 'C:\\Users\\hefangshi\\AppData\\Roaming\
+\\npm\\node_modules\\npm\\node_modules\\minimatch',
+expect: [
+'C:\\Users\\hefangshi\\AppData\\Roaming\
+\\npm\\node_modules\\npm\\node_modules\\minimatch\\node_modules',
+'C:\\Users\\hefangshi\\AppData\\Roaming\
+\\npm\\node_modules\\npm\\node_modules',
+'C:\\Users\\hefangshi\\AppData\\Roaming\\npm\\node_modules',
+'C:\\Users\\hefangshi\\AppData\\Roaming\\node_modules',
+'C:\\Users\\hefangshi\\AppData\\node_modules',
+'C:\\Users\\hefangshi\\node_modules',
+'C:\\Users\\node_modules',
+'C:\\node_modules'
+]
+},{
 file: 'C:\\Users\\Rocko Artischocko\\node_stuff\\foo',
 expect: [
 'C:\\Users\\Rocko Artischocko\\node_stuff\\foo\\node_modules',
@@ -36,6 +51,20 @@ Artischocko\\node_stuff\\foo_node_modules\\node_modules',
 ]
 }],
 'POSIX': [{
+file: '/usr/lib/node_modules/npm/node_modules/\
+node-gyp/node_modules/glob/node_modules/minimatch',
+expect: [
+'/usr/lib/node_modules/npm/node_modules/\
+node-gyp/node_modules/glob/node_modules/minimatch/node_modules',
+'/usr/lib/node_modules/npm/node_modules/\
+node-gyp/node_modules/glob/node_modules',
+'/usr/lib/node_modules/npm/node_modules/node-gyp/node_modules',
+'/usr/lib/node_modules/npm/node_modules',
+'/usr/lib/node_modules',
+'/usr/node_modules',
+'/node_modules'
+]
+},{
 file: '/usr/test/lib/node_modules/npm/foo',
 expect: [
 '/usr/test/lib/node_modules/npm/foo/node_modules',