Skip to content

test(bidi): fix large screenshot test#39295

Open
hbenl wants to merge 1 commit intomicrosoft:mainfrom
hbenl:large-screenshot
Open

test(bidi): fix large screenshot test#39295
hbenl wants to merge 1 commit intomicrosoft:mainfrom
hbenl:large-screenshot

Conversation

@hbenl
Copy link
Collaborator

@hbenl hbenl commented Feb 17, 2026

Note that the screenshot size limit was recently increased in Firefox, so this could be increased in Juggler as well.

@github-actions
Copy link
Contributor

Test results for "tests 1"

4 failed
❌ [playwright-test] › playwright.trace.spec.ts:345 › should retain traces for interrupted tests @macos-latest-node20
❌ [playwright-test] › test-step.spec.ts:1106 › should report expect steps @macos-latest-node20
❌ [playwright-test] › to-have-screenshot.spec.ts:191 › should have scale:css by default @macos-latest-node20
❌ [playwright-test] › to-have-screenshot.spec.ts:238 › should report toHaveScreenshot step with expectation name in title @macos-latest-node20

4 flaky ⚠️ [chromium-library] › library/popup.spec.ts:258 › should not throw when click closes popup `@chromium-ubuntu-22.04-arm-node20`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1082 › cli codegen › should not throw csp directive violation errors `@firefox-ubuntu-22.04-node20`
⚠️ [firefox-page] › page/page-wait-for-function.spec.ts:104 › should work with strict CSP policy `@firefox-ubuntu-22.04-node20`
⚠️ [playwright-test] › ui-mode-trace.spec.ts:812 › should update state on subsequent run `@macos-latest-node20`

38550 passed, 843 skipped


Merge workflow run.

@github-actions
Copy link
Contributor

Test results for "MCP"

3 failed
❌ [chromium] › mcp/cli-session.spec.ts:184 › list --all lists sessions from all workspaces @mcp-windows-latest
❌ [chromium] › mcp/roots.spec.ts:47 › check that trace is saved in workspace @mcp-windows-latest
❌ [chrome] › mcp/launch.spec.ts:21 › test reopen browser @mcp-macos-15

4880 passed, 135 skipped


Merge workflow 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.

1 participant

Comments