test: do not swallow uncaughtException errors in exit code tests

PR-URL: #54039 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: nektro

test/common/process-exit-code-cases.js

@@ -62,23 +62,25 @@ function getTestCases(isWorker = false){
 cases.push({func: changeCodeInsideExit,result: 99});
 
 functionzeroExitWithUncaughtHandler(){
+constnoop=()=>{};
 process.on('exit',(code)=>{
-assert.strictEqual(process.exitCode,0);
+process.off('uncaughtException',noop);
+assert.strictEqual(process.exitCode,undefined);
 assert.strictEqual(code,0);
 });
-process.on('uncaughtException',()=>{});
+process.on('uncaughtException',noop);
 thrownewError('ok');
 }
 cases.push({func: zeroExitWithUncaughtHandler,result: 0});
 
 functionchangeCodeInUncaughtHandler(){
+constmodifyExitCode=()=>{process.exitCode=97;};
 process.on('exit',(code)=>{
+process.off('uncaughtException',modifyExitCode);
 assert.strictEqual(process.exitCode,97);
 assert.strictEqual(code,97);
 });
-process.on('uncaughtException',()=>{
-process.exitCode=97;
-});
+process.on('uncaughtException',modifyExitCode);
 thrownewError('ok');
 }
 cases.push({func: changeCodeInUncaughtHandler,result: 97});