Improve error message for missing parser

eslint · Feb 13, 2019 · 7051641 · 7051641
1 parent 75b6907
commit 7051641
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/lib/config/config-file.js b/lib/config/config-file.js
@@ -481,11 +481,10 @@ function loadObject(configObject, configContext, configPath, configFullName) {
             if (err.code === "MODULE_NOT_FOUND" && config.parser === "espree") {
                 config.parser = require.resolve("espree");
             } else {
+                err.message += `\nFailed to resolve parser '${config.parser}' declared in '${configPath}'.`;
                 throw err;
             }
         }
-
-        // FIXME: add exception for "espree", add appropriate error message
     }
 
     const ruleMap = configContext.ruleMap;

diff --git a/tests/lib/config/config-file.js b/tests/lib/config/config-file.js
@@ -141,7 +141,7 @@ describe("ConfigFile", () => {
                     extends: "plugin:enable-nonexistent-parser/bar",
                     rules: { eqeqeq: 2 }
                 }, configContext, "/whatever");
-            }, /Cannot find module 'nonexistent-parser'/);
+            }, /Failed to resolve parser 'nonexistent-parser' declared in '[\w-/.:\\]+'.\nReferenced from: \/whatever/);
         });
 
         it("should fall back to default parser when a config called 'espree' is not found", () => {