Reject test runners when non-zero exit code (#269)

neutrinojs · Jun 19, 2017 · 1435477 · 1435477
1 parent d957604
commit 1435477
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/packages/neutrino-preset-karma/index.js b/packages/neutrino-preset-karma/index.js
@@ -39,12 +39,12 @@ module.exports = (neutrino, opts = {}) => {
     }
   };
 
-  neutrino.on('test', ({ files, watch }) => new Promise(resolve =>
+  neutrino.on('test', ({ files, watch }) => new Promise((resolve, reject) =>
     new Server(merge.all([
       opts.override ? opts : merge(defaults, opts),
       { singleRun: !watch, autoWatch: watch, webpack: omit(['plugins'], neutrino.config.toConfig()) },
       files && files.length ? { files } : {}
-    ]), resolve)
+    ]), exitCode => (exitCode !== 0 ? reject() : resolve()))
     .start()
   ));
 };
diff --git a/packages/neutrino-preset-mocha/src/mocha.js b/packages/neutrino-preset-mocha/src/mocha.js
@@ -3,7 +3,7 @@ const toParam = require('change-case').paramCase;
 
 let proc;
 
-module.exports = (mochaOpts = {}, babelOpts = {}, files = []) => new Promise((resolve) => {
+module.exports = (mochaOpts = {}, babelOpts = {}, files = []) => new Promise((resolve, reject) => {
   if (proc) {
     proc.kill();
   }
@@ -26,5 +26,5 @@ module.exports = (mochaOpts = {}, babelOpts = {}, files = []) => new Promise((re
     stdio: 'inherit'
   });
 
-  proc.on('close', resolve);
+  proc.on('close', code => (code !== 0 ? reject() : resolve()));
 });