[Fix] resolve.sync should re-throw non `ENOENT errors.

Fixes #79.

Author: stefanpenner

lib/sync.js

@@ -8,7 +8,10 @@ module.exports = function (x, opts) {
     if (!opts) opts = {};
     var isFile = opts.isFile || function (file) {
         try { var stat = fs.statSync(file) }
-        catch (err) { if (err && err.code === 'ENOENT') return false }
+        catch (err) {
+          if (err && err.code === 'ENOENT') return false;
+          throw err;
+        }
         return stat.isFile() || stat.isFIFO();
     };
     var readFileSync = opts.readFileSync || fs.readFileSync;

test/resolver_sync.js

@@ -178,3 +178,29 @@ test('#25: node modules with the same name as node stdlib modules', function (t)
 
     t.end()
 });
+
+function stubStatSync(fn) {
+    var fs = require('fs')
+    var statSync = fs.statSync;
+    try {
+        fs.statSync = function () {
+          throw new EvalError('Unknown Error');
+        };
+        return fn();
+    } finally {
+       fs.statSync = statSync;
+    }
+}
+
+test('#79 - re-throw non ENOENT errors from stat', function (t) {
+    var dir = __dirname + '/resolver';
+
+    stubStatSync(function () {
+      t.throws(function () {
+        resolve.sync('foo', { basedir : dir });
+      }, /Unknown Error/);
+    })
+
+    t.end()
+});
+