-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Closed
Labels
[Status] In ProgressTracking issues with work in progressTracking issues with work in progress[Tool] E2E Test Utils/packages/e2e-test-utils/packages/e2e-test-utils[Type] EnhancementA suggestion for improvement.A suggestion for improvement.
Description
What problem does this address?
Currently, the editor.openDocumentSettingsSidebar()
of the @wordpress/e2e-test-utils-playwright
uses this Locator to match the Open Settings Icon:
const toggleButton = this.page
.getByRole( 'region', { name: 'Editor top bar' } )
.getByRole( 'button', {
name: 'Settings',
disabled: false,
} );
but this fails when the post name includes the word Settings, as shown here:
What is your proposed solution?
This could be resolved by improving the locator as follows:
const toggleButton = this.page
.getByRole( 'region', { name: 'Editor top bar' } )
.getByRole( 'button', {
name: 'Settings',
exact: true,
disabled: false,
} );
Metadata
Metadata
Assignees
Labels
[Status] In ProgressTracking issues with work in progressTracking issues with work in progress[Tool] E2E Test Utils/packages/e2e-test-utils/packages/e2e-test-utils[Type] EnhancementA suggestion for improvement.A suggestion for improvement.