Merge pull request #384 from jamestalmage/hide-annoying-error-printouts

Hide annoying error print outs.

Author: sindresorhus

lib/fork.js

@@ -12,7 +12,8 @@ module.exports = function (file, opts) {
 
 	var ps = childProcess.fork(path.join(__dirname, 'test-worker.js'), [JSON.stringify(opts)], {
 		cwd: path.dirname(file),
-		stdio: ['ignore', process.stderr, process.stderr]
+		stdio: ['ignore', process.stderr, process.stderr],
+		silent: opts.silent
 	});
 
 	var relFile = path.relative('.', file);

package.json

@@ -122,6 +122,7 @@
   "devDependencies": {
     "coveralls": "^2.11.4",
     "delay": "^1.3.0",
+    "get-stream": "^1.1.0",
     "signal-exit": "^2.1.2",
     "sinon": "^1.17.2",
     "source-map-fixtures": "^0.4.0",

test/cli.js

@@ -1,6 +1,8 @@
 'use strict';
 var childProcess = require('child_process');
 var test = require('tap').test;
+global.Promise = require('bluebird');
+var getStream = require('get-stream');
 
 function execCli(args, cb) {
 	if (!Array.isArray(args)) {
@@ -13,10 +15,34 @@ function execCli(args, cb) {
 		env.AVA_APPVEYOR = 1;
 	}
 
-	childProcess.execFile(process.execPath, ['../cli.js'].concat(args), {
-		cwd: __dirname,
-		env: env
-	}, cb);
+	var stdout;
+	var stderr;
+
+	var processPromise = new Promise(function (resolve) {
+		var child = childProcess.spawn(process.execPath, ['../cli.js'].concat(args), {
+			cwd: __dirname,
+			env: env,
+			stdio: [null, 'pipe', 'pipe']
+		});
+
+		child.on('close', function (code, signal) {
+			if (code) {
+				var err = new Error('test-worker exited with a non-zero exit code: ' + code);
+				err.code = code;
+				err.signal = signal;
+				resolve(err);
+				return;
+			}
+			resolve(code);
+		});
+
+		stdout = getStream(child.stdout);
+		stderr = getStream(child.stderr);
+	});
+
+	Promise.all([processPromise, stdout, stderr]).then(function (args) {
+		cb.apply(null, args);
+	});
 }
 
 test('throwing a named function will report the to the console', function (t) {

test/fork.js

@@ -36,7 +36,7 @@ test('resolves promise with tests info', function (t) {
 test('rejects on error and streams output', function (t) {
 	t.plan(2);
 
-	fork(fixture('broken.js'))
+	fork(fixture('broken.js'), {silent: true})
 		.run()
 		.catch(function (err) {
 			t.ok(err);