fix: separate playwright configs for scraper vs e2e tests

Scraper tests use mock servers, don't need webServer.
E2E tests need the real server via webServer config.

Author: atomantic

package.json

@@ -20,7 +20,7 @@
     "test": "vitest",
     "test:unit": "vitest run tests/unit",
     "test:integration": "vitest run tests/integration",
-    "test:scraper": "playwright test tests/scraper",
+    "test:scraper": "playwright test --config playwright.scraper.config.ts",
     "test:e2e": "playwright test tests/e2e",
     "test:ci": "vitest run --coverage",
     "test:watch": "vitest watch",

playwright.config.ts

@@ -1,10 +1,11 @@
 import { defineConfig, devices } from '@playwright/test';
 
 /**
- * Playwright configuration for scraper and E2E tests
+ * Playwright configuration for E2E tests
+ * For scraper tests, use playwright.scraper.config.ts
  */
 export default defineConfig({
-  testDir: './tests',
+  testDir: './tests/e2e',
   fullyParallel: true,
   forbidOnly: !!process.env.CI,
   retries: process.env.CI ? 2 : 0,
@@ -19,17 +20,8 @@ export default defineConfig({
   },
 
   projects: [
-    {
-      name: 'scraper-tests',
-      testMatch: 'tests/scraper/**/*.spec.ts',
-      use: {
-        ...devices['Desktop Chrome'],
-        headless: true,
-      },
-    },
     {
       name: 'e2e-tests',
-      testMatch: 'tests/e2e/**/*.spec.ts',
       use: {
         ...devices['Desktop Chrome'],
         headless: true,

playwright.scraper.config.ts

@@ -0,0 +1,31 @@
+import { defineConfig, devices } from '@playwright/test';
+
+/**
+ * Playwright configuration for scraper tests only
+ * These tests use their own mock servers, no webServer needed
+ */
+export default defineConfig({
+  testDir: './tests/scraper',
+  fullyParallel: true,
+  forbidOnly: !!process.env.CI,
+  retries: process.env.CI ? 2 : 0,
+  workers: process.env.CI ? 1 : undefined,
+  reporter: 'html',
+  timeout: 30000,
+
+  use: {
+    trace: 'on-first-retry',
+    screenshot: 'only-on-failure',
+  },
+
+  projects: [
+    {
+      name: 'scraper-tests',
+      use: {
+        ...devices['Desktop Chrome'],
+        headless: true,
+      },
+    },
+  ],
+  // No webServer - scraper tests use mock servers
+});