module: fix resolution of filename with trailing slash

A recent optimization of module loading performance [1] forgot to check that
extensions were set in a certain code path.

[1] ae18bbe

Fixes: #6214
PR-URL: #6215
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Brian White <[email protected]>

Author: targos

lib/module.js

@@ -170,6 +170,8 @@ Module._findPath = function(request, paths) {
     }
 
     if (!filename) {
+      if (exts === undefined)
+        exts = Object.keys(Module._extensions);
       filename = tryPackage(basePath, exts);
     }
 

test/fixtures/module-require/not-found/node_modules/module1/package.json

@@ -0,0 +1 @@
+require('module1/');

test/fixtures/module-require/not-found/trailingSlash.js

@@ -14,9 +14,21 @@ assert.throws(function() {
 
 // Requiring a module that does not exist should throw an
 // error with its `code` set to MODULE_NOT_FOUND
-assert.throws(function() {
-  require(common.fixturesDir + '/DOES_NOT_EXIST');
-}, function(e) {
-  assert.equal('MODULE_NOT_FOUND', e.code);
-  return true;
-});
+assertModuleNotFound('/DOES_NOT_EXIST');
+
+assertExists('/module-require/not-found/trailingSlash.js');
+assertExists('/module-require/not-found/node_modules/module1/package.json');
+assertModuleNotFound('/module-require/not-found/trailingSlash');
+
+function assertModuleNotFound(path) {
+  assert.throws(function() {
+    require(common.fixturesDir + path);
+  }, function(e) {
+    assert.strictEqual(e.code, 'MODULE_NOT_FOUND');
+    return true;
+  });
+}
+
+function assertExists(fixture) {
+  assert(common.fileExists(common.fixturesDir + fixture));
+}