nodejs
diff --git a/Collapse file
‎lib/internal/test_runner/utils.js‎
Copy file name to clipboardExpand all lines: lib/internal/test_runner/utils.js
+1-1Lines changed: 1 addition & 1 deletion b/Collapse file
‎lib/internal/test_runner/utils.js‎
Copy file name to clipboardExpand all lines: lib/internal/test_runner/utils.js
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/Collapse file
‎test/fixtures/test-runner/todo-suite-failing-hook.mjs‎
Copy file name to clipboard
+10Lines changed: 10 additions & 0 deletions b/Collapse file
‎test/fixtures/test-runner/todo-suite-failing-hook.mjs‎
Copy file name to clipboard
+10Lines changed: 10 additions & 0 deletions
diff --git a/Collapse file
‎test/parallel/test-runner-todo-suite-hook-failure.js‎
Copy file name to clipboard
+24Lines changed: 24 additions & 0 deletions b/Collapse file
‎test/parallel/test-runner-todo-suite-hook-failure.js‎
Copy file name to clipboard
+24Lines changed: 24 additions & 0 deletions
@@ -384,7 +384,7 @@ function countCompletedTest(test, harness = test.root.harness) {
   }
   if (test.reportedType === 'suite') {
     harness.counters.suites++;
-    if (!test.passed) {
+    if (!test.passed && !test.isTodo) {
       harness.success = false;
     }
     return;
 
@@ -0,0 +1,10 @@
+import { before, describe, it } from 'node:test';
+
+describe('todo suite with failing before hook', { todo: 'evaluating' }, () => {
+  before(() => {
+    throw new Error('simulated cleanup failure');
+  });
+
+  it('child 1', () => {});
+  it('child 2', () => {});
+});
@@ -0,0 +1,24 @@
+'use strict';
+require('../common');
+const assert = require('assert');
+const { spawnSync } = require('child_process');
+const fixtures = require('../common/fixtures');
+
+// A `before` hook failure inside a suite marked `todo` must not fail the run.
+// The `todo` flag on a suite signals that its results are advisory, the same
+// way it does on individual tests. Before this fix, the suite branch of
+// `countCompletedTest` flipped `harness.success` for any non-passing suite
+// regardless of `isTodo`, so a hook failure exited with code 1 even though
+// no failing tests were reported.
+const child = spawnSync(process.execPath, [
+  '--test',
+  '--test-reporter=tap',
+  fixtures.path('test-runner', 'todo-suite-failing-hook.mjs'),
+]);
+
+const stdout = child.stdout.toString();
+assert.strictEqual(child.signal, null);
+assert.strictEqual(child.status, 0,
+                   `expected exit 0, got ${child.status}\nstdout:\n${stdout}\nstderr:\n${child.stderr}`);
+assert.match(stdout, /# fail 0/);
+assert.match(stdout, /# todo 2/);
Original file line number	Diff line number	Diff line change
`@@ -384,7 +384,7 @@ function countCompletedTest(test, harness = test.root.harness) {`
`384`	`384`	`}`
`385`	`385`	`if (test.reportedType === 'suite') {`
`386`	`386`	`harness.counters.suites++;`
`387`		`- if (!test.passed) {`
	`387`	`+ if (!test.passed && !test.isTodo) {`
`388`	`388`	`harness.success = false;`
`389`	`389`	`}`
`390`	`390`	`return;`