[New] add searched extensions to error messages

Author: Alexander Wenzowski

lib/async.js

@@ -3,6 +3,7 @@ var fs = require('fs');
 var path = require('path');
 var caller = require('./caller.js');
 var nodeModulesPaths = require('./node-modules-paths.js');
+var format = require('util').format;
 
 module.exports = function resolve(x, options, callback) {
     var cb = callback;
@@ -43,7 +44,7 @@ module.exports = function resolve(x, options, callback) {
         else if (n) cb(null, n, pkg);
         else if (core[x]) return cb(null, x);
         else {
-            var moduleError = new Error("Cannot find module '" + x + "' from '" + y + "'");
+            var moduleError = new Error(format("Cannot find module '%s' from '%s' with extension in %j", x, y, extensions));
             moduleError.code = 'MODULE_NOT_FOUND';
             cb(moduleError);
         }
@@ -56,7 +57,7 @@ module.exports = function resolve(x, options, callback) {
             if (err) cb(err);
             else if (d) cb(null, d, pkg);
             else {
-                var moduleError = new Error("Cannot find module '" + x + "' from '" + y + "'");
+                var moduleError = new Error(format("Cannot find module '%s' from '%s' with extension in %j", x, y, extensions));
                 moduleError.code = 'MODULE_NOT_FOUND';
                 cb(moduleError);
             }

lib/sync.js

@@ -3,6 +3,7 @@ var fs = require('fs');
 var path = require('path');
 var caller = require('./caller.js');
 var nodeModulesPaths = require('./node-modules-paths.js');
+var format = require('util').format;
 
 module.exports = function (x, options) {
     var opts = options || {};
@@ -34,7 +35,7 @@ module.exports = function (x, options) {
 
     if (core[x]) return x;
 
-    var err = new Error("Cannot find module '" + x + "' from '" + y + "'");
+    var err = new Error(format("Cannot find module '%s' from '%s' with extension in %j", x, y, extensions));
     err.code = 'MODULE_NOT_FOUND';
     throw err;
 

test/mock.js

@@ -33,12 +33,12 @@ test('mock', function (t) {
     });
 
     resolve('baz', opts('/foo/bar'), function (err, res) {
-        t.equal(err.message, "Cannot find module 'baz' from '" + path.resolve('/foo/bar') + "'");
+        t.equal(err.message, "Cannot find module 'baz' from '" + path.resolve('/foo/bar') + "' with extension in [\".js\"]");
         t.equal(err.code, 'MODULE_NOT_FOUND');
     });
 
     resolve('../baz', opts('/foo/bar'), function (err, res) {
-        t.equal(err.message, "Cannot find module '../baz' from '" + path.resolve('/foo/bar') + "'");
+        t.equal(err.message, "Cannot find module '../baz' from '" + path.resolve('/foo/bar') + "' with extension in [\".js\"]");
         t.equal(err.code, 'MODULE_NOT_FOUND');
     });
 });
@@ -75,12 +75,12 @@ test('mock from package', function (t) {
     });
 
     resolve('baz', opts('/foo/bar'), function (err, res) {
-        t.equal(err.message, "Cannot find module 'baz' from '" + path.resolve('/foo/bar') + "'");
+        t.equal(err.message, "Cannot find module 'baz' from '" + path.resolve('/foo/bar') + "' with extension in [\".js\"]");
         t.equal(err.code, 'MODULE_NOT_FOUND');
     });
 
     resolve('../baz', opts('/foo/bar'), function (err, res) {
-        t.equal(err.message, "Cannot find module '../baz' from '" + path.resolve('/foo/bar') + "'");
+        t.equal(err.message, "Cannot find module '../baz' from '" + path.resolve('/foo/bar') + "' with extension in [\".js\"]");
         t.equal(err.code, 'MODULE_NOT_FOUND');
     });
 });

test/resolver.js

@@ -31,7 +31,7 @@ test('async foo', function (t) {
     });
 
     resolve('foo', { basedir: dir }, function (err) {
-        t.equal(err.message, "Cannot find module 'foo' from '" + path.resolve(dir) + "'");
+        t.equal(err.message, "Cannot find module 'foo' from '" + path.resolve(dir) + "' with extension in [\".js\"]");
         t.equal(err.code, 'MODULE_NOT_FOUND');
     });
 });
@@ -190,7 +190,7 @@ test('cup', function (t) {
     });
 
     resolve('./cup', { basedir: dir, extensions: ['.js'] }, function (err, res) {
-        t.equal(err.message, "Cannot find module './cup' from '" + path.resolve(dir) + "'");
+        t.equal(err.message, "Cannot find module './cup' from '" + path.resolve(dir) + "' with extension in [\".js\"]");
         t.equal(err.code, 'MODULE_NOT_FOUND');
     });
 });
@@ -231,12 +231,12 @@ test('other path', function (t) {
     });
 
     resolve('root', { basedir: dir }, function (err, res) {
-        t.equal(err.message, "Cannot find module 'root' from '" + path.resolve(dir) + "'");
+        t.equal(err.message, "Cannot find module 'root' from '" + path.resolve(dir) + "' with extension in [\".js\"]");
         t.equal(err.code, 'MODULE_NOT_FOUND');
     });
 
     resolve('zzz', { basedir: dir, paths: [otherDir] }, function (err, res) {
-        t.equal(err.message, "Cannot find module 'zzz' from '" + path.resolve(dir) + "'");
+        t.equal(err.message, "Cannot find module 'zzz' from '" + path.resolve(dir) + "' with extension in [\".js\"]");
         t.equal(err.code, 'MODULE_NOT_FOUND');
     });
 });