ci

- removed backend unit tests job (will add later)
- tests fix

Author: HardMax71

.github/workflows/ci.yml

@@ -43,36 +43,8 @@ jobs:
           name: frontend-coverage
           path: frontend/coverage/
 
-  backend-unit:
-    name: Backend Unit Tests
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v6
-
-      - name: Set up uv
-        uses: astral-sh/setup-uv@v7
-        with:
-          enable-cache: true
-          cache-dependency-glob: "backend/uv.lock"
-
-      - name: Install Python dependencies
-        run: |
-          cd backend
-          uv python install 3.12
-          uv sync --frozen
-
-      - name: Run unit tests
-        timeout-minutes: 5
-        run: |
-          cd backend
-          uv run pytest tests/unit -v --cov=app --cov-branch --cov-report=xml --cov-report=term
-
-      - name: Upload coverage
-        uses: actions/upload-artifact@v6
-        if: always()
-        with:
-          name: backend-unit-coverage
-          path: backend/coverage.xml
+  # NOTE: Backend has no pure unit tests (all tests require MongoDB/Redis)
+  # All backend tests run in backend-integration job below
 
   # ============================================================
   # Phase 2: Integration/E2E tests run in parallel after unit tests
@@ -196,7 +168,6 @@ jobs:
 
   backend-integration:
     name: Backend Integration Tests
-    needs: backend-unit
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v6

frontend/e2e/auth.spec.ts

@@ -2,18 +2,19 @@ import { test, expect } from '@playwright/test';
 
 test.describe('Authentication', () => {
   test.beforeEach(async ({ page }) => {
-    // Clear all storage (localStorage stores auth state, not cookies)
+    // Clear ALL auth state: cookies (HTTP-only auth token) + localStorage (cached state)
+    await page.context().clearCookies();
     await page.goto('/login');
     await page.evaluate(() => {
       localStorage.clear();
       sessionStorage.clear();
     });
-    await page.reload();
   });
 
   test('shows login page with form elements', async ({ page }) => {
     await page.goto('/login');
-    await page.waitForLoadState('networkidle');
+    // Wait for the login form to render
+    await page.waitForSelector('#username');
 
     await expect(page.locator('h2')).toContainText('Sign in to your account');
     await expect(page.locator('#username')).toBeVisible();
@@ -23,27 +24,26 @@ test.describe('Authentication', () => {
 
   test('shows validation when submitting empty form', async ({ page }) => {
     await page.goto('/login');
-    await page.waitForLoadState('networkidle');
+    await page.waitForSelector('#username');
 
     const usernameInput = page.locator('#username');
     await expect(usernameInput).toHaveAttribute('required', '');
   });
 
   test('shows error with invalid credentials', async ({ page }) => {
     await page.goto('/login');
-    await page.waitForLoadState('networkidle');
+    await page.waitForSelector('#username');
 
     await page.fill('#username', 'invaliduser');
     await page.fill('#password', 'wrongpassword');
     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();
   });
 
   test('redirects to editor on successful login', async ({ page }) => {
     await page.goto('/login');
-    await page.waitForLoadState('networkidle');
+    await page.waitForSelector('#username');
 
     await page.fill('#username', 'user');
     await page.fill('#password', 'user123');
@@ -54,55 +54,41 @@ test.describe('Authentication', () => {
 
   test('shows loading state during login', async ({ page }) => {
     await page.goto('/login');
-    await page.waitForLoadState('networkidle');
+    await page.waitForSelector('#username');
 
     await page.fill('#username', 'user');
     await page.fill('#password', 'user123');
 
     const submitButton = page.locator('button[type="submit"]');
     await submitButton.click();
 
-    // Button should show loading text or sign in
     await expect(submitButton).toContainText(/Logging in|Sign in/);
   });
 
   test('redirects unauthenticated users from protected routes', async ({ page }) => {
-    // Ensure clean state
-    await page.goto('/login');
-    await page.evaluate(() => localStorage.clear());
-
-    // Try to access protected route
     await page.goto('/editor');
-    await page.waitForLoadState('networkidle');
-
-    // Should redirect to login
+    // Should redirect to login and show login form
+    await page.waitForSelector('#username');
     await expect(page).toHaveURL(/\/login/);
   });
 
   test('preserves redirect path after login', async ({ page }) => {
-    // Ensure clean state
-    await page.goto('/login');
-    await page.evaluate(() => localStorage.clear());
-
-    // Try to access specific protected route
     await page.goto('/settings');
-    await page.waitForLoadState('networkidle');
-
     // Should redirect to login
+    await page.waitForSelector('#username');
     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/);
   });
 
   test('has link to registration page', async ({ page }) => {
     await page.goto('/login');
-    await page.waitForLoadState('networkidle');
+    await page.waitForSelector('#username');
 
     const registerLink = page.locator('a[href="/register"]');
     await expect(registerLink).toBeVisible();
@@ -111,7 +97,7 @@ test.describe('Authentication', () => {
 
   test('can navigate to registration page', async ({ page }) => {
     await page.goto('/login');
-    await page.waitForLoadState('networkidle');
+    await page.waitForSelector('#username');
 
     await page.click('a[href="/register"]');
 
@@ -121,14 +107,14 @@ test.describe('Authentication', () => {
 
 test.describe('Logout', () => {
   test.beforeEach(async ({ page }) => {
-    // Clear state first
+    // Clear all state first
+    await page.context().clearCookies();
     await page.goto('/login');
     await page.evaluate(() => {
       localStorage.clear();
       sessionStorage.clear();
     });
-    await page.reload();
-    await page.waitForLoadState('networkidle');
+    await page.waitForSelector('#username');
 
     // Login
     await page.fill('#username', 'user');