Fixes #182

src/Config.js

@@ -1,7 +1,16 @@
+const chalk = require("chalk");
 const TemplateConfig = require("./TemplateConfig");
+const EleventyErrorHandler = require("./EleventyErrorHandler");
 const debug = require("debug")("Eleventy:Config");
 
 debug("Setting up global TemplateConfig.");
-let config = new TemplateConfig();
+let config;
+try {
+  config = new TemplateConfig();
+} catch (e) {
+  console.log("\n" + chalk.red("Problem with Eleventy configuration: "));
+
+  EleventyErrorHandler.log(e);
+}
 
 module.exports = config;

src/Engines/Liquid.js

@@ -2,7 +2,7 @@ const LiquidLib = require("liquidjs");
 const TemplateEngine = require("./TemplateEngine");
 const lodashMerge = require("lodash.merge");
 const config = require("../Config");
-const debug = require("debug")("Eleventy:Liquid");
+// const debug = require("debug")("Eleventy:Liquid");
 
 class Liquid extends TemplateEngine {
   constructor(name, inputDir) {

src/TemplateConfig.js

@@ -2,9 +2,13 @@ const chalk = require("chalk");
 const fs = require("fs-extra");
 const lodashMerge = require("lodash.merge");
 const TemplatePath = require("./TemplatePath");
+const EleventyBaseError = require("./EleventyBaseError");
+const EleventyErrorHandler = require("./EleventyErrorHandler");
 const eleventyConfig = require("./EleventyConfig");
 const debug = require("debug")("Eleventy:TemplateConfig");
 
+class EleventyConfigError extends EleventyBaseError {}
+
 class TemplateConfig {
   constructor(customRootConfig, localProjectConfigPath) {
     this.overrides = {};
@@ -78,11 +82,10 @@ class TemplateConfig {
         localConfig = require(path);
         // debug( "localConfig require return value: %o", localConfig );
       } catch (err) {
-        // TODO if file does exist, rethrow the error or console.log the error (file has syntax problem)
-
-        // if file does not exist, return empty obj
-        localConfig = {};
-        debug(chalk.red("Problem getting localConfig file, %o"), err);
+        throw new EleventyConfigError(
+          `Error in your Eleventy config file '${path}'`,
+          err
+        );
       }
     } else {
       debug("Eleventy local project config file not found, skipping.");

test/TemplateConfigTest.js

@@ -219,3 +219,12 @@ test("libraryOverrides", t => {
   t.truthy(cfg.libraryOverrides.md);
   t.deepEqual(mdLib, cfg.libraryOverrides.md);
 });
+
+test("Properly throws error on missing module #182", t => {
+  t.throws(function() {
+    new TemplateConfig(
+      require("../config.js"),
+      "./test/stubs/broken-config.js"
+    );
+  });
+});

test/stubs/broken-config.js

@@ -0,0 +1,9 @@
+const missingModule = require("this-is-a-module-that-does-not-exist");
+
+module.exports = function(eleventyConfig) {
+  eleventyConfig.addFilter("cssmin", function(code) {
+    return missingModule(code);
+  });
+
+  return {};
+};