-
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
Remove old test now migrated to Playwright. #47247
Conversation
Size Change: 0 B Total Size: 1.33 MB ℹ️ View Unchanged
|
I'm not sure I understand why some unrelated tests are failing, mostly because of timeouts. |
Flaky tests detected in 9dde70d. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/3949550397
|
Thank you for the ping and sorry for the delay in reviewing, @afercia ! Thank you for working on this cleanup PR 🙌 |
Thank you! |
What?
#46038 migrated
packages/e2e-tests/specs/editor/various/a11y.test.js
totest/e2e/specs/editor/various/a11y.spec.js
but missed to remove the old file.Maybe because I rebased the PR and accidentally restored the old file in the process. This PR just removes the old file.
A quick review would be much appreciated. Cc @ciampo 🙏
Why?
How?
Testing Instructions
Testing Instructions for Keyboard
Screenshots or screencast