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

CBL-5434: Vector Index API Tests 21-26 [Obj-C] #3247

Merged
merged 7 commits into from
Mar 8, 2024
Merged

CBL-5434: Vector Index API Tests 21-26 [Obj-C] #3247

merged 7 commits into from
Mar 8, 2024

Conversation

velicuvlad
Copy link
Contributor

@velicuvlad velicuvlad commented Mar 5, 2024

  • as per the last spec, the whole obj-c vector index api test suite is now complete

  • 16/26 are currently running, the rest being disabled and have their respective ticket at the start of the test (in a comment)

  • update TestVectorMatchOnNonExistingIndex off spec to check for consistency with fts

  • added 2 more index related tests to confirm behaviour - I think we can leave them here as they don't require the creation of a query such that to be moved under the query tests... and also not so many that we need to have a separate file for now

@velicuvlad velicuvlad requested a review from pasin March 5, 2024 01:29
@@ -6,6 +6,7 @@
// COUCHBASE CONFIDENTIAL -- part of Couchbase Lite Enterprise Edition
//

#import "CBLErrors.h"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is already included in CouchbaseLite.h which is included in the CBLTestCase.h so this is not needed.

@pasin pasin merged commit 4e50fc0 into master Mar 8, 2024
9 checks passed
@pasin pasin deleted the CBL-5434 branch March 8, 2024 01:29
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.

2 participants