-
Notifications
You must be signed in to change notification settings - Fork 4.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrate Heading block tests to Playwright #47955
Conversation
Size Change: 0 B Total Size: 1.33 MB ℹ️ View Unchanged
|
Flaky tests detected in 3aa1b8d. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/4142413898
|
☝️ It still reported a flaky test, is it the same issue or a new one? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wonderful work as always! Also so nice to see that the test is much more readable now 😆 .
Thanks for the review, @kevin940726 🙇 |
What?
Part of #38851
Fixes #46984
PR migrates Heading block e2e tests to Playwright and tries to fix the flaky
should correctly apply custom colors
test.Why
While looking at test artifacts, I noticed that color validation failed after the input, and used the fallback color (black). I can only reproduce this manually when typing very slowly or fast. This would never happen in the real-world scenario since validation happens again on blur.
P.S. We might want to look into the validation issues separately.
How?
I've replaced
page.keyboard.type
withLocator.fill()
, which should be closer to the paste action.Testing Instructions
Screenshots or screencast