-
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
E2E (multi-entity-saving.test.js): Wait for the font size picker controls to appear #47242
Conversation
Size Change: 0 B Total Size: 1.33 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.
Looks good. Puppeteer and Performance tests are failing miserably on this branch, but that's caused by some external factors.
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.
Nice fix 👍 Passing for me locally. As Jarda confirmed, failures are not related.
@Mamaduka, looks like all the Puppeteer jobs are failing. I've tried rerunning it, but it's still failing. Any idea what's up? 🤔 |
@WunderBart, I see the following errors in the logs:
Can you reproduce the issue locally? You might want to run |
I just confirmed. The following URL, which is used to load admin styles, triggers a 500 error on the latest trunk.
|
35941b0
to
ff6844a
Compare
I rebased the PR, let's see if tests pass now 👍 |
Flaky tests detected in ff6844a. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/3958135170
|
What?
Fix failing spec when the concurrent mode is enabled (#46467 (comment)):
multi-entity-saving.test.js: Site Editor > Save flow should allow re-saving after changing the same block attribute
How?
Wait for the font size picker controls to appear before interacting with them.
✅ Tested against changes in #46467.