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
@@ -58,6 +58,8 @@ if (shardOption) {
 }
 
 run({ concurrency, inspectPort, watch: getOptionValue('--watch'), setup: setupTestReporters, shard })
-.once('test:fail', () => {
-  process.exitCode = kGenericUserError;
+.on('test:fail', (data) => {
+  if (data.todo === undefined || data.todo === false) {
+    process.exitCode = kGenericUserError;
+  }
 });
@@ -191,8 +191,10 @@ let reportersSetup;
 function getGlobalRoot() {
   if (!globalRoot) {
     globalRoot = createTestTree();
-    globalRoot.reporter.once('test:fail', () => {
-      process.exitCode = kGenericUserError;
+    globalRoot.reporter.on('test:fail', (data) => {
+      if (data.todo === undefined || data.todo === false) {
+        process.exitCode = kGenericUserError;
+      }
     });
     reportersSetup = setupTestReporters(globalRoot);
   }
 
@@ -282,8 +282,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;
@@ -634,7 +634,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')
+});
@@ -47,6 +47,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
@@ -282,8 +282,8 @@ class Test extends AsyncResource {
282	282	this.harness = null; // Configured on the root test by the test harness.
283	283	this.mock = null;
284	284	this.cancelled = false;
285		- this.skipped = !!skip;
286		- this.isTodo = !!todo;
	285	+ this.skipped = skip !== undefined && skip !== false;
	286	+ this.isTodo = todo !== undefined && todo !== false;
287	287	this.startTime = null;
288	288	this.endTime = null;
289	289	this.passed = false;
@@ -634,7 +634,7 @@ class Test extends AsyncResource {
634	634	subtest.#cancel(pendingSubtestsError);
635	635	subtest.postRun(pendingSubtestsError);
636	636	}
637		- if (!subtest.passed) {
	637	+ if (!subtest.passed && !subtest.isTodo) {
638	638	failed++;
639	639	}
640	640	}