nodejs
diff --git a/Collapse file
‎lib/internal/test_runner/reporter/spec.js‎
Copy file name to clipboardExpand all lines: lib/internal/test_runner/reporter/spec.js
+4-1Lines changed: 4 additions & 1 deletion b/Collapse file
‎lib/internal/test_runner/reporter/spec.js‎
Copy file name to clipboardExpand all lines: lib/internal/test_runner/reporter/spec.js
+4-1Lines changed: 4 additions & 1 deletion
diff --git a/Collapse file
‎test/fixtures/test-runner/output/spec_reporter.snapshot‎
Copy file name to clipboardExpand all lines: test/fixtures/test-runner/output/spec_reporter.snapshot
-9Lines changed: 0 additions & 9 deletions b/Collapse file
‎test/fixtures/test-runner/output/spec_reporter.snapshot‎
Copy file name to clipboardExpand all lines: test/fixtures/test-runner/output/spec_reporter.snapshot
-9Lines changed: 0 additions & 9 deletions
diff --git a/Collapse file
‎test/fixtures/test-runner/output/spec_reporter_cli.snapshot‎
Copy file name to clipboardExpand all lines: test/fixtures/test-runner/output/spec_reporter_cli.snapshot
-12Lines changed: 0 additions & 12 deletions b/Collapse file
‎test/fixtures/test-runner/output/spec_reporter_cli.snapshot‎
Copy file name to clipboardExpand all lines: test/fixtures/test-runner/output/spec_reporter_cli.snapshot
-12Lines changed: 0 additions & 12 deletions
diff --git a/Collapse file
‎test/pseudo-tty/test_runner_default_reporter.out‎
Copy file name to clipboardExpand all lines: test/pseudo-tty/test_runner_default_reporter.out
-3Lines changed: 0 additions & 3 deletions b/Collapse file
‎test/pseudo-tty/test_runner_default_reporter.out‎
Copy file name to clipboardExpand all lines: test/pseudo-tty/test_runner_default_reporter.out
-3Lines changed: 0 additions & 3 deletions
@@ -15,6 +15,7 @@ const assert = require('assert');
 const Transform = require('internal/streams/transform');
 const { inspectWithNoCustomRetry } = require('internal/errors');
 const { green, blue, red, white, gray, hasColors } = require('internal/util/colors');
+const { kSubtestsFailed } = require('internal/test_runner/test');
 const { getCoverageReport } = require('internal/test_runner/utils');
 
 const inspectOptions = { __proto__: null, colors: hasColors, breakLength: Infinity };
@@ -107,7 +108,9 @@ class SpecReporter extends Transform {
   #handleEvent({ type, data }) {
     switch (type) {
       case 'test:fail':
-        ArrayPrototypePush(this.#failedTests, data);
+        if (data.details?.error?.failureType !== kSubtestsFailed) {
+          ArrayPrototypePush(this.#failedTests, data);
+        }
         return this.#handleTestReportEvent(type, data);
       case 'test:pass':
         return this.#handleTestReportEvent(type, data);
 
@@ -399,18 +399,12 @@
       *
       *
 
- subtest sync throw fail (*ms)
-  '1 subtest failed'
-
  sync throw non-error fail (*ms)
   Symbol(thrown symbol from sync throw non-error fail)
 
  +long running (*ms)
   'test did not finish before its parent and was cancelled'
 
- top level (*ms)
-  '1 subtest failed'
-
  sync skip option is false fail (*ms)
   Error: this should be executed
       *
@@ -487,9 +481,6 @@
       *
       *
 
- subtest sync throw fails (*ms)
-  '2 subtests failed'
-
  timed out async test (*ms)
   'test timed out after *ms'
 
 
@@ -408,10 +408,6 @@
       *
       *
 
- subtest sync throw fail (*ms)
-  [Error: 1 subtest failed
-  ]
-
  sync throw non-error fail (*ms)
   [Error: Symbol(thrown symbol from sync throw non-error fail)
   ]
@@ -420,10 +416,6 @@
   [Error: test did not finish before its parent and was cancelled
   ]
 
- top level (*ms)
-  [Error: 1 subtest failed
-  ]
-
  sync skip option is false fail (*ms)
   Error: this should be executed
       *
@@ -504,10 +496,6 @@
       *
       *
 
- subtest sync throw fails (*ms)
-  [Error: 2 subtests failed
-  ]
-
  timed out async test (*ms)
   [Error: test timed out after *ms
   ]
 
@@ -55,6 +55,3 @@
 
 [31m* should pass but parent fail [90m(*ms)[39m[39m
   [32m'test did not finish before its parent and was cancelled'[39m
-
-[31m* parent [90m(*ms)[39m[39m
-  [32m'2 subtests failed'[39m
Original file line number	Diff line number	Diff line change
`@@ -399,18 +399,12 @@`
`399`	`399`	`*`
`400`	`400`	`*`
`401`	`401`
`402`		`- subtest sync throw fail (*ms)`
`403`		`- '1 subtest failed'`
`404`		`-`
`405`	`402`	`sync throw non-error fail (*ms)`
`406`	`403`	`Symbol(thrown symbol from sync throw non-error fail)`
`407`	`404`
`408`	`405`	`+long running (*ms)`
`409`	`406`	`'test did not finish before its parent and was cancelled'`
`410`	`407`
`411`		`- top level (*ms)`
`412`		`- '1 subtest failed'`
`413`		`-`
`414`	`408`	`sync skip option is false fail (*ms)`
`415`	`409`	`Error: this should be executed`
`416`	`410`	`*`
`@@ -487,9 +481,6 @@`
`487`	`481`	`*`
`488`	`482`	`*`
`489`	`483`
`490`		`- subtest sync throw fails (*ms)`
`491`		`- '2 subtests failed'`
`492`		`-`
`493`	`484`	`timed out async test (*ms)`
`494`	`485`	`'test timed out after *ms'`
`495`	`486`
Original file line number	Diff line number	Diff line change
`@@ -408,10 +408,6 @@`
`408`	`408`	`*`
`409`	`409`	`*`
`410`	`410`
`411`		`- subtest sync throw fail (*ms)`
`412`		`- [Error: 1 subtest failed`
`413`		`- ]`
`414`		`-`
`415`	`411`	`sync throw non-error fail (*ms)`
`416`	`412`	`[Error: Symbol(thrown symbol from sync throw non-error fail)`
`417`	`413`	`]`
`@@ -420,10 +416,6 @@`
`420`	`416`	`[Error: test did not finish before its parent and was cancelled`
`421`	`417`	`]`
`422`	`418`
`423`		`- top level (*ms)`
`424`		`- [Error: 1 subtest failed`
`425`		`- ]`
`426`		`-`
`427`	`419`	`sync skip option is false fail (*ms)`
`428`	`420`	`Error: this should be executed`
`429`	`421`	`*`
`@@ -504,10 +496,6 @@`
`504`	`496`	`*`
`505`	`497`	`*`
`506`	`498`
`507`		`- subtest sync throw fails (*ms)`
`508`		`- [Error: 2 subtests failed`
`509`		`- ]`
`510`		`-`
`511`	`499`	`timed out async test (*ms)`
`512`	`500`	`[Error: test timed out after *ms`
`513`	`501`	`]`