Merge branch '282-main-branch-testing-suite-faalt' into 'main'

Resolve "main branch testing suite faalt"

See merge request elgentos/magento2-playwright!45

Author: Myron van Velsen

tests/mainmenu.spec.ts

@@ -78,6 +78,9 @@ test.describe('Guest tests (not logged in)', () => {
    */
   test('Open_the_minicart', { tag: ['@mainmenu', '@cold'] }, async ({page}) => {
     const mainMenu = new MainMenuPage(page);
+    await page.goto(requireEnv('PLAYWRIGHT_BASE_URL'));
+    await mainMenu.mainMenuMiniCartButton.waitFor();
+
     await mainMenu.openMiniCart();
   });
 

tests/poms/frontend/mainmenu.page.ts

@@ -88,7 +88,7 @@ class MainMenuPage {
     await this.mainMenuAccountButton.click();
 
     await this.mainMenuLoginItem.click();
-    await this.page.waitForURL(`${slugs.account.loginSlug}/**`);
+    await this.page.waitForURL(`${slugs.account.loginSlug}**`);
     await expect(loginHeader, 'Login header text is visible').toBeVisible();
   }
 
@@ -160,7 +160,7 @@ class MainMenuPage {
    * Function for the test Open_the_minicart
    */
   async openMiniCart() {
-    await this.page.goto(requireEnv('PLAYWRIGHT_BASE_URL'));
+    // await this.page.goto(requireEnv('PLAYWRIGHT_BASE_URL'));
     await this.mainMenuMiniCartButton.waitFor();
     // By adding 'force', we can bypass the 'aria-disabled' tag.
     await this.mainMenuMiniCartButton.click({force: true});

tests/poms/frontend/product.page.ts

@@ -38,15 +38,20 @@ class ProductPage {
   }
 
   async addProductToWishlist(product:string, url: string){
+    /**
+     * Note that the test Add_product_to_wishlist is currently set to fixme
+     */
     let addedToWishlistNotification = `${product} ${outcomeMarker.wishListPage.wishListAddedNotification}`;
     await this.page.goto(url);
-    await this.addToWishlistButton.click();
-    await this.page.waitForLoadState();
+    await this.addToWishlistButton.waitFor();
+    this.addToWishlistButton.click();
 
     await expect(async () => {
-      await expect(this.page.getByText(addedToWishlistNotification)).toBeVisible();
+      await this.page.waitForSelector(UIReference.general.messageLocator, { state: 'visible' });
     }).toPass();
 
+    await expect(this.page.getByText(addedToWishlistNotification), "Notification that product has been added is visible").toBeVisible();
+
     let productNameInWishlist = this.page.locator(UIReference.wishListPage.wishListItemGridLabel).getByText(UIReference.productPage.simpleProductTitle, {exact: true});
 
     await expect(this.page).toHaveURL(new RegExp(slugs.wishList.wishListRegex));

tests/product.spec.ts

@@ -13,7 +13,11 @@ test.describe('Product page tests',{ tag: '@product',}, () => {
     await productPage.addProductToCompare(UIReference.productPage.simpleProductTitle, slugs.productPage.simpleProductSlug);
   });
 
-  test('Add_product_to_wishlist',{ tag: '@cold'}, async ({page, browserName}) => {
+  test.fixme('Add_product_to_wishlist',{ tag: '@cold'}, async ({page, browserName}) => {
+    /**
+     * This test is currently (October 2025) set to be fixed, since it causes regular timeouts.
+     * Various fixes have been tried, unsuccessfully.
+     */
     await test.step('Log in with account', async () =>{
       const browserEngine = browserName?.toUpperCase() || "UNKNOWN";
       const emailInputValue = requireEnv(`MAGENTO_EXISTING_ACCOUNT_EMAIL_${browserEngine}`);