DC-207 Add: Pa11y runs on CI

Author: noxmwalsh

.github/workflows/playwright-e2e.yaml

@@ -78,9 +78,10 @@ jobs:
       - name: Install Playwright browsers
         run: cd src/SEBT.Portal.Web && pnpm exec playwright install --with-deps chromium
 
-      - name: Run Playwright tests
+      - name: Run Pa11y and Playwright E2E tests
         env:
           CI: true
+          SKIP_WEB_SERVER: "1"
           STATE: dc
           NEXT_PUBLIC_STATE: dc
           ASPNETCORE_ENVIRONMENT: Development
@@ -89,7 +90,18 @@ jobs:
           IdentifierHasher__SecretKey: "ci-e2e-identifier-hasher-key-32chars"
           Oidc__CompleteLoginSigningKey: "ci-e2e-oidc-signing-key-at-least-32-chars"
           UseMockHouseholdData: "true"
-        run: pnpm ci:test:e2e:ci
+        run: |
+          pnpm dev &
+          echo "Waiting for server at http://localhost:3000..."
+          for i in $(seq 1 90); do
+            curl -sf http://localhost:3000 > /dev/null && echo "Server ready" && break
+            sleep 2
+          done
+          curl -sf http://localhost:3000 > /dev/null || (echo "Server failed to start" && exit 1)
+          echo "Running Pa11y accessibility tests..."
+          cd src/SEBT.Portal.Web && pnpm test:a11y
+          echo "Running Playwright E2E tests..."
+          cd ../.. && pnpm ci:test:e2e:ci
 
       - name: Upload Playwright report
         uses: actions/upload-artifact@v4

package.json

@@ -16,6 +16,7 @@
     "ci:test": "./scripts/ci/test-backend.sh && ./scripts/ci/test-frontend.sh",
     "ci:test:e2e": "cd src/SEBT.Portal.Web && pnpm test:e2e",
     "ci:test:e2e:ci": "cd src/SEBT.Portal.Web && pnpm exec playwright test --project=chromium",
+    "ci:test:a11y": "cd src/SEBT.Portal.Web && pnpm test:a11y",
     "ci:test:frontend": "./scripts/ci/test-frontend.sh",
     "ci:test:backend": "./scripts/ci/test-backend.sh",
     "ci:list": "./act-test.sh list",

src/SEBT.Portal.Web/e2e/example.spec.ts

@@ -9,9 +9,15 @@ test.describe('Homepage', () => {
   test('should load and display the homepage', async ({ page }) => {
     await page.goto('/')
 
-    // Check that the page loaded
+    // The homepage currently redirects to /login
+    await expect(page).toHaveURL(/\/login\/?$/)
+
+    // Check that metadata loaded (state-based title)
     await expect(page).toHaveTitle(/SUN Bucks/i)
 
+    const continueButton = page.getByRole('button', { name: /continue/i })
+    await expect(continueButton).toBeVisible()
+
     // Check for USWDS JavaScript initialization
     const html = page.locator('html')
     await expect(html).not.toHaveClass(/usa-js-loading/, { timeout: 10000 })
@@ -33,4 +39,23 @@ test.describe('Homepage', () => {
     const main = page.locator('main, [role="main"]')
     await expect(main).toBeVisible()
   })
+
+  test('switches language to Spanish and shows translated UI', async ({ page }) => {
+    await page.goto('/login')
+
+    // Initially loads in English
+    await expect(page.getByRole('button', { name: 'Continue' })).toBeVisible()
+
+    // Switch to Spanish (handles both desktop + mobile language selector variants)
+    const desktopSelector = page.locator('.usa-language__desktop')
+    if (await desktopSelector.isVisible()) {
+      await page.locator('.usa-language__desktop button[lang="es"]').click()
+    } else {
+      await page.locator('button[aria-controls="language-options"]').click()
+      await page.locator('button[role="menuitem"][lang="es"]').click()
+    }
+
+    // Assert at least one string is translated
+    await expect(page.getByRole('button', { name: 'Continuar' })).toBeVisible()
+  })
 })

src/SEBT.Portal.Web/playwright.config.ts

@@ -7,6 +7,7 @@ const __dirname = path.dirname(fileURLToPath(import.meta.url))
 /**
  * Playwright E2E Testing Configuration
  * Cross-browser testing with mobile viewport support
+ * Set SKIP_WEB_SERVER=1 in CI when the server is already running (for Pa11y)
  */
 export default defineConfig({
   testDir: './e2e',
@@ -45,10 +46,15 @@ export default defineConfig({
     }
   ],
 
-  webServer: {
-    command: 'pnpm dev',
-    url: process.env.BASE_URL || 'http://localhost:3000',
-    cwd: path.resolve(__dirname, '../..'),
-    reuseExistingServer: !process.env.CI
-  }
+  // Omit webServer when SKIP_WEB_SERVER is set
+  ...(process.env.SKIP_WEB_SERVER
+    ? {}
+    : {
+        webServer: {
+          command: 'pnpm dev',
+          url: process.env.BASE_URL || 'http://localhost:3000',
+          cwd: path.resolve(__dirname, '../..'),
+          reuseExistingServer: !process.env.CI
+        }
+      })
 })