tests: skip vector-search integration tests when chromadb/sentence-transformers unavailable - #14
Merged
vicentebolea merged 1 commit intoAug 13, 2026
Conversation
…ansformers unavailable CI's test job never installs chromadb/sentence-transformers, so vector_search_vtk_examples always returned its "missing dependencies" message. The fixture only skipped on missing podman, so once podman became available on the runner these tests started failing outright instead of skipping.
vicentebolea
deleted the
fix/skip-vector-search-integration-without-deps
branch
August 13, 2026 00:33
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
testjob never installschromadb/sentence-transformers(they're not inpyproject.toml), sovector_search_vtk_examples(in the legacyvtk_mcp_serverpackage) always hit its own "missing dependencies" branch in this environment.embeddings_databasefixture intests/test_vector_search_integration.pyonly skipped whenpodmanwas unavailable. It used to pass because the runner didn't have podman (whole class skipped); now that podman is available on the runner, the fixture proceeds and the tests fail on assertions they were never able to satisfy in CI.pytest.importorskip("chromadb")/pytest.importorskip("sentence_transformers")to the fixture so it skips for the right reason, same pattern as the existing podman check.Test plan
pytest tests/test_vector_search_integration.py -v→ 3 skipped (chromadb not installed locally, matches CI)pytest -m integration -v→ 11 passed, 3 skipped, no failures