From 87a38b5a2d9e14514c83abb14612165c23eef81f Mon Sep 17 00:00:00 2001 From: Moshe Atlow Date: Sun, 2 Apr 2023 10:05:53 +0300 Subject: [PATCH] test_runner: hide failing tests title when all tests pass --- lib/internal/test_runner/reporter/spec.js | 4 ++++ test/message/test_runner_spec_reporter_successful.js | 6 ++++++ test/message/test_runner_spec_reporter_successful.out | 9 +++++++++ 3 files changed, 19 insertions(+) create mode 100644 test/message/test_runner_spec_reporter_successful.js create mode 100644 test/message/test_runner_spec_reporter_successful.out diff --git a/lib/internal/test_runner/reporter/spec.js b/lib/internal/test_runner/reporter/spec.js index 2e3ba834351970..4628cbe2fc3f03 100644 --- a/lib/internal/test_runner/reporter/spec.js +++ b/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( diff --git a/test/message/test_runner_spec_reporter_successful.js b/test/message/test_runner_spec_reporter_successful.js new file mode 100644 index 00000000000000..e5e21e29e15e3e --- /dev/null +++ b/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', () => {}); diff --git a/test/message/test_runner_spec_reporter_successful.out b/test/message/test_runner_spec_reporter_successful.out new file mode 100644 index 00000000000000..855778c777ec92 --- /dev/null +++ b/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 *