You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[api review 1.65] Failures reporter, onlyFailures and --reporter-only-failures #43794
Hi, I'm the Playwright bot and I checked this API review item.
Fix proposed. Removed --reporter-only-failures from both test and merge-reports, including its internal configuration override. The per-reporter onlyFailures option remains. The HTML, JSON, and JUnit documentation now explicitly says that flaky tests and their successful retries are included.
Verification
On Linux, using 1.65.0-next at base commit 1d5572d5ddfb7f3c8dfbbe19c8e97def0b2738db:
The updated command-line test failed before the change because the removed flag was still accepted. It passes with the change for both commands.
Existing report tests pass with onlyFailures configured separately for HTML, JSON, and JUnit, for direct runs and merged blob reports. They verify that flaky tests retain both the failed and successful attempts, while blob reports retain all tests.
Activity
github-actions commentedon Oct 30, 2026
github-actions commentedon Oct 30, 2026
Hi, I'm the Playwright bot and I checked this API review item.
Fix proposed. Removed
--reporter-only-failuresfrom bothtestandmerge-reports, including its internal configuration override. The per-reporteronlyFailuresoption remains. The HTML, JSON, and JUnit documentation now explicitly says that flaky tests and their successful retries are included.Verification
On Linux, using
1.65.0-nextat base commit1d5572d5ddfb7f3c8dfbbe19c8e97def0b2738db:onlyFailuresconfigured separately for HTML, JSON, and JUnit, for direct runs and merged blob reports. They verify that flaky tests retain both the failed and successful attempts, while blob reports retain all tests.npm run ttest -- reporter-only-failures reporter-json reporter-junit reporter-blob --workers=4: 129 passed, 3 skipped.npm run buildandnpm run flintpassed.These are test-runner and report-generation checks; the changed tests do not launch browsers.
Draft PR: #43805
Triaged on Linux by the Playwright bot - agent run
fix(reporter): remove reporter-only-failures CLI flag