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
+3Lines changed: 3 additions & 0 deletions b/Collapse file
‎lib/internal/test_runner/utils.js‎
Copy file name to clipboardExpand all lines: lib/internal/test_runner/utils.js
+3Lines changed: 3 additions & 0 deletions
diff --git a/Collapse file
‎test/fixtures/test-runner/describe_error.js‎
Copy file name to clipboard
+10Lines changed: 10 additions & 0 deletions b/Collapse file
‎test/fixtures/test-runner/describe_error.js‎
Copy file name to clipboard
+10Lines changed: 10 additions & 0 deletions
diff --git a/Collapse file
‎test/parallel/test-runner-exit-code.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-runner-exit-code.js
+8Lines changed: 8 additions & 0 deletions b/Collapse file
‎test/parallel/test-runner-exit-code.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-runner-exit-code.js
+8Lines changed: 8 additions & 0 deletions
@@ -384,6 +384,9 @@ function countCompletedTest(test, harness = test.root.harness) {
   }
   if (test.reportedType === 'suite') {
     harness.counters.suites++;
+    if (!test.passed) {
+      harness.success = false;
+    }
     return;
   }
   // Check SKIP and TODO tests first, as those should not be counted as
 
@@ -0,0 +1,10 @@
+'use strict';
+const { describe, it } = require('node:test');
+
+describe('should fail', () => {
+  throw new Error('error in describe');
+});
+
+describe('should pass', () => {
+  it('ok', () => {});
+});
@@ -70,6 +70,14 @@ if (process.argv[2] === 'child') {
   assert.strictEqual(child.status, 1);
   assert.strictEqual(child.signal, null);
 
+  // An error thrown inside describe() should cause a non-zero exit code.
+  child = spawnSync(process.execPath, [
+    '--test',
+    fixtures.path('test-runner', 'describe_error.js'),
+  ]);
+  assert.strictEqual(child.status, 1);
+  assert.strictEqual(child.signal, null);
+
   // With process isolation (default), the test name shown is the file path
   // because the parent runner only knows about file-level tests
   const neverEndingSync = fixtures.path('test-runner', 'never_ending_sync.js');
Original file line number	Diff line number	Diff line change
`@@ -384,6 +384,9 @@ function countCompletedTest(test, harness = test.root.harness) {`
`384`	`384`	`}`
`385`	`385`	`if (test.reportedType === 'suite') {`
`386`	`386`	`harness.counters.suites++;`
	`387`	`+ if (!test.passed) {`
	`388`	`+ harness.success = false;`
	`389`	`+ }`
`387`	`390`	`return;`
`388`	`391`	`}`
`389`	`392`	`// Check SKIP and TODO tests first, as those should not be counted as`