Skip to content

Commit

Permalink
[e2e tests] External sites - update products/add-variable-product fol…
Browse files Browse the repository at this point in the history
…der (woocommerce#50942)

* Include add-variable-product/**/*.spec.js for Pressable env

* Use unique identifier for the Update button

* Add changefile(s) from automation for the following project(s): woocommerce

* Add changefile(s) from automation for the following project(s): woocommerce

* Remove @Local tag

* Use getBy* instead of locator

* Use `locator( '#publishing-action' )`

* Fix lint issues

---------

Co-authored-by: github-actions <github-actions@github.com>
  • Loading branch information
Stojdza and github-actions authored Aug 27, 2024
1 parent e13febd commit dd79e4b
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 1 deletion.
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
Significance: patch
Type: update

Update `add-variable-product` e2e tests, so they are passing against Pressable env.
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ config = {
use: { ...devices[ 'Desktop Chrome' ] },
testMatch: [
'**/basic.spec.js',
'**/merchant/products/add-variable-product/**/*.spec.js',
'**/activate-and-setup/**/*.spec.js',
'**/merchant/products/block-editor/**/*.spec.js',
],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,10 @@ test.describe( 'Add product attributes', { tag: '@gutenberg' }, () => {
'options=woocommerce_task_list_reminder_bar_hidden'
)
);
await page.getByRole( 'button', { name: 'Update' } ).click();
await page
.locator( '#publishing-action' )
.getByRole( 'button', { name: 'Update' } )
.click();

const response = await finalRequestResolution;
expect( response.ok() ).toBeTruthy();
Expand Down

0 comments on commit dd79e4b

Please sign in to comment.