-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
e2e: fix flaky columns test #49676
e2e: fix flaky columns test #49676
Conversation
Size Change: 0 B Total Size: 1.37 MB ℹ️ View Unchanged
|
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.
LGTM as long as it fixes the flakiness :). Thank you for handling it so fast! ❤️
Flaky tests detected in 87d6a46. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/4654271583
|
I ran the E2E test four times and all passed with no problems, so I'll merge 👍 |
Fixes: #49534
Thankfully, @kevin told me about it in this comment.
What?
This PR fixes the Playwright test instability reported in #49534 regarding column blocks.
Why?
Looking at the artifacts reported in trunk, it appears that the sidebar is not opened before manipulating the input element to change the number of columns. I guessed that perhaps we need to explicitly run
openDocumentSettingsSidebar()
.Testing Instructions
I would like to run the GitHub Action several times and see if the problem is fixed.