From 7fa2d0d535f6541a651fbd21ff24b352a7278348 Mon Sep 17 00:00:00 2001 From: Pilar Martinez Date: Mon, 2 Feb 2026 18:58:19 -0300 Subject: [PATCH] fix: remove double test line printed with pw-test [red-114] --- packages/cli/src/commands/pw-test.ts | 4 +++- packages/cli/src/reporters/abstract-list.ts | 5 ++++- packages/cli/src/reporters/reporter.ts | 17 +++++++++++------ 3 files changed, 18 insertions(+), 8 deletions(-) diff --git a/packages/cli/src/commands/pw-test.ts b/packages/cli/src/commands/pw-test.ts index 1d3aec4ad..6c6726db8 100644 --- a/packages/cli/src/commands/pw-test.ts +++ b/packages/cli/src/commands/pw-test.ts @@ -250,7 +250,9 @@ export default class PwTestCommand extends AuthCommand { return } - const reporters = createReporters(reporterTypes, location, verboseFlag) + const reporters = createReporters(reporterTypes, location, verboseFlag, { + showStreamingHeaders: false, + }) const repoInfo = getGitInformation(project.repoUrl) const ciInfo = getCiInformation() // TODO: ADD PROPER RETRY STRATEGY HANDLING diff --git a/packages/cli/src/reporters/abstract-list.ts b/packages/cli/src/reporters/abstract-list.ts index f3ad4a6ed..1b3b66727 100644 --- a/packages/cli/src/reporters/abstract-list.ts +++ b/packages/cli/src/reporters/abstract-list.ts @@ -32,13 +32,16 @@ export default abstract class AbstractListReporter implements Reporter { verbose: boolean testSessionId?: string _isSchedulingDelayExceeded?: boolean + showStreamingHeaders: boolean constructor ( runLocation: RunLocation, verbose: boolean, + options: { showStreamingHeaders?: boolean } = {}, ) { this.runLocation = runLocation this.verbose = verbose + this.showStreamingHeaders = options.showStreamingHeaders ?? true } onBegin (checks: Array<{ check: any, sequenceId: SequenceId }>, testSessionId?: string): void { @@ -114,7 +117,7 @@ export default abstract class AbstractListReporter implements Reporter { // Display the check title if this is the first time we're streaming logs for this check const isFirstLogBatch = !checkFile.hasStreamedLogs checkFile.hasStreamedLogs = true - if (isFirstLogBatch) { + if (isFirstLogBatch && this.showStreamingHeaders) { // For Playwright tests, we need to create a better display name const displayCheck = { ...check, diff --git a/packages/cli/src/reporters/reporter.ts b/packages/cli/src/reporters/reporter.ts index 76ab015fb..861d7544d 100644 --- a/packages/cli/src/reporters/reporter.ts +++ b/packages/cli/src/reporters/reporter.ts @@ -19,23 +19,28 @@ export interface Reporter { export type ReporterType = 'list' | 'dot' | 'ci' | 'github' | 'json' +export type ReporterOptions = { + showStreamingHeaders?: boolean +} + export const createReporters = ( types: ReporterType[], runLocation: RunLocation, verbose: boolean, + options: ReporterOptions = {}, ): Reporter[] => types.map(t => { switch (t) { case 'dot': - return new DotReporter(runLocation, verbose) + return new DotReporter(runLocation, verbose, options) case 'list': - return new ListReporter(runLocation, verbose) + return new ListReporter(runLocation, verbose, options) case 'ci': - return new CiReporter(runLocation, verbose) + return new CiReporter(runLocation, verbose, options) case 'github': - return new GithubReporter(runLocation, verbose) + return new GithubReporter(runLocation, verbose, options) case 'json': - return new JsonReporter(runLocation, verbose) + return new JsonReporter(runLocation, verbose, options) default: - return new ListReporter(runLocation, verbose) + return new ListReporter(runLocation, verbose, options) } })