nodejs
diff --git a/‎lib/internal/main/test_runner.js‎
Lines changed: 4 additions & 2 deletions b/‎lib/internal/main/test_runner.js‎
Lines changed: 4 additions & 2 deletions
diff --git a/‎lib/internal/test_runner/harness.js‎
Lines changed: 4 additions & 2 deletions b/‎lib/internal/test_runner/harness.js‎
Lines changed: 4 additions & 2 deletions
diff --git a/‎lib/internal/test_runner/test.js‎
Lines changed: 3 additions & 3 deletions b/‎lib/internal/test_runner/test.js‎
Lines changed: 3 additions & 3 deletions
diff --git a/‎test/fixtures/test-runner/todo_exit_code.js‎
Lines changed: 15 additions & 0 deletions b/‎test/fixtures/test-runner/todo_exit_code.js‎
Lines changed: 15 additions & 0 deletions
diff --git a/‎test/parallel/test-runner-exit-code.js‎
Lines changed: 13 additions & 0 deletions b/‎test/parallel/test-runner-exit-code.js‎
Lines changed: 13 additions & 0 deletions
@@ -57,6 +57,8 @@ if (shardOption) {
 }
 
 run({ concurrency, inspectPort, watch: getOptionValue('--watch'), setup: setupTestReporters, shard })
-.once('test:fail', () => {
-  process.exitCode = 1;
+.on('test:fail', (data) => {
+  if (data.todo === undefined || data.todo === false) {
+    process.exitCode = 1;
+  }
 });
@@ -189,8 +189,10 @@ let reportersSetup;
 function getGlobalRoot() {
   if (!globalRoot) {
     globalRoot = createTestTree();
-    globalRoot.reporter.once('test:fail', () => {
-      process.exitCode = 1;
+    globalRoot.reporter.on('test:fail', (data) => {
+      if (data.todo === undefined || data.todo === false) {
+        process.exitCode = 1;
+      }
     });
     reportersSetup = setupTestReporters(globalRoot);
   }
 
@@ -311,8 +311,8 @@ class Test extends AsyncResource {
     this.harness = null; // Configured on the root test by the test harness.
     this.mock = null;
     this.cancelled = false;
-    this.skipped = !!skip;
-    this.isTodo = !!todo;
+    this.skipped = skip !== undefined && skip !== false;
+    this.isTodo = todo !== undefined && todo !== false;
     this.startTime = null;
     this.endTime = null;
     this.passed = false;
@@ -667,7 +667,7 @@ class Test extends AsyncResource {
         subtest.#cancel(pendingSubtestsError);
         subtest.postRun(pendingSubtestsError);
       }
-      if (!subtest.passed) {
+      if (!subtest.passed && !subtest.isTodo) {
         failed++;
       }
     }
 
@@ -0,0 +1,15 @@
+const { describe, test } = require('node:test');
+
+describe('suite should pass', () => {
+  test.todo('should fail without harming suite', () => {
+    throw new Error('Fail but not badly')
+  });
+});
+
+test.todo('should fail without effecting exit code', () => {
+  throw new Error('Fail but not badly')
+});
+
+test('empty string todo', { todo: '' }, () => {
+  throw new Error('Fail but not badly')
+});
@@ -50,6 +50,19 @@ if (process.argv[2] === 'child') {
   assert.strictEqual(child.status, 0);
   assert.strictEqual(child.signal, null);
 
+
+  child = spawnSync(process.execPath, [
+    '--test',
+    fixtures.path('test-runner', 'todo_exit_code.js'),
+  ]);
+  assert.strictEqual(child.status, 0);
+  assert.strictEqual(child.signal, null);
+  const stdout = child.stdout.toString();
+  assert.match(stdout, /# tests 3/);
+  assert.match(stdout, /# pass 0/);
+  assert.match(stdout, /# fail 0/);
+  assert.match(stdout, /# todo 3/);
+
   child = spawnSync(process.execPath, [__filename, 'child', 'fail']);
   assert.strictEqual(child.status, 1);
   assert.strictEqual(child.signal, null);
Original file line number	Diff line number	Diff line change
@@ -311,8 +311,8 @@ class Test extends AsyncResource {
311	311	this.harness = null; // Configured on the root test by the test harness.
312	312	this.mock = null;
313	313	this.cancelled = false;
314		- this.skipped = !!skip;
315		- this.isTodo = !!todo;
	314	+ this.skipped = skip !== undefined && skip !== false;
	315	+ this.isTodo = todo !== undefined && todo !== false;
316	316	this.startTime = null;
317	317	this.endTime = null;
318	318	this.passed = false;
@@ -667,7 +667,7 @@ class Test extends AsyncResource {
667	667	subtest.#cancel(pendingSubtestsError);
668	668	subtest.postRun(pendingSubtestsError);
669	669	}
670		- if (!subtest.passed) {
	670	+ if (!subtest.passed && !subtest.isTodo) {
671	671	failed++;
672	672	}
673	673	}