module: fix addition of filetypes, if require ends with ..

lib/module.js

@@ -167,22 +167,23 @@ Module._findPath = function(request, paths) {
  // For each path
  for (var i = 0, PL = paths.length; i < PL; i++) {
  var basePath = path.resolve(paths[i], request);
+ var stats = statPath(basePath);
  var filename;
 
  if (!trailingSlash) {
  // try to join the request to the path
  filename = tryFile(basePath);
 
- if (!filename && !trailingSlash) {
+ if (!filename && stats && stats.isDirectory()) {
+ filename = tryPackage(basePath, exts);
+ }
+
+ if (!filename) {
  // try it with each of the extensions
  filename = tryExtensions(basePath, exts);
  }
  }
 
- if (!filename) {
- filename = tryPackage(basePath, exts);
- }
-
  if (!filename) {
  // try it with each of the extensions at "index"
  filename = tryExtensions(path.resolve(basePath, 'index'), exts);

test/fixtures/json-with-directory-name-module/module-stub.json

@@ -0,0 +1 @@
+{"rocko": "artischocko"}

test/fixtures/json-with-directory-name-module/module-stub/index.js

@@ -0,0 +1 @@
+module.exports = "hello from module-stub!"

test/fixtures/json-with-directory-name-module/module-stub/package.json

@@ -0,0 +1,7 @@
+{
+ "version": "1.1.12",
+ "name": "module-stub",
+ "main": "./index.js",
+ "description": "A stub for node tests",
+ "author": "Robert Kowalski <rok@kowalski.gd>"
+}

test/fixtures/json-with-directory-name-module/module-stub/test/tap/test.js

@@ -0,0 +1 @@
+module.exports = require('../..');

test/simple/test-require-extensions-same-filename-as-dir.js

@@ -0,0 +1,29 @@
+// Copyright Joyent, Inc. and other Node contributors.
+//
+// Permission is hereby granted, free of charge, to any person obtaining a
+// copy of this software and associated documentation files (the
+// "Software"), to deal in the Software without restriction, including
+// without limitation the rights to use, copy, modify, merge, publish,
+// distribute, sublicense, and/or sell copies of the Software, and to permit
+// persons to whom the Software is furnished to do so, subject to the
+// following conditions:
+//
+// The above copyright notice and this permission notice shall be included
+// in all copies or substantial portions of the Software.
+//
+// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
+// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN
+// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
+// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR
+// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE
+// USE OR OTHER DEALINGS IN THE SOFTWARE.
+
+var common = require('../common');
+var assert = require('assert');
+
+var content = require(common.fixturesDir +
+ '/json-with-directory-name-module/module-stub/test/tap/test.js');
+
+assert.ok(content.rocko !== 'artischocko');
+assert.equal(content, 'hello from module-stub!');