ci frontend e2e tests fixes

Author: HardMax71

.github/actions/setup-ci-compose/action.yml

@@ -12,7 +12,7 @@ runs:
     - name: Install yq
       shell: bash
       run: |
-        sudo wget -qO /usr/local/bin/yq https://github.com/mikefarah/yq/releases/latest/download/yq_linux_amd64
+        sudo wget -qO /usr/local/bin/yq https://github.com/mikefarah/yq/releases/download/v4.50.1/yq_linux_amd64
         sudo chmod +x /usr/local/bin/yq
 
     - name: Create CI compose configuration
@@ -47,9 +47,8 @@ runs:
         yq eval '.services.kafka.depends_on.zookeeper.condition = "service_started"' -i docker-compose.ci.yaml
 
         # Cert-generator CI configuration
-        yq eval 'select(.services."cert-generator".extra_hosts == null).services."cert-generator".extra_hosts = []' -i docker-compose.ci.yaml
-        yq eval '.services."cert-generator".extra_hosts += ["host.docker.internal:host-gateway"]' -i docker-compose.ci.yaml
-        yq eval '.services."cert-generator".environment += ["CI=true"]' -i docker-compose.ci.yaml
+        yq eval '.services."cert-generator".extra_hosts = ((.services."cert-generator".extra_hosts // []) + ["host.docker.internal:host-gateway"] | unique)' -i docker-compose.ci.yaml
+        yq eval '.services."cert-generator".environment = ((.services."cert-generator".environment // []) + ["CI=true"] | unique)' -i docker-compose.ci.yaml
         yq eval ".services.\"cert-generator\".volumes += [\"${KUBECONFIG_PATH}:/root/.kube/config:ro\"]" -i docker-compose.ci.yaml
 
         echo "Created docker-compose.ci.yaml"

.github/workflows/ci.yml

@@ -174,7 +174,7 @@ jobs:
         if: always()
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
-          file: backend/coverage.xml
+          files: backend/coverage.xml
           flags: backend
           name: backend-coverage
           slug: HardMax71/Integr8sCode

frontend/e2e/auth.spec.ts

@@ -31,7 +31,7 @@ test.describe('Authentication', () => {
     await page.click('button[type="submit"]');
 
     // Wait for error message to appear
-    await expect(page.locator('p.text-red-600, p.text-red-400')).toBeVisible({ timeout: 10000 });
+    await expect(page.locator('p.text-red-600, p.text-red-400')).toBeVisible();
   });
 
   test('redirects to editor on successful login', async ({ page }) => {
@@ -43,7 +43,7 @@ test.describe('Authentication', () => {
     await page.click('button[type="submit"]');
 
     // Should redirect to editor
-    await expect(page).toHaveURL(/\/editor/, { timeout: 15000 });
+    await expect(page).toHaveURL(/\/editor/);
   });
 
   test('shows loading state during login', async ({ page }) => {
@@ -65,23 +65,23 @@ test.describe('Authentication', () => {
     await page.goto('/editor');
 
     // Should redirect to login
-    await expect(page).toHaveURL(/\/login/, { timeout: 10000 });
+    await expect(page).toHaveURL(/\/login/);
   });
 
   test('preserves redirect path after login', async ({ page }) => {
     // Try to access specific protected route
     await page.goto('/settings');
 
     // Should redirect to login
-    await expect(page).toHaveURL(/\/login/, { timeout: 10000 });
+    await expect(page).toHaveURL(/\/login/);
 
     // Login
     await page.fill('#username', 'user');
     await page.fill('#password', 'user123');
     await page.click('button[type="submit"]');
 
     // Should redirect back to settings
-    await expect(page).toHaveURL(/\/settings/, { timeout: 15000 });
+    await expect(page).toHaveURL(/\/settings/);
   });
 
   test('has link to registration page', async ({ page }) => {
@@ -108,7 +108,7 @@ test.describe('Logout', () => {
     await page.fill('#username', 'user');
     await page.fill('#password', 'user123');
     await page.click('button[type="submit"]');
-    await expect(page).toHaveURL(/\/editor/, { timeout: 15000 });
+    await expect(page).toHaveURL(/\/editor/);
   });
 
   test('can logout from authenticated state', async ({ page }) => {
@@ -118,7 +118,7 @@ test.describe('Logout', () => {
     if (await logoutButton.isVisible()) {
       await logoutButton.click();
       // Should redirect to login
-      await expect(page).toHaveURL(/\/login/, { timeout: 10000 });
+      await expect(page).toHaveURL(/\/login/);
     }
   });
 });

frontend/playwright.config.ts

@@ -4,13 +4,13 @@ export default defineConfig({
   testDir: './e2e',
   fullyParallel: true,
   forbidOnly: !!process.env.CI,
-  retries: process.env.CI ? 2 : 0,
-  workers: process.env.CI ? 1 : undefined,
-  timeout: process.env.CI ? 60000 : 30000,
+  retries: process.env.CI ? 1 : 0,  // Reduced: 1 retry is enough to catch flakes
+  workers: process.env.CI ? 2 : undefined,  // Increased: tests are independent
+  timeout: 30000,  // 30s is plenty for page operations
   expect: {
-    timeout: process.env.CI ? 10000 : 5000,
+    timeout: 5000,  // 5s for element expectations
   },
-  reporter: 'html',
+  reporter: process.env.CI ? [['html'], ['github']] : 'html',
   use: {
     baseURL: 'https://localhost:5001',
     ignoreHTTPSErrors: true,