From 133a1cd85375c18b10b6e766b1c0f200425cbceb Mon Sep 17 00:00:00 2001
From: Andrew Tavis McAllister <andrew.t.mcallister@gmail.com>
Date: Sun, 8 Dec 2024 19:25:29 +0100
Subject: [PATCH] Run playwright sequentially and remove env var echo

---
 .github/workflows/pr_ci_playwright_e2e.yaml | 3 ---
 frontend/playwright.config.ts               | 4 ++--
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/pr_ci_playwright_e2e.yaml b/.github/workflows/pr_ci_playwright_e2e.yaml
index b5be32060..e6d9eeb99 100644
--- a/.github/workflows/pr_ci_playwright_e2e.yaml
+++ b/.github/workflows/pr_ci_playwright_e2e.yaml
@@ -41,9 +41,6 @@ jobs:
         with:
           env-file: .env.dev
 
-      - name: Check Env Variables
-        run: echo $VITE_FRONTEND_URL
-
       - name: Clean Yarn Cache
         working-directory: ./frontend
         run: sudo yarn cache clean
diff --git a/frontend/playwright.config.ts b/frontend/playwright.config.ts
index 228b7031f..d1b4eb449 100644
--- a/frontend/playwright.config.ts
+++ b/frontend/playwright.config.ts
@@ -22,7 +22,7 @@ const ENV = (process.env.TEST_ENV || "local") as keyof typeof environments;
 export default defineConfig({
   testDir: "./test-e2e/specs",
   /* Run tests in files in parallel. */
-  fullyParallel: true,
+  fullyParallel: false,
   /* Fail the build on CI if you accidentally left test.only in the source code. */
   forbidOnly: !!process.env.CI,
   /* Retry on CI only. */
@@ -76,7 +76,7 @@ export default defineConfig({
       use: { ...devices["Galaxy S9+"], isMobile: true },
     },
     {
-      name: "Mobile iPad",
+      name: "Mobile iPad Portrait",
       use: { ...devices["iPad (gen 7)"], isMobile: true },
     },
     {