Skip to content

fix: aria snapshot issues#39786

Merged
yury-s merged 2 commits intomicrosoft:mainfrom
dgozman:fix-aria-snapshot-bugs
Mar 20, 2026
Merged

fix: aria snapshot issues#39786
yury-s merged 2 commits intomicrosoft:mainfrom
dgozman:fix-aria-snapshot-bugs

Conversation

@dgozman
Copy link
Contributor

@dgozman dgozman commented Mar 20, 2026

  • frame.locator().ariaSnapshot had the wrong base frame
  • depth option was not respected across iframe boundaries

@github-actions
Copy link
Contributor

Test results for "MCP"

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

5457 passed, 340 skipped


Merge workflow run.

@github-actions
Copy link
Contributor

Test results for "tests 1"

8 flaky ⚠️ [chromium-library] › library/trace-viewer.spec.ts:1223 › should display language-specific locators `@ubuntu-22.04-chromium-tip-of-tree`
⚠️ [chromium-library] › library/beforeunload.spec.ts:130 › should support dismissing the dialog multiple times `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/video.spec.ts:358 › screencast › should capture navigation `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/trace-viewer.spec.ts:1223 › should display language-specific locators `@chromium-ubuntu-22.04-node24`
⚠️ [chromium-library] › library/trace-viewer.spec.ts:1223 › should display language-specific locators `@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`
⚠️ [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-test-output.spec.ts:118 › should collapse repeated console messages for test `@ubuntu-latest-node20`

38820 passed, 845 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