Skip to content

Commit 41ed7b2

Browse files
author
Codex
committed
Fix ci workflow missing cryptography and run full pytest
The api CI job was hard-coding a subset of pip packages and only running two test files. After provider-key encryption landed in security.py, the test_user_auth_cache and test_provider_and_extraction_parsing tests import cryptography via api.services.security and the install step no longer provided it, causing collection errors and a red workflow on every recent push. Use requirements.txt as the install source of truth and run the full pytest suite under api/ so future tests can't silently miss deps.
1 parent 8e43251 commit 41ed7b2

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@ jobs:
1313
with:
1414
python-version: "3.11"
1515
- run: python -m compileall api
16-
- run: python -m pip install asyncpg==0.29.0 pydantic==2.7.0 pydantic-settings==2.3.0 httpx==0.27.0 pytest==8.2.2
17-
- run: python -m pytest api/test_user_auth_cache.py api/test_provider_and_extraction_parsing.py
16+
- run: python -m pip install -r api/requirements.txt
17+
- run: python -m pytest api
1818

1919
mcp:
2020
runs-on: ubuntu-latest

0 commit comments

Comments
 (0)