🤖 refactor: use TEST_OLLAMA env var to control Ollama tests

Simplifies CI by using an explicit environment variable instead of
path filtering:

**Test Changes:**
- Ollama tests now require both TEST_INTEGRATION=1 and TEST_OLLAMA=1
- Uses `describeOllama` that checks `process.env.TEST_OLLAMA === '1'`
- Auto-skips when TEST_OLLAMA is not set (no manual filtering needed)

**CI Changes:**
- `integration-test` job: runs all tests, Ollama tests skip automatically
- `ollama-test` job: sets TEST_OLLAMA=1 to enable Ollama tests
- Removed `--testPathIgnorePatterns` (no longer needed)
- Cleaner and more explicit test gating

Benefits:
- Simpler CI configuration (no path filtering)
- Consistent pattern with TEST_INTEGRATION
- Easy to run Ollama tests locally: TEST_INTEGRATION=1 TEST_OLLAMA=1 bun x jest tests/ipcMain/ollama.test.ts

_Generated with `cmux`_

Author: ammar-agent

.github/workflows/ci.yml

@@ -104,8 +104,8 @@ jobs:
 
       - name: Run integration tests with coverage
         # --silent suppresses per-test output (17 test files × 32 workers = overwhelming logs)
-        # Exclude Ollama tests (run separately in ollama-test job)
-        run: TEST_INTEGRATION=1 bun x jest --coverage --maxWorkers=100% --silent --testPathIgnorePatterns=ollama.test.ts ${{ github.event.inputs.test_filter || 'tests' }}
+        # Ollama tests are skipped automatically (require TEST_OLLAMA=1)
+        run: TEST_INTEGRATION=1 bun x jest --coverage --maxWorkers=100% --silent ${{ github.event.inputs.test_filter || 'tests' }}
         env:
           OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
           ANTHROPIC_API_KEY: ${{ secrets.ANTHROPIC_API_KEY }}
@@ -144,8 +144,8 @@ jobs:
         run: make build-main
 
       - name: Run Ollama integration tests with coverage
-        # Run only Ollama-specific tests
-        run: TEST_INTEGRATION=1 bun x jest --coverage --maxWorkers=100% tests/ipcMain/ollama.test.ts
+        # TEST_OLLAMA=1 enables Ollama-specific tests
+        run: TEST_INTEGRATION=1 TEST_OLLAMA=1 bun x jest --coverage --maxWorkers=100% tests/ipcMain/ollama.test.ts
         env:
           OLLAMA_BASE_URL: http://localhost:11434/api
 

tests/ipcMain/ollama.test.ts

@@ -7,11 +7,13 @@ import {
 } from "./helpers";
 import { spawn } from "child_process";
 
-// Skip all tests if TEST_INTEGRATION is not set
-const describeIntegration = shouldRunIntegrationTests() ? describe : describe.skip;
+// Skip all tests if TEST_INTEGRATION or TEST_OLLAMA is not set
+const shouldRunOllamaTests = shouldRunIntegrationTests() && process.env.TEST_OLLAMA === "1";
+const describeOllama = shouldRunOllamaTests ? describe : describe.skip;
 
 // Ollama doesn't require API keys - it's a local service
 // Tests require Ollama to be running and will pull models idempotently
+// Set TEST_OLLAMA=1 to enable these tests
 
 const OLLAMA_MODEL = "gpt-oss:20b";
 
@@ -73,7 +75,7 @@ async function ensureOllamaModel(model: string): Promise<void> {
   });
 }
 
-describeIntegration("IpcMain Ollama integration tests", () => {
+describeOllama("IpcMain Ollama integration tests", () => {
   // Enable retries in CI for potential network flakiness with Ollama
   if (process.env.CI && typeof jest !== "undefined" && jest.retryTimes) {
     jest.retryTimes(3, { logErrorsBeforeRetry: true });