Skip to content

test: make embedding tests offline#59

Merged
ramonlimaramos merged 1 commit into
mainfrom
codex/fix-embedding-tests-offline
Jun 14, 2026
Merged

test: make embedding tests offline#59
ramonlimaramos merged 1 commit into
mainfrom
codex/fix-embedding-tests-offline

Conversation

@ramonlimaramos

Copy link
Copy Markdown
Owner

Summary

  • replace the shared unit-test embedding fixture with a deterministic offline provider
  • keep SentenceTransformerProvider unit tests focused on the wrapper contract by stubbing model loading
  • make provider auto-selection deterministic in embedding tests by clearing OPENAI_API_KEY

Validation

  • uv run pytest tests/unit/test_embeddings.py -q
  • uv run --python 3.11 --extra dev pytest tests/unit/test_embeddings.py -q
  • uv run --python 3.11 --extra dev pytest tests/unit -q
  • uv run --python 3.11 --extra dev pytest tests/ -v --cov=synapto --cov-report=term-missing
  • uv run --python 3.11 --extra dev ruff check src/ tests/
  • uv run --python 3.11 --extra dev bandit -r src/synapto/ -c pyproject.toml
  • uv run --python 3.11 --extra dev pip-audit --ignore-vuln CVE-2025-3000

@ramonlimaramos ramonlimaramos merged commit b7681b3 into main Jun 14, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant