Skip to content

fix(test): surface merge reporter errors#41368

Open
mturac wants to merge 1 commit into
microsoft:mainfrom
mturac:fix-40983-merge-reporter-errors
Open

fix(test): surface merge reporter errors#41368
mturac wants to merge 1 commit into
microsoft:mainfrom
mturac:fix-40983-merge-reporter-errors

Conversation

@mturac

@mturac mturac commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Fixes #40983

Summary

  • keep reporter-thrown errors separate from normal reported test errors
  • print reporter errors when merge-reports fails because a reporter throws

Tests

  • npm run build
  • npm run ttest -- tests/playwright-test/reporter-blob.spec.ts --grep "should fail merge and report error when reporter throws in onEnd"
  • npx eslint packages/playwright/src/reporters/merge.ts packages/playwright/src/reporters/multiplexer.ts
  • npm run tsc -- --pretty false
  • git diff --check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: JSON reporter silently produces 0-byte file when merged report exceeds V8 max string length

1 participant