Skip to content

test: enable times:1 test on chromium after roll - #42924

Merged
Dmitry Gozman (dgozman) merged 1 commit into
microsoft:mainfrom
dgozman:test-fix-41802
Sep 25, 2026
Merged

Dmitry Gozman (dgozman) merged 1 commit into
microsoft:mainfrom
dgozman:test-fix-41802

Conversation

@dgozman

Copy link
Copy Markdown
Collaborator

Closes #41802.

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "MCP"

1 failed
❌ [firefox] › mcp/cli-core.spec.ts:394 › --raw on command without output @mcp-windows-latest-firefox

8750 passed, 1480 skipped


Merge workflow run.

@github-actions

Copy link
Copy Markdown
Contributor

Hi, I'm the Playwright bot and I took a first look at the CI failures here.

🟢 The one failure is a pre-existing flake, not this PR

mcp/cli-core.spec.ts:394 fails intermittently on Windows Firefox and was already failing elsewhere before this PR. This PR only touches tests/page/page-route.spec.ts.

Details

The PR changes only one Chromium page test (tests/page/page-route.spec.ts). It doesn't touch MCP, the CLI, or any product code, so nothing in the diff can reach the failing test.

Pre-existing flake / infra

Triaged by the Playwright bot - agent run

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "tests 1"

3 flaky ⚠️ [chromium-library] › library/video.spec.ts:762 › screencast › should work with video+trace `@frozen-time-library-chromium-linux`
⚠️ [firefox-library] › library/browsercontext-cookies-third-party.spec.ts:257 › third party 'Partitioned;' cookies `@firefox-ubuntu-22.04-node20`
⚠️ [firefox-library] › library/browsercontext-cookies-third-party.spec.ts:470 › top level 'Partitioned;' cookie and same origin iframe `@firefox-ubuntu-22.04-node20`

52287 passed, 1249 skipped


Merge workflow run.

@dgozman
Dmitry Gozman (dgozman) merged commit 28b4e74 into microsoft:main Sep 25, 2026
44 of 45 checks passed
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]: When aborting only first request with {times: 1} from page route then the second request randomly failed

2 participants