Add additional logging for Supabase schema migration on GitHub workflow

Author: webstackdev

.github/workflows/test.yml

@@ -188,6 +188,10 @@ jobs:
           npm run containers:supabase:db-push
           echo "::endgroup::"
 
+          echo "::group::Supabase migration status"
+          FORCE_COLOR=1 npx supabase migration list --local --workdir suprabase
+          echo "::endgroup::"
+
           echo "::group::Supabase schema snapshot"
           schema_dir="artifacts/supabase"
           schema_path="${schema_dir}/schema.sql"
@@ -198,6 +202,39 @@ jobs:
           echo "(full schema saved to ${schema_path})"
           echo "::endgroup::"
 
+          echo "::group::Supabase schema guard"
+          bash <<'BASH'
+          set -euo pipefail
+
+          db_container_line=$(docker ps --format '{{.ID}} {{.Names}} {{.Image}}' | awk '$3 ~ /supabase\/postgres/ {print $1" "$2; exit}')
+          if [ -z "${db_container_line}" ]; then
+            echo "Supabase database container not found; cannot verify migrations" >&2
+            exit 1
+          fi
+
+          container_id=$(echo "${db_container_line}" | awk '{print $1}')
+          container_name=$(echo "${db_container_line}" | awk '{print $2}')
+          echo "Inspecting tables inside ${container_name} (${container_id})"
+
+          required_tables=(newsletter_confirmations consent_records dsar_requests)
+          missing=0
+
+          for table in "${required_tables[@]}"; do
+            if ! docker exec "${container_id}" psql -U postgres -d postgres -At -c "select to_regclass('public.${table}');" | grep -q "public.${table}"; then
+              echo "❌  Missing table public.${table}"
+              missing=1
+            else
+              echo "✅  Found table public.${table}"
+            fi
+          done
+
+          if [ "${missing}" -ne 0 ]; then
+            echo "Required tables missing after Supabase migrations" >&2
+            exit 1
+          fi
+          BASH
+          echo "::endgroup::"
+
           echo "::group::Restart Supabase REST container"
           bash <<'BASH'
           set -euo pipefail

test/e2e/specs/09-pwa/service-worker.spec.ts

@@ -1,43 +1,12 @@
 /**
- * Service Worker Tests
- * Validates service worker registration, caching, and offline fallbacks.
+ * Service Worker Tests - Must be QA'd manually
+ *
+ * Note: Automated testing of service workers is not feasible in this E2E test suite
+ * due to limitations with the Astro build process and Vercel adapter.
  */
 
-import { expect, test } from '@test/e2e/helpers'
-import { PwaPage } from '@test/e2e/helpers/pageObjectModels/PwaPage'
+import { test } from '@test/e2e/helpers'
 
 test.describe('Service Worker', () => {
-  test.skip('@ready service worker registers and activates', async ({ page: playwrightPage }) => {
-    const pwaPage: PwaPage = await PwaPage.init(playwrightPage)
-    await pwaPage.navigateToHomeAndWaitForSW()
-
-    await pwaPage.expectServiceWorkerRegistered()
-    await pwaPage.expectServiceWorkerActivated()
-  })
-
-  test.skip('@ready service worker populates caches after first run', async ({ page: playwrightPage }) => {
-    const pwaPage: PwaPage = await PwaPage.init(playwrightPage)
-    await pwaPage.navigateToHomeAndWaitForSW()
-
-    const cachedAssets = await pwaPage.getCachedAssetsCount()
-    expect(cachedAssets).toBeGreaterThan(0)
-    await pwaPage.expectCacheVersioning()
-  })
-
-  test.skip('@ready offline navigation falls back to 404 page', async ({ page: playwrightPage, context, browserName }) => {
-    test.skip(browserName === 'webkit', 'Playwright WebKit cannot perform navigation requests while offline')
-
-    const pwaPage: PwaPage = await PwaPage.init(playwrightPage)
-    await pwaPage.navigateToHomeAndWaitForSW()
-
-    await pwaPage.goOffline(context)
-    try {
-      const response = await pwaPage.goto('/definitely-not-real')
-      expect(response).not.toBeNull()
-      await pwaPage.expectNotFoundFallback()
-    } finally {
-      await pwaPage.goOnline(context)
-    }
-  })
+  test.fixme('@ready service worker cannot be tested in automated E2E tests and must be QAed manually, because the @vite-pwa/astro integration that generates sw.js runs on the astro:build:done so that it has access to all generated build artifacts. The Vercel adapter for Astro is incompatible with astro serve, so it is not possible to test against a built environment.', async () => {})
 })
-