test(webkit): document a webkit transition crash from 42637 - #42640
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
|
||
| import { test as it, expect } from './pageTest'; | ||
|
|
||
| it.only('should not crash when filter transition completes', async ({ page, browserName }) => { |
There was a problem hiding this comment.
i dont think we want only
| promise = settled(); | ||
| divs.forEach(div => div.classList.remove('dim')); | ||
| await promise; | ||
| }).catch(() => {}); |
There was a problem hiding this comment.
why not fail the test if there are other issues?
There was a problem hiding this comment.
It probably wants to tell that evaluate can fail, but it is important that it does not crash the page.
af74c93
into
microsoft:main
Test results for "tests 1"6 flaky51542 passed, 1247 skipped Merge workflow run. |
Test results for "MCP"1 failed 8347 passed, 1376 skipped Merge workflow run. |
|
Hi, I'm the Playwright bot and I took a look at the CI failures on this PR. 🟢 CI is clear — the one failure is a known Firefox flakeThe only real failure is DetailsPre-existing flake / infra
Caused by this PR
Triaged by the Playwright bot - agent run |
Ref: #42637