fix #172: tests processing should not stop when uncaught exception occurs

Author: 1999

test/index.sh

@@ -209,6 +209,10 @@ echo 'TESTCASE: unhandled rejections should not force subprocess to exit'
 test test/q-promises/index.js
 showTestResult $?
 
+echo 'TESTCASE: uncaught exceptions should not force subprocess to exit'
+test test/uncaught-exception/index.js
+showTestResult $?
+
 echo "Passes: $PASSES Failes: $FAILES"
 
 if [ $FAILES -gt 0 ]; then

test/uncaught-exception/README

@@ -0,0 +1 @@
+Make sure that uncaught exceptions do not stop executing the tests

test/uncaught-exception/index.js

@@ -0,0 +1,25 @@
+#!/usr/bin/env node
+
+const assert = require('assert');
+const sinon = require('sinon');
+
+const MochaParallelTests = require('../../dist/main/mocha').default;
+const { setProcessExitListeners } = require('../../dist/util');
+const SilentReporter = require('../util/silent-reporter');
+
+setProcessExitListeners();
+
+const onRunnerEnd = sinon.fake();
+const onTestEnd = sinon.fake();
+const mocha = new MochaParallelTests;
+mocha.reporter(SilentReporter);
+mocha.addFile(`${__dirname}/index.spec.js`);
+
+mocha.run()
+    .on('end', onRunnerEnd)
+    .on('test end', onTestEnd);
+
+process.on('exit', () => {
+    assert.strictEqual(onRunnerEnd.callCount, 1, `runner "end" event occured wrong number of times: ${onRunnerEnd.callCount}`);
+    assert.strictEqual(onTestEnd.callCount, 4, `runner "test end" event occured wrong number of times: ${onTestEnd.callCount}`);
+});

test/uncaught-exception/index.spec.js

@@ -0,0 +1,20 @@
+describe('suite', function () {
+    it('case 1', function () {
+        throw new Error('foo');
+    });
+
+    // eslint-disable-next-line no-unused-vars
+    it('case 2', function (done) {
+        setTimeout(() => {
+            throw new Error('foo');
+        }, 100);
+    });
+
+    it('case 3', () => {});
+
+    it('case 4', () => {
+        return new Promise(() => {
+            throw new Error('foo');
+        });
+    });
+});