Skip to content

Astra document store filter tests are skipped #869

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

Jump to bottom
Open
shadeMe opened this issue Jul 4, 2024 · 1 comment · May be fixed by #1000
Open

Astra document store filter tests are skipped #869

shadeMe opened this issue Jul 4, 2024 · 1 comment · May be fixed by #1000
Assignees
@vblagoje
Labels
bug Something isn't working integration:astra P2

Comments

@shadeMe
Copy link
Contributor

shadeMe commented Jul 4, 2024

Describe the bug
The tests in the Astra core integration override the common document store/filters test suite and mark them for skipping. We need to remove those overrides to ensure that we fully test the integrations supported filters.
@shadeMe shadeMe added bug Something isn't working integration:astra labels Jul 4, 2024
@shadeMe shadeMe added the P2 label Jul 15, 2024
@vblagoje vblagoje self-assigned this Aug 15, 2024
@vblagoje vblagoje linked a pull request Aug 16, 2024 that will close this issue
Draft
@vblagoje vblagoje added the contributions wanted! Looking for external contributions label Sep 2, 2024
@vblagoje
Copy link
Member

vblagoje commented Sep 2, 2024

Currently on hold due to missing operations in Astra. See #1000 for more details.

@anakin87 anakin87 removed the contributions wanted! Looking for external contributions label Apr 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vblagoje vblagoje

Labels
bug Something isn't working integration:astra P2
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

draft: Remove astra test overrides deepset-ai/haystack-core-integrations
3 participants
@shadeMe @vblagoje @anakin87