Skip to content
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.

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

First cut at tests #11

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

bensteinberg
Copy link
Contributor

This is a first cut at tests. It only tests the ingest and visualization commands so far, not the web application or the API.

Because I had to move and restore an existing .env to get this to work, I think either I have something wrong with test set-up, or there's an infelicity with loading the application's config. TBD.

These tests should eventually exercise all of the application, possibly under different regimes, say, of VECTOR_SEARCH_SENTENCE_TRANSFORMER_MODEL and VECTOR_SEARCH_SENTENCE_TRANSFORMER_DEVICE. They could also track times for different ingests on specific hardware, and fail or warn when times go over baseline.

I haven't thought much yet about how to handle OLLAMA_API_URL in CI. There may not be a way to do that without standing up an external service.

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