Skip to content

feat(reporter): add omitTags option to omit auto-appended tags#41968

Open
dcrousso wants to merge 1 commit into
microsoft:mainfrom
dcrousso:fix-41945
Open

feat(reporter): add omitTags option to omit auto-appended tags#41968
dcrousso wants to merge 1 commit into
microsoft:mainfrom
dcrousso:fix-41945

Conversation

@dcrousso

Copy link
Copy Markdown
Contributor

the terminal reporters automatically append a test's tags to its title, which clutters the output when a test declares many tags

add an off-by-default omitTags option that suppresses the appended tags in the list, line, dot, github, and junit reporters

note that tags written directly into a test title are left untouched

fixes #41945

the terminal reporters automatically append a test's tags to its title, which clutters the output when a test declares many tags

add an off-by-default `omitTags` option that suppresses the appended tags in the `list`, `line`, `dot`, `github`, and `junit` reporters

note that tags written directly into a test title are left untouched
@github-actions

Copy link
Copy Markdown
Contributor

Test results for "MCP"

1 failed
❌ [firefox] › mcp/dashboard.spec.ts:108 › should activate session when show is called with -s @mcp-windows-latest-firefox

7794 passed, 1266 skipped


Merge workflow run.

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "tests 1"

5 flaky ⚠️ [chromium-library] › library/video.spec.ts:356 › screencast › should work for popups `@realtime-time-library-chromium-linux`
⚠️ [chromium-library] › library/video.spec.ts:736 › screencast › should work with video+trace `@chromium-ubuntu-22.04-node20`
⚠️ [chromium-library] › library/video.spec.ts:495 › screencast › should capture static page in persistent context @smoke `@chromium-ubuntu-22.04-node22`
⚠️ [firefox-page] › page/page-emulate-media.spec.ts:144 › should keep reduced motion and color emulation after reload `@firefox-ubuntu-22.04-node20`
⚠️ [firefox-page] › page/page-event-request.spec.ts:181 › should return response body when Cross-Origin-Opener-Policy is set `@firefox-ubuntu-22.04-node20`

50292 passed, 1190 skipped


Merge workflow run.

@github-actions

Copy link
Copy Markdown
Contributor

🟢 CI is clear — the one failure is a pre-existing flake

Hi, I'm the Playwright bot and I took a look at the failing CI.

The single hard failure — mcp/dashboard.spec.ts:108 on mcp-windows-latest-firefox — is a known flake, not something this PR caused. This PR only touches reporter code (base.ts, github.ts, junit.ts, and the omitTags type), which the MCP dashboard's session-activation flow doesn't exercise.

Details

This PR adds an off-by-default omitTags reporter option. The diff is confined to packages/playwright/src/reporters/{base,github,junit}.ts, the generated types, docs, and tests/playwright-test/reporter-base.spec.ts. Nothing in it reaches the MCP server, browser automation, or the dashboard UI.

Pre-existing flake / infra

Triaged by the Playwright bot - agent run

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.

[Feature]: Make tags auto-appending configurable

1 participant