fix: remove supabase from gh action

Author: AdamJ

.github/workflows/test.yml

@@ -32,9 +32,15 @@ jobs:
 
       - name: Run tests
         run: npm run test -- --run
+        env:
+          VITE_SUPABASE_URL: https://test.supabase.co
+          VITE_SUPABASE_ANON_KEY: test-key
 
       - name: Build project
         run: npm run build
+        env:
+          VITE_SUPABASE_URL: https://test.supabase.co
+          VITE_SUPABASE_ANON_KEY: test-key
 
       - name: Upload test results
         if: always()

docs/CI_CD.md

@@ -17,10 +17,17 @@ TimeTracker Pro uses GitHub Actions for continuous integration and continuous de
 2. **Setup Node.js** - Configures Node.js 20.x environment
 3. **Install dependencies** - Runs `npm ci` for clean install
 4. **Run linter** - Executes ESLint to check code quality
-5. **Run tests** - Runs the Vitest test suite (41 tests)
-6. **Build project** - Verifies the production build succeeds
+5. **Run tests** - Runs the Vitest test suite (41 tests) with mock Supabase environment
+6. **Build project** - Verifies the production build succeeds with mock Supabase environment
 7. **Upload test results** - Archives test results and coverage (if generated)
 
+**Environment Variables:**
+The workflow sets mock Supabase credentials for testing:
+- `VITE_SUPABASE_URL`: Test URL (not a real Supabase instance)
+- `VITE_SUPABASE_ANON_KEY`: Test key (not a real key)
+
+These are only used for CI/CD and are completely mocked in tests.
+
 **Status Badge:**
 The test workflow status is displayed at the top of the README.md file.
 

src/test-setup.ts

@@ -1,7 +1,40 @@
-import { expect, afterEach, vi } from "vitest";
+import { expect, afterEach, vi, beforeAll } from "vitest";
 import { cleanup } from "@testing-library/react";
 import "@testing-library/jest-dom/vitest";
 
+// Mock Supabase before any imports
+vi.mock("@/lib/supabase", () => ({
+	supabase: {
+		auth: {
+			getSession: vi.fn(() => Promise.resolve({ data: { session: null }, error: null })),
+			getUser: vi.fn(() => Promise.resolve({ data: { user: null }, error: null })),
+			onAuthStateChange: vi.fn(() => ({
+				data: { subscription: { unsubscribe: vi.fn() } }
+			}))
+		},
+		from: vi.fn(() => ({
+			select: vi.fn(() => ({
+				eq: vi.fn(() => Promise.resolve({ data: [], error: null }))
+			})),
+			insert: vi.fn(() => Promise.resolve({ data: null, error: null })),
+			update: vi.fn(() => ({
+				eq: vi.fn(() => Promise.resolve({ data: null, error: null }))
+			})),
+			delete: vi.fn(() => ({
+				eq: vi.fn(() => Promise.resolve({ data: null, error: null }))
+			}))
+		}))
+	},
+	getCachedUser: vi.fn(() => Promise.resolve(null)),
+	clearUserCache: vi.fn()
+}));
+
+// Set required environment variables for tests
+beforeAll(() => {
+	process.env.VITE_SUPABASE_URL = "https://test.supabase.co";
+	process.env.VITE_SUPABASE_ANON_KEY = "test-anon-key";
+});
+
 // Cleanup after each test case (e.g., clearing jsdom)
 afterEach(() => {
 	cleanup();