Update 03-forms E2E tests after change in consent checkbox gating and refactor to Astro Actions

Author: webstackdev

test/e2e/helpers/cookieHelper.ts

@@ -37,7 +37,7 @@ export async function dismissCookieModal(page: Page): Promise<void> {
     await page.evaluate(() => {
       document.cookie = 'consent_analytics=true; path=/; max-age=31536000'
       document.cookie = 'consent_marketing=true; path=/; max-age=31536000'
-      document.cookie = 'consent_functional=true; path=/; max-age=31536000'
+      document.cookie = 'consent_functional=false; path=/; max-age=31536000'
 
       // Clear localStorage
       if (typeof localStorage !== 'undefined') {
@@ -74,7 +74,7 @@ export async function setConsentCookies(page: Page): Promise<void> {
   await page.evaluate(() => {
     document.cookie = 'consent_analytics=true; path=/; max-age=31536000'
     document.cookie = 'consent_marketing=true; path=/; max-age=31536000'
-    document.cookie = 'consent_functional=true; path=/; max-age=31536000'
+    document.cookie = 'consent_functional=false; path=/; max-age=31536000'
   })
 }
 

test/e2e/helpers/pageObjectModels/BasePage.ts

@@ -39,9 +39,9 @@ export class BasePage extends BuiltInsPage {
     try {
       // Set consent cookies
       await this._page.evaluate(() => {
-        document.cookie = 'consent_analytics=true; path=/; max-age=31536000'
-        document.cookie = 'consent_marketing=true; path=/; max-age=31536000'
-        document.cookie = 'consent_functional=true; path=/; max-age=31536000'
+        document.cookie = 'consent_analytics=false; path=/; max-age=31536000'
+        document.cookie = 'consent_marketing=false; path=/; max-age=31536000'
+        document.cookie = 'consent_functional=false; path=/; max-age=31536000'
 
         // Clear localStorage
         if (typeof localStorage !== 'undefined') {
@@ -51,14 +51,14 @@ export class BasePage extends BuiltInsPage {
       })
 
       const cookieDialog = this._page.getByRole('dialog', { name: /cookie consent/i })
-      const allowAllButton = this._page.getByRole('button', { name: /allow all/i })
 
       // Wait briefly for the dialog to appear on client-side hydrated pages.
       await cookieDialog.waitFor({ state: 'visible', timeout: wait.tinyUi }).catch(() => undefined)
 
       if (await cookieDialog.isVisible().catch(() => false)) {
-        if (await allowAllButton.isVisible().catch(() => false)) {
-          await allowAllButton.click({ timeout: wait.tinyUi }).catch(() => undefined)
+        const closeButton = this._page.getByRole('button', { name: /close cookie consent/i })
+        if (await closeButton.isVisible().catch(() => false)) {
+          await closeButton.click({ timeout: wait.tinyUi }).catch(() => undefined)
         }
       }
 
@@ -119,6 +119,21 @@ export class BasePage extends BuiltInsPage {
         window.__disableServiceWorkerForE2E = true
       }
 
+      // Establish a deterministic consent baseline before any app scripts run.
+      // GDPR form consent uses the `functional` consent store.
+      try {
+        document.cookie = 'consent_analytics=false; path=/; max-age=31536000'
+        document.cookie = 'consent_marketing=false; path=/; max-age=31536000'
+        document.cookie = 'consent_functional=false; path=/; max-age=31536000'
+
+        if (typeof localStorage !== 'undefined') {
+          localStorage.removeItem('cookieConsent')
+          localStorage.removeItem('gdprConsent')
+        }
+      } catch {
+        // Ignore failures (e.g. blocked storage in some browser modes)
+      }
+
       if (typeof window.__astroPageLoadCounter !== 'number') {
         window.__astroPageLoadCounter = 0
       }
@@ -722,22 +737,6 @@ export class BasePage extends BuiltInsPage {
     })
   }
 
-  /**
-   * ================================================================
-   *
-   * CoPilot-Added Methods
-   *
-   * ================================================================
-   */
-
-  /**
-   * Verify hero section is present and visible
-   */
-  async expectHeroSection(): Promise<void> {
-    const hero = this._page.locator('[data-component="hero"], section').first()
-    await expect(hero).toBeVisible()
-  }
-
   /**
    * Verify text is visible on the page
    */

test/e2e/specs/03-forms/consent-checkbox.spec.ts

@@ -69,9 +69,6 @@ test.describe('Newsletter GDPR Consent', () => {
     pageUnderTest = await BasePage.init(page)
     await pageUnderTest.goto(HOME_PATH)
     await waitForNewsletterSection(pageUnderTest)
-
-    // Ensure a consistent starting state for tests (checkbox may be pre-checked).
-    await pageUnderTest.locator(NEWSLETTER_CONSENT_SELECTOR).uncheck({ force: true }).catch(() => undefined)
   })
 
   test('@ready GDPR consent checkbox is visible', async () => {
@@ -100,8 +97,6 @@ test.describe('Newsletter GDPR Consent', () => {
     try {
       await fillNewsletterEmail(pageUnderTest)
       await submitNewsletterForm(pageUnderTest)
-
-      await expect(pageUnderTest.locator(NEWSLETTER_MESSAGE_SELECTOR)).toContainText('Please consent')
       await expectConsentErrorVisible(pageUnderTest)
 
       const callCount = await fetchSpy.getCallCount()
@@ -122,13 +117,20 @@ test.describe('Newsletter GDPR Consent', () => {
 
   test('@ready GDPR checkbox is accessible via keyboard', async () => {
     const checkbox = pageUnderTest.locator(NEWSLETTER_CONSENT_SELECTOR)
+    await expect(checkbox).toBeVisible()
+    await expect(checkbox).toBeEnabled()
+    await expect(checkbox).not.toBeChecked()
     await checkbox.focus()
 
     await pageUnderTest.keyboard.press('Space')
     await expect(checkbox).toBeChecked()
 
+    // ConsentCheckboxElement disables the checkbox once consent is granted.
+    // The second keypress should not revoke consent.
+    await expect(checkbox).toBeDisabled()
+
     await pageUnderTest.keyboard.press('Space')
-    await expect(checkbox).not.toBeChecked()
+    await expect(checkbox).toBeChecked()
   })
 
   test('@ready GDPR error message is displayed', async () => {

test/e2e/specs/03-forms/contact.spec.ts

@@ -91,6 +91,7 @@ test.describe('Contact Form', () => {
     await fillRequiredFields(page)
     await page.locator('#project_type').selectOption('website')
     await page.locator('#timeline').selectOption('asap')
+    await page.check('#contact-gdpr-consent')
 
     const fetchSpy = await spyOnFetchEndpoint(playwrightPage, contactSubmitActionEndpoint)
 

test/e2e/specs/03-forms/newsletter-subscription.spec.ts

@@ -45,8 +45,6 @@ test.describe('Newsletter Subscription Form', () => {
 
     try {
       await newsletterPage.fillEmail(TEST_EMAILS.valid)
-      // Ensure consent is explicitly unchecked (it can be pre-checked in some states)
-      await newsletterPage.uncheckGdprConsent()
       await newsletterPage.submitForm()
 
       // Verify that no API call was made (client-side validation prevented it)
@@ -56,7 +54,9 @@ test.describe('Newsletter Subscription Form', () => {
       }
 
       // Should show consent required error message
-      await newsletterPage.expectMessageContains('Please consent to receive marketing communications')
+      const consentError = newsletterPage.locator('#newsletter-gdpr-consent-error')
+      await expect(consentError).toBeVisible()
+      await expect(consentError).toContainText('consent')
     } finally {
       await fetchSpy.restore()
     }