test: ensure server is only manually started in CI (#584)

Author: cngonzalez

apps/kitchensink-react/playwright.config.ts

@@ -3,11 +3,13 @@ import {createPlaywrightConfig} from '@repo/e2e'
 export default createPlaywrightConfig({
   testDir: './e2e',
   /* Run your local dev server before starting the tests */
-  webServer: {
-    command: process.env['CI']
-      ? 'pnpm build --mode e2e && pnpm preview --mode e2e --port 3333'
-      : 'pnpm dev --mode e2e',
-    reuseExistingServer: true,
-    stdout: 'pipe',
-  },
+  ...(process.env['CI']
+    ? {} // In CI, don't start a webServer since it's started manually
+    : {
+        webServer: {
+          command: 'pnpm dev --mode e2e',
+          reuseExistingServer: true,
+          stdout: 'pipe',
+        },
+      }),
 })

packages/@repo/e2e/src/setup/auth.setup.ts

@@ -44,10 +44,7 @@ const authenticateUser = async (context: BrowserContext, config: AuthConfig) =>
   await page.goto(loginUrl.toString())
 
   // Wait for the redirect to complete AND network to be idle
-  await Promise.all([
-    page.waitForURL(config.expectedRedirectUrl),
-    page.waitForLoadState('networkidle'),
-  ])
+  await Promise.all([page.waitForURL(config.expectedRedirectUrl)])
 
   await page.close()
 }