Skip to content

fix(tests): isolate the LangCache integration suite from its shared cache #725

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
vishal-bala wants to merge 1 commit into
base: main
Choose a base branch
from
+263 −183
Open

fix(tests): isolate the LangCache integration suite from its shared cache #725

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions CONTRIBUTING.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -177,6 +177,13 @@ uv run pytest --cov=redisvl --cov-report=html

**Note:** Tests requiring external APIs need appropriate API keys set as environment variables.

Where such an API is a *shared, stateful* service -- one managed instance reached by
every xdist worker and every concurrent CI run -- tests must namespace everything
they write and must never issue a whole-service flush, which would delete data
belonging to other workers and other pull requests' CI runs. See the module
docstring in `tests/integration/test_langcache_semantic_cache_integration.py` for a
worked example.

## Documentation

Documentation is served from the `docs/` directory and built using Sphinx.
Expand Down
Loading
Loading