Revert "Quit swallowing errors while trying to find modules (#185)"

This reverts commit 421468b.

Author: jdalton

src/ModuleCache.js

@@ -1,61 +1,60 @@
-import _ from 'lodash'
-import fs from 'fs'
-import glob from 'glob'
-import MapCache from './MapCache'
-import Module from 'module'
-import { normalizePath } from './util'
-import path from 'path'
+import _ from 'lodash';
+import fs from 'fs';
+import glob from 'glob';
+import MapCache from './MapCache';
+import Module from 'module';
+import { normalizePath } from './util';
+import path from 'path';
 
 /*----------------------------------------------------------------------------*/
 
 export default class ModuleCache extends MapCache {
   constructor(moduleRoot) {
-    super()
+    super();
 
-    moduleRoot = _.toString(moduleRoot)
-
-    if (! moduleRoot) {
-      return
+    moduleRoot = _.toString(moduleRoot);
+    if (!moduleRoot) {
+      return;
     }
-
-    const pkgPath = path.join(moduleRoot, 'package.json')
-    const pkgMain = fs.existsSync(pkgPath) && require(pkgPath).main || 'index.js'
-    const mainPath = normalizePath(path.dirname(path.resolve(moduleRoot, pkgMain)))
+    const pkgPath = path.join(moduleRoot, 'package.json');
+    const pkgMain = fs.existsSync(pkgPath) && require(pkgPath).main || 'index.js';
+    const mainPath = normalizePath(path.dirname(path.resolve(moduleRoot, pkgMain)));
 
     // Sort paths by the “main” entry first.
     const dirPaths = _.orderBy(glob.sync(path.join(moduleRoot, '**/'), {
       'ignore': path.join(moduleRoot, 'node_modules/**/')
-    }), dirPath => _.startsWith(dirPath, mainPath), ['desc'])
-
-    _.each(dirPaths, (dirPath) => {
-      const base = path.relative(moduleRoot, dirPath)
-      const filePaths = glob.sync(path.join(dirPath, '*.js'))
-      const pairs = _.map(filePaths, (filePath) => {
-        const name = path.basename(filePath, '.js')
-        return [name.toLowerCase(), name]
-      })
-      this.set(base, new MapCache(pairs))
-    })
+    }), dirPath => _.startsWith(dirPath, mainPath), ['desc']);
+
+    _.each(dirPaths, dirPath => {
+      const base = path.relative(moduleRoot, dirPath);
+      const filePaths = glob.sync(path.join(dirPath, '*.js'));
+      const pairs = _.map(filePaths, filePath => {
+        const name = path.basename(filePath, '.js');
+        return [name.toLowerCase(), name];
+      });
+      this.set(base, new MapCache(pairs));
+    });
   }
 
-  static resolve(id, from = process.cwd()) {
-    const dirs = path.dirname(Module._resolveFilename(id, _.assign(new Module, {
-      'paths': Module._nodeModulePaths(from)
-    }))).split(path.sep)
-
-    let { length } = dirs
-
-    while (length--) {
-      const dirSub  = dirs.slice(0, length + 1)
-      const dirPath = dirSub.join('/')
-      const pkgPath = path.join(dirPath, 'package.json')
-
-      if ((length && dirs[length - 1] === 'node_modules') ||
-          (fs.existsSync(pkgPath) && require(pkgPath).name === id)) {
-        return dirPath
+  static resolve(id, from=process.cwd()) {
+    try {
+      const dirs = path.dirname(Module._resolveFilename(id, _.assign(new Module, {
+        'paths': Module._nodeModulePaths(from)
+      }))).split(path.sep);
+
+      let { length } = dirs;
+      while (length--) {
+        const dirSub  = dirs.slice(0, length + 1);
+        const dirPath = dirSub.join('/');
+        const pkgPath = path.join(dirPath, 'package.json');
+
+        if ((length && dirs[length - 1] == 'node_modules') ||
+            (fs.existsSync(pkgPath) && require(pkgPath).name == id)) {
+          return dirPath;
+        }
       }
-    }
-
-    return dirs.join('/')
+      return dirs.join('/');
+    } catch (e) {}
+    return '';
   }
-}
+};