delegate taking down the process to the bin file not the cli lib

clux · Mar 28, 2015 · f2fad27 · f2fad27
1 parent 8c680e7
commit f2fad27
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/bin.js b/bin.js
@@ -14,4 +14,9 @@ var argv = require('yargs')
   .help('h')
   .argv;
 
+process.on('uncaughtException', function (err) {
+  console.error(err.message);
+  process.exit(1);
+});
+
 require('./lib/cli').run(argv);
diff --git a/lib/cli.js b/lib/cli.js
@@ -4,15 +4,15 @@ var async = require('async');
 
 var handleError = function (err) {
   if (err) {
-    console.error(err.message);
-    process.exit(1);
+    throw err;
   }
 };
 
 var dryRunHandler = function (err, cmds) {
   handleError(err);
   console.log(JSON.stringify(cmds, null, " "));
 };
+
 var executeHandler = function (err, cmds) {
   handleError(err);
   // create one cp function cmd that execs and cbs to next in async series
@@ -27,10 +27,7 @@ var executeHandler = function (err, cmds) {
   });
 
   // exec commands synchronously
-  async.series(execs, function (err) {
-    handleError(err);
-    process.exit(0);
-  });
+  async.series(execs, handleError);
 };