perf: optimize Playwright tests with workspace reuse and parallel workers (#164)

Author: gricha

playwright.config.ts

@@ -5,7 +5,7 @@ export default defineConfig({
   fullyParallel: false,
   forbidOnly: !!process.env.CI,
   retries: process.env.CI ? 2 : 0,
-  workers: 1,
+  workers: process.env.CI ? 3 : undefined,
   reporter: 'list',
   use: {
     baseURL: 'http://127.0.0.1:7391',

test/web/fixtures.ts

@@ -3,6 +3,7 @@ import { startTestAgent, type TestAgent } from '../helpers/agent';
 
 type TestFixtures = {
   agent: TestAgent;
+  workspaceName: string;
 };
 
 export const test = base.extend<TestFixtures>({
@@ -14,6 +15,14 @@ export const test = base.extend<TestFixtures>({
     },
     { scope: 'worker' },
   ],
+  workspaceName: [
+    async ({ agent }, use) => {
+      const name = agent.generateWorkspaceName();
+      await agent.api.createWorkspace({ name });
+      await use(name);
+    },
+    { scope: 'worker' },
+  ],
 });
 
 export { expect } from '@playwright/test';

test/web/mcp.spec.ts

@@ -1,22 +1,13 @@
 import { test, expect } from './fixtures';
-import { generateTestWorkspaceName } from '../helpers/agent';
 
 test.describe('Web UI - MCP', () => {
-  test('mcp page loads from sidebar', async ({ agent, page }) => {
-    // Create a workspace first to bypass setup guard
-    const workspaceName = generateTestWorkspaceName();
-    await agent.api.createWorkspace({ name: workspaceName });
+  test('mcp page loads from sidebar', async ({ agent, workspaceName, page }) => {
+    await page.goto(`http://127.0.0.1:${agent.port}/`);
 
-    try {
-      await page.goto(`http://127.0.0.1:${agent.port}/`);
+    await page.getByRole('button', { name: 'Integrations' }).click();
+    await page.getByRole('link', { name: 'MCP' }).click();
+    await expect(page.getByRole('heading', { name: 'MCP Servers' })).toBeVisible();
 
-      await page.getByRole('button', { name: 'Integrations' }).click();
-      await page.getByRole('link', { name: 'MCP' }).click();
-      await expect(page.getByRole('heading', { name: 'MCP Servers' })).toBeVisible();
-
-      await expect(page.getByRole('button', { name: 'Add MCP Server' })).toBeVisible();
-    } finally {
-      await agent.api.deleteWorkspace(workspaceName);
-    }
-  }, 120000);
+    await expect(page.getByRole('button', { name: 'Add MCP Server' })).toBeVisible();
+  });
 });

test/web/skills.spec.ts

@@ -1,22 +1,13 @@
 import { test, expect } from './fixtures';
-import { generateTestWorkspaceName } from '../helpers/agent';
 
 test.describe('Web UI - Skills', () => {
-  test('skills page loads from sidebar', async ({ agent, page }) => {
-    // Create a workspace first to bypass setup guard
-    const workspaceName = generateTestWorkspaceName();
-    await agent.api.createWorkspace({ name: workspaceName });
+  test('skills page loads from sidebar', async ({ agent, workspaceName, page }) => {
+    await page.goto(`http://127.0.0.1:${agent.port}/`);
 
-    try {
-      await page.goto(`http://127.0.0.1:${agent.port}/`);
+    await page.getByRole('button', { name: 'Integrations' }).click();
+    await page.getByRole('link', { name: 'Skills' }).click();
+    await expect(page.getByRole('heading', { name: 'Skills' })).toBeVisible();
 
-      await page.getByRole('button', { name: 'Integrations' }).click();
-      await page.getByRole('link', { name: 'Skills' }).click();
-      await expect(page.getByRole('heading', { name: 'Skills' })).toBeVisible();
-
-      await expect(page.getByRole('button', { name: 'Add Skill' })).toBeVisible();
-    } finally {
-      await agent.api.deleteWorkspace(workspaceName);
-    }
-  }, 120000);
+    await expect(page.getByRole('button', { name: 'Add Skill' })).toBeVisible();
+  });
 });