Ensure AVA exits with code 1 after an unexpected process.exit() in a test worker

novemberborn · novemberborn · commit cc8b8397247a · 2024-01-11T22:33:57.000Z
It prints the error, so it should fail as such.
diff --git a/lib/run-status.js b/lib/run-status.js
@@ -38,6 +38,7 @@ export default class RunStatus extends Emittery {
 			timeouts: 0,
 			todoTests: 0,
 			uncaughtExceptions: 0,
+			unexpectedProcessExits: 0,
 			unhandledRejections: 0,
 		};
 	}
@@ -56,6 +57,7 @@ export default class RunStatus extends Emittery {
 			skippedTests: 0,
 			todoTests: 0,
 			uncaughtExceptions: 0,
+			unexpectedProcessExits: 0,
 			unhandledRejections: 0,
 			...stats,
 		});
@@ -167,6 +169,8 @@ export default class RunStatus extends Emittery {
 			}
 
 			case 'process-exit': {
+				stats.unexpectedProcessExits++;
+				fileStats.unexpectedProcessExits++;
 				event.pendingTests = this.pendingTests;
 				event.pendingTestsLogs = this.pendingTestsLogs;
 				this.pendingTests = new Map();
@@ -237,6 +241,7 @@ export default class RunStatus extends Emittery {
 			|| this.stats.sharedWorkerErrors > 0
 			|| this.stats.timeouts > 0
 			|| this.stats.uncaughtExceptions > 0
+			|| this.stats.unexpectedProcessExits > 0
 			|| this.stats.unhandledRejections > 0
 		) {
 			return 1;
