Fix e2e test cases in 01-smoke

Author: webstackdev

test/e2e/specs/01-smoke/critical-paths.spec.ts

@@ -22,13 +22,14 @@ test.describe('Critical Paths @smoke', () => {
     test.skip(isMobile, 'Desktop navigation test - skipping on mobile viewport')
 
     const page = await BasePage.init(playwrightPage)
-
-    // Import setupTestPage here to avoid unused import warnings on other tests
-    const { setupTestPage } = await import('../../helpers/cookieHelper')
-    await setupTestPage(playwrightPage, '/')
-
     for (const { url: path } of page.navigationItems) {
-      await page.click(`a[href="${path}"]`)
+      await page.goto('/')
+      const navigationComplete = page.waitForPageLoad()
+      await page.navigateToPage(path)
+      await navigationComplete
+      await playwrightPage.waitForFunction(() => {
+        return !document.documentElement.hasAttribute('data-astro-transition')
+      })
       // eslint-disable-next-line security/detect-non-literal-regexp
       await page.expectUrl(new RegExp(path))
     }
@@ -41,21 +42,26 @@ test.describe('Critical Paths @smoke', () => {
     test.skip(!isMobile, 'Mobile navigation test - skipping on desktop viewport')
 
     const page = await BasePage.init(playwrightPage)
-    await page.goto('/')
 
     const navItems = page.navigationItems
     for (let i = 0; i < navItems.length; i++) {
       const item = navItems[i]
       if (!item) continue
 
       const { url: path } = item
+      await page.goto('/')
 
       // Open mobile menu before each navigation
       await page.click('button[aria-label="toggle menu"]')
-      await playwrightPage.waitForTimeout(600) // Wait for mobile menu animation
+      await playwrightPage.waitForSelector('.menu-visible', { state: 'visible' })
 
       // Click navigation link
+      const navigationComplete = page.waitForPageLoad()
       await page.click(`a[href="${path}"]`)
+      await navigationComplete
+      await playwrightPage.waitForFunction(() => {
+        return !document.documentElement.hasAttribute('data-astro-transition')
+      })
       // eslint-disable-next-line security/detect-non-literal-regexp
       await page.expectUrl(new RegExp(path))
     }

test/e2e/specs/01-smoke/homepage.spec.ts

@@ -2,8 +2,11 @@
  * Homepage Smoke Test
  * Dedicated test for homepage basic functionality and app initialization
  */
+import themeConfig from '@content/themes.json' assert { type: 'json' }
 import { BasePage, test, expect } from '@test/e2e/helpers'
 
+const defaultThemeId = themeConfig.defaultTheme?.id ?? 'light'
+
 test.describe('Homepage @smoke', () => {
   test('@ready homepage loads successfully', async ({ page: playwrightPage }) => {
     const page = await BasePage.init(playwrightPage)
@@ -23,7 +26,7 @@ test.describe('Homepage @smoke', () => {
     await page.goto('/')
     await themeKeyPromise
     const result = await page.getThemeKeyValue()
-    expect(result).toBe('default')
+    expect(result).toBe(defaultThemeId)
   })
 
   test('@ready homepage has no errors', async ({ page: playwrightPage  }) => {