test(api-server): add real provenance-safe unit checks

Replace brittle pytest collection on the manual smoke script with a real unit suite for alias resolution and model catalog exposure. Align CI and provenance verification so both workflows use the same non-Ollama Python checks.

Author: root

.github/workflows/ci.yml

@@ -36,15 +36,12 @@ jobs:
       - name: Lint (ruff)
         run: ruff check .
 
-      - name: Collect tests (no Ollama required)
-        # test_api.py is an integration smoke test that requires a live Ollama server.
-        # We collect-only to confirm the file is importable and has no syntax errors.
-        # Set SKIP_INTEGRATION=1 as a convention for future test guards.
+      - name: Run unit tests (no Ollama required)
         env:
           SKIP_INTEGRATION: "1"
         run: |
           pip install pytest
-          python -m pytest test_api.py --collect-only -q || true
+          python -m pytest test_server.py -q
 
   chrome-bridge:
     name: chrome-bridge (Node.js)

.github/workflows/provenance.yml

@@ -70,7 +70,7 @@ jobs:
           pip install -r requirements.txt
           pip install pytest ruff
           ruff check .
-          python -m pytest test_api.py --collect-only -q
+          python -m pytest test_server.py -q
 
       - name: Validate chrome-bridge
         working-directory: chrome-bridge

api-server/test_api.py

@@ -1,4 +1,4 @@
-"""Quick test — calls both models and prints responses."""
+"""Manual integration smoke script for a live local LLM API server."""
 import httpx
 
 BASE = "http://localhost:9000"

api-server/test_server.py

@@ -0,0 +1,32 @@
+from fastapi import HTTPException
+from fastapi.testclient import TestClient
+
+from server import app, resolve
+
+
+def test_resolve_accepts_documented_aliases() -> None:
+    assert resolve("nano") == "gemma3:1b"
+    assert resolve("best") == "gemma3:12b"
+    assert resolve("qwen") == "qwen3:8b"
+
+
+def test_resolve_rejects_unknown_models() -> None:
+    try:
+        resolve("missing-model")
+    except HTTPException as exc:
+        assert exc.status_code == 404
+        assert "Unknown model" in str(exc.detail)
+    else:
+        raise AssertionError("resolve() should reject unknown models")
+
+
+def test_list_models_endpoint_exposes_catalog() -> None:
+    client = TestClient(app)
+
+    response = client.get("/v1/models")
+
+    assert response.status_code == 200
+    payload = response.json()
+    ids = {model["id"] for model in payload["data"]}
+    assert payload["object"] == "list"
+    assert {"gemma3:12b", "qwen3:8b", "gemma3:1b", "phi3:mini"} <= ids