-
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
test: Expand mobile e2e test helpers #48978
Conversation
Block buttons that happen to be rendered behind the virtual home button could not be clicked, even though they were technically visible. This adds a method to evaluate whether an element is within a given bounds.
Size Change: 0 B Total Size: 1.34 MB ℹ️ View Unchanged
|
The previous command requires rebooting the device to take effect. The new commands updates the appearance immediately.
This will avoid repeating logic to find a block type button within the insterter, and is a common practice for nested blocks.
The test that relied upon this functionality, adding a block before the currently selected block, is now an integration test.
…nd-mobile-e2e-test-helpers
Flaky tests detected in 0d11b28. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/4451455041
|
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! Nice to see new helpers like toggleDarkMode
🚀
Great work!
Related PRs
What?
Expand the mobile e2e test helpers for account for new test coverage.
Why?
button on mobile devices.
How?
Testing Instructions
n/a
Testing Instructions for Keyboard
n/a
Screenshots or screencast
n/a