log the file causing the problem when an initializer is not a class

actionhero · Oct 18, 2017 · f52ffdb · f52ffdb
1 parent 300bc49
commit f52ffdb
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 3 deletions.
diff --git a/classes/process.js b/classes/process.js
@@ -61,7 +61,14 @@ module.exports = class Process {
     ].forEach(async (file) => {
       delete require.cache[require.resolve(file)]
       const InitializerClass = require(file)
-      const initializer = new InitializerClass()
+      let initializer
+
+      try {
+        initializer = new InitializerClass()
+      } catch (error) {
+        this.fatalError(error, file)
+      }
+
       try {
         initializer.validate()
         await initializer.initialize(api)
@@ -97,7 +104,13 @@ module.exports = class Process {
 
       delete require.cache[require.resolve(file)]
       const InitializerClass = require(file)
-      const initializer = new InitializerClass()
+      let initializer
+
+      try {
+        initializer = new InitializerClass()
+      } catch (error) {
+        this.fatalError(error, file)
+      }
 
       // check if initializer already exists (exclude utils and config)
       if (

diff --git a/tutorials/running-actionhero.md b/tutorials/running-actionhero.md
@@ -140,7 +140,7 @@ ActionHero is not designed to function when installed globally.  Do not install
   "name": "my ActionHero project",
   "scripts": {
     "start" : "actionhero start",
-    "actionhero" : "actionhero",
+    "help" : "actionhero help",
     "pretest": "standard",
     "test" : "cross-env NODE_ENV=test mocha"
   }