Merge pull request #680 from sindresorhus/sigabrt

Reject test runs when forks exit with an unexpected signal

Author: novemberborn

lib/fork.js

@@ -78,11 +78,15 @@ module.exports = function (file, opts) {
 			send(ps, 'teardown');
 		});
 
-		ps.on('exit', function (code) {
-			if (code > 0 && code !== 143) {
+		ps.on('exit', function (code, signal) {
+			if (code > 0) {
 				return reject(new AvaError(relFile + ' exited with a non-zero exit code: ' + code));
 			}
 
+			if (code === null && signal) {
+				return reject(new AvaError(relFile + ' exited due to ' + signal));
+			}
+
 			if (results) {
 				resolve(results);
 			} else {

test/fixture/immediate-3-exit.js

@@ -0,0 +1 @@
+process.exit(3);

test/fork.js

@@ -60,6 +60,34 @@ test('exit after tests are finished', function (t) {
 		});
 });
 
+test('rejects promise if the process exits with a non-zero code', function (t) {
+	return fork(fixture('immediate-3-exit.js'))
+		.catch(function (err) {
+			t.is(err.name, 'AvaError');
+			t.is(err.message, path.join('test', 'fixture', 'immediate-3-exit.js') + ' exited with a non-zero exit code: 3');
+		});
+});
+
+test('rejects promise if the process exits without results', function (t) {
+	return fork(fixture('immediate-0-exit.js'))
+		.catch(function (err) {
+			t.is(err.name, 'AvaError');
+			t.is(err.message, 'Test results were not received from ' + path.join('test', 'fixture', 'immediate-0-exit.js'));
+		});
+});
+
+test('rejects promise if the process is killed', function (t) {
+	var forked = fork(fixture('es2015.js'));
+	return forked
+		.on('stats', function () {
+			this.kill('SIGKILL');
+		})
+		.catch(function (err) {
+			t.is(err.name, 'AvaError');
+			t.is(err.message, path.join('test', 'fixture', 'es2015.js') + ' exited due to SIGKILL');
+		});
+});
+
 test('fake timers do not break duration', function (t) {
 	fork(fixture('fake-timers.js'))
 		.run({})