test_runner: hide failing tests when all tests pass

Author: MoLow

lib/internal/test_runner/reporter/spec.js

@@ -121,6 +121,10 @@ class SpecReporter extends Transform {
     callback(null, this.#handleEvent({ type, data }));
   }
   _flush(callback) {
+    if (this.#failedTests.length === 0) {
+      callback(null, '');
+      return;
+    }
     const results = [`\n${colors['test:fail']}${symbols['test:fail']}failing tests:${white}\n`];
     for (let i = 0; i < this.#failedTests.length; i++) {
       ArrayPrototypePush(results, this.#formatTestReport(

test/message/test_runner_spec_reporter_successful.js

@@ -0,0 +1,6 @@
+// Flags: --no-warnings --test-reporter=spec
+'use strict';
+require('../common');
+const { it } = require('node:test');
+
+it("should pass", () => {});

test/message/test_runner_spec_reporter_successful.out

@@ -0,0 +1,9 @@
+* should pass *(*ms)*
+*ℹ tests 1*
+*ℹ suites 0*
+*ℹ pass 1*
+*ℹ fail 0*
+*ℹ cancelled 0*
+*ℹ skipped 0*
+*ℹ todo 0*
+*ℹ duration_ms *