Skip to content

test: reduce flakiness in a few tests#40200

Merged
dgozman merged 1 commit intomicrosoft:mainfrom
dgozman:test-reduce-flakes
Apr 14, 2026
Merged

test: reduce flakiness in a few tests#40200
dgozman merged 1 commit intomicrosoft:mainfrom
dgozman:test-reduce-flakes

Conversation

@dgozman
Copy link
Copy Markdown
Collaborator

@dgozman dgozman commented Apr 14, 2026

Summary

  • rewrite extension service-worker lifecycle test to wait for explicit CDP events instead of polling shared state
  • bump audio playback wait and lower currentTime threshold in capabilities test
  • remove persistently flaky ui-mode selection-reset test

- extensions: wait for explicit CDP events instead of polling shared state
- capabilities: give audio more time to play and lower currentTime threshold
- ui-mode network tab: remove persistently flaky selection-reset test
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions
Copy link
Copy Markdown
Contributor

Test results for "tests 1"

4 flaky ⚠️ [chromium-library] › library/beforeunload.spec.ts:130 › should support dismissing the dialog multiple times `@chromium-ubuntu-22.04-node24`
⚠️ [chromium-library] › library/video.spec.ts:687 › screencast › should capture full viewport on hidpi `@chromium-ubuntu-22.04-node20`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1080 › cli codegen › should not throw csp directive violation errors `@firefox-ubuntu-22.04-node20`
⚠️ [playwright-test] › ui-mode-trace.spec.ts:700 › should indicate current test status `@windows-latest-node20`

39174 passed, 847 skipped


Merge workflow run.

@dgozman dgozman merged commit 0aae124 into microsoft:main Apr 14, 2026
62 of 66 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

Test results for "MCP"

1 failed
❌ [chrome] › mcp/dashboard.spec.ts:26 › should show browser session chip @mcp-windows-latest

5973 passed, 916 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.

2 participants