test: remove unnecessary process.exit calls from test files

nodejs · nodejs-github-bot · Mar 1, 2026 · Feb 27, 2026 · Feb 27, 2026 · Feb 28, 2026
commit d0a22f912faa8c272575e0baadf5733de7bbbea8
diff --git a/benchmark/buffers/buffer-transcode.js b/benchmark/buffers/buffer-transcode.js
@@ -3,12 +3,10 @@ const common = require('../common.js');
 const assert = require('node:assert');
 const buffer = require('node:buffer');
 
-const hasIntl = !!process.config.variables.v8_enable_i18n_support;
 const encodings = ['latin1', 'ascii', 'ucs2', 'utf8'];
 
-if (!hasIntl) {
-  console.log('Skipping: `transcode` is only available on platforms that support i18n`');
-  process.exit(0);
+if (!common.hasIntl) {
+  common.skip('`transcode` is only available on platforms that support i18n`');
 }
 
 const bench = common.createBenchmark(main, {

diff --git a/test/parallel/test-child-process-fork-advanced-header-serialization.js b/test/parallel/test-child-process-fork-advanced-header-serialization.js
@@ -13,11 +13,9 @@ if (process.argv[2] === 'child-buffer') {
     v & 0xFF,
   ]);
   const fd = process.channel?.fd;
-  if (fd === undefined) {
-    // skip test
-    process.exit(0);
+  if (fd !== undefined) {
+    fs.writeSync(fd, payload);
   }
-  fs.writeSync(fd, payload);
   return;
 }
 

diff --git a/test/parallel/test-child-process-internal.js b/test/parallel/test-child-process-internal.js
@@ -34,9 +34,6 @@ if (process.argv[2] === 'child') {
 
   // Send internal message
   process.send(internal);
-
-  process.exit(0);
-
 } else {
 
   const fork = require('child_process').fork;

diff --git a/test/parallel/test-child-process-silent.js b/test/parallel/test-child-process-silent.js
@@ -20,7 +20,7 @@
 // USE OR OTHER DEALINGS IN THE SOFTWARE.
 
 'use strict';
-require('../common');
+const common = require('../common');
 const assert = require('assert');
 const childProcess = require('child_process');
 
@@ -44,9 +44,7 @@ if (process.argv[2] === 'pipe') {
   // Allow child process to self terminate
   child.disconnect();
 
-  child.on('exit', function() {
-    process.exit(0);
-  });
+  child.on('exit', common.mustCall());
 
 } else {
   // Testcase | start primary && child IPC test

diff --git a/test/parallel/test-child-process-spawnsync-input.js b/test/parallel/test-child-process-spawnsync-input.js
@@ -65,7 +65,6 @@ if (process.argv.includes('spawnchild')) {
       checkSpawnSyncRet(ret);
       break;
   }
-  process.exit(0);
   return;
 }
 

diff --git a/test/parallel/test-child-process-stdout-flush-exit.js b/test/parallel/test-child-process-stdout-flush-exit.js
@@ -36,7 +36,6 @@ if (process.argv[2] === 'child') {
     console.log('filler');
   }
   console.log('goodbye');
-  process.exit(0);
 } else {
   // parent process
   const spawn = require('child_process').spawn;

diff --git a/test/parallel/test-cli-eval.js b/test/parallel/test-cli-eval.js
@@ -35,7 +35,7 @@ const fixtures = require('../common/fixtures');
 
 if (process.argv.length > 2) {
   console.log(process.argv.slice(2).join(' '));
-  process.exit(0);
+  return;
 }
 
 // Assert that nothing is written to stdout.

diff --git a/test/parallel/test-inspector-port-zero-cluster.js b/test/parallel/test-inspector-port-zero-cluster.js
@@ -1,3 +1,3 @@
 // Flags: --inspect=0
 'use strict';
 const common = require('../common');
@@ -50,6 +50,4 @@
         console.error(err);
         process.exit(1);
       });
-} else {
-  process.exit(0);
 }
diff --git a/test/parallel/test-net-listen-twice.js b/test/parallel/test-net-listen-twice.js
@@ -19,7 +19,7 @@ if (cluster.isPrimary) {
     server.listen();
   } catch (e) {
     console.error(e);
-    process.exit(0);
+    return;
   }
   let i = 0;
   process.on('internalMessage', (msg) => {

diff --git a/test/parallel/test-permission-allow-child-process-cli.js b/test/parallel/test-permission-allow-child-process-cli.js
@@ -24,7 +24,7 @@ if (process.argv[2] === 'child') {
     code: 'ERR_ACCESS_DENIED',
     permission: 'FileSystemWrite',
   }));
-  process.exit(0);
+  return;
 }
 
 // Guarantee the initial state

diff --git a/test/parallel/test-permission-allow-worker-cli.js b/test/parallel/test-permission-allow-worker-cli.js
@@ -6,7 +6,7 @@ const assert = require('assert');
 const { isMainThread, Worker } = require('worker_threads');
 
 if (!isMainThread) {
-  process.exit(0);
+  return;
 }
 
 // Guarantee the initial state

diff --git a/test/parallel/test-permission-child-process-cli.js b/test/parallel/test-permission-child-process-cli.js
@@ -12,7 +12,7 @@ const assert = require('assert');
 const childProcess = require('child_process');
 
 if (process.argv[2] === 'child') {
-  process.exit(0);
+  return;
 }
 
 // Guarantee the initial state

diff --git a/test/parallel/test-process-env.js b/test/parallel/test-process-env.js
@@ -32,7 +32,7 @@ if (process.argv[2] === 'you-are-the-child') {
   assert.strictEqual(process.env[42], 'forty-two');
   const has = Object.hasOwn(process.env, 'hasOwnProperty');
   assert.strictEqual(has, true);
-  process.exit(0);
+  return;
 }
 
 {

diff --git a/test/parallel/test-runner-run-files-undefined.mjs b/test/parallel/test-runner-run-files-undefined.mjs
@@ -6,8 +6,7 @@ import { fork } from 'node:child_process';
 import assert from 'node:assert';
 
 if (common.hasCrypto) {
-  console.log('1..0 # Skipped: no crypto');
-  process.exit(0);
+  common.skip('no crypto');
 }
 
 if (process.env.CHILD === 'true') {

diff --git a/test/parallel/test-trace-exit.js b/test/parallel/test-trace-exit.js
@@ -7,26 +7,22 @@ const { Worker, isMainThread, workerData } = require('worker_threads');
 
 const variant = process.argv[process.argv.length - 1];
 switch (true) {
-  case variant === 'main-thread': {
-    return;
-  }
-  case variant === 'main-thread-exit': {
+  case isMainThread ? variant === 'main-thread-exit' : workerData === 'worker-thread-exit':
     return process.exit(0);
-  }
   case variant.startsWith('worker-thread'): {
     const worker = new Worker(__filename, { workerData: variant });
     worker.on('error', common.mustNotCall());
     worker.on('exit', common.mustCall((code) => {
       assert.strictEqual(code, 0);
     }));
-    return;
   }
-  case !isMainThread: {
-    if (workerData === 'worker-thread-exit') {
-      process.exit(0);
-    }
+  // eslint-disable-next-line no-fallthrough
+  case variant === 'main-thread':
+  // eslint-disable-next-line no-fallthrough
+  case variant === 'main-thread-exit':
+  // eslint-disable-next-line no-fallthrough
+  case !isMainThread:
     return;
-  }
 }
 
 (async function() {

diff --git a/test/parallel/test-worker-safe-getters.js b/test/parallel/test-worker-safe-getters.js
@@ -29,6 +29,4 @@ if (isMainThread) {
     assert.strictEqual(w.stdout, stdout);
     assert.strictEqual(w.stderr, stderr);
   }));
-} else {
-  process.exit(0);
 }
diff --git a/test/parallel/test-worker-stdio-flush-inflight.js b/test/parallel/test-worker-stdio-flush-inflight.js
@@ -20,5 +20,4 @@ if (isMainThread) {
   process.stdout.write('hello');
   process.stdout.write(' ');
   process.stdout.write('world');
-  process.exit(0);
 }
diff --git a/test/parallel/test-worker-thread-name.js b/test/parallel/test-worker-thread-name.js
@@ -8,7 +8,6 @@ const name = 'test-worker-thread-name';
 
 if (workerData?.isWorker) {
   assert.strictEqual(threadName, name);
-  process.exit(0);
 } else {
   const w = new Worker(__filename, { name, workerData: { isWorker: true } });
   assert.strictEqual(w.threadName, name);
-Original file line number
+Diff line change
@@ Expand Up / @@ -65,7 +65,6 @@ if (process.argv.includes('spawnchild')) { @@
           checkSpawnSyncRet(ret);
           break;
       }
-      process.exit(0);
       return;
     }
@@ Expand Down @@