Skip to content

Conversation

@yury-s
Copy link
Member

@yury-s yury-s commented Oct 25, 2024

  • Changed error header to align with other expect matchers
  • Prepend timeout error message same way as in other matchers

New message:

image

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions
Copy link
Contributor

Test results for "tests 1"

3 flaky ⚠️ [playwright-test] › ui-mode-test-ct.spec.ts:215:5 › should watch component via util @ubuntu-latest-node20-1
⚠️ [webkit-library] › library/browsercontext-viewport-mobile.spec.ts:206:5 › mobile viewport › view scale should reset after navigation @webkit-ubuntu-22.04-node18
⚠️ [playwright-test] › ui-mode-test-watch.spec.ts:145:5 › should watch all @windows-latest-node18-1

36675 passed, 626 skipped
✔️✔️✔️

Merge workflow run.

@yury-s yury-s merged commit 1e88846 into microsoft:main Oct 25, 2024
28 of 29 checks passed
@yury-s yury-s deleted the timeout-to-have-screenshot branch October 25, 2024 19:36
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