-
Notifications
You must be signed in to change notification settings - Fork 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: Update close block button selector #95592
Conversation
Jetpack Cloud live (direct link)
Automattic for Agencies live (direct link)
|
This PR does not affect the size of JS and CSS bundles shipped to the user's browser. Generated by performance advisor bot at iscalypsofastyet.com. |
@@ -4,7 +4,8 @@ import { EditorComponent } from './editor-component'; | |||
|
|||
const sidebarParentSelector = '.block-editor-inserter__main-area'; | |||
const selectors = { | |||
closeBlockInserterButton: 'button[aria-label="Close block inserter"]', | |||
closeBlockInserterButton: | |||
'button[aria-label="Close Block Inserter"], button[aria-label="Close block inserter"]', |
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.
I'm assuming once GB 19.5.0 is released, we won't need the old selector?
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.
I'm not sure as well. Maybe it's worth adding a comment?
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.
Only one test is still failing for me when testing locally -- OpenTable: Expected content is in published post
. It's possible that it's related to my local setup so let's ship it and fix it later if it's still failing in CI.
@@ -4,7 +4,8 @@ import { EditorComponent } from './editor-component'; | |||
|
|||
const sidebarParentSelector = '.block-editor-inserter__main-area'; | |||
const selectors = { | |||
closeBlockInserterButton: 'button[aria-label="Close block inserter"]', | |||
closeBlockInserterButton: | |||
'button[aria-label="Close Block Inserter"], button[aria-label="Close block inserter"]', |
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.
I'm not sure as well. Maybe it's worth adding a comment?
That particular test fails for me locally too but it does pass in CI. I'm guessing its an access issue to the OpenTable API when running locally. |
Related to p1729558813594279-slack-CBTN58FTJ
Proposed Changes
Why are these changes being made?
Testing Instructions
GUTENBERG_EDGE=true yarn workspace wp-e2e-tests test -- [test path-1] [test-path-2]
. You can find the failing test paths in this build.Pre-merge Checklist