JAVA-6099 Add new fields for Auto embedding#1936
JAVA-6099 Add new fields for Auto embedding#1936strogiyotec wants to merge 5 commits intomongodb:mainfrom
Conversation
|
Assigned |
There was a problem hiding this comment.
Pull request overview
Adds functional coverage for new Atlas Search “autoEmbed” index fields and related vector search behaviors in the sync driver test suite (JAVA-6099), primarily by expanding the existing automated-embedding functional test class.
Changes:
- Adds new functional tests for auto-embedding search index creation scenarios (quantization variants, filter fields, invalid configurations).
- Adds query coverage for model override and exact vector search options.
- Introduces a disabled (intended-to-skip) test case for
numDimensionsonautoEmbed.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| */ | ||
| public abstract class AbstractAutomatedEmbeddingVectorSearchFunctionalTest extends OperationTest { | ||
| @TestInstance(TestInstance.Lifecycle.PER_CLASS) | ||
| public abstract class AbstractAutoEmbeddingVectorSearchFunctionalTest extends OperationTest { |
There was a problem hiding this comment.
Do we need to extend OperationTest?
OperationTest has a @BeforeEach that calls CollectionHelper.drop(getNamespace()). With @TestInstance(PER_CLASS), JUnit 5 still invokes inherited @BeforeEach before each test method - so when Assumptions.assumeTrue(false) is removed (TODO-JAVA-6059), the collection will be dropped before each test.
JAVA-6099
The doc in JIRA list new fields that must be supported by the drivers
but because the Search index doesn't use builders, java driver makes it possible to use new fields without changing API
As part of this PR I added a new unit test to disabled test class and enabled it locally for testing
The reason it's disabled is
For other details please check the JIRA ticket comment