This is an automated email from the ASF dual-hosted git repository.
dsmiley pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/solr.git
The following commit(s) were added to refs/heads/main by this push:
new 5c784013126 Test fix: flaky doc-order assertions in
TextToVectorUpdateProcessorTest (#4879)
5c784013126 is described below
commit 5c7840131265e233f09e40f149d300da04b6365c
Author: David Smiley <[email protected]>
AuthorDate: Thu Sep 10 09:56:21 2026 -0400
Test fix: flaky doc-order assertions in TextToVectorUpdateProcessorTest
(#4879)
Co-authored-by: Claude Sonnet 5 <[email protected]>
---
.../update/processor/TextToVectorUpdateProcessorTest.java | 11 +++++++----
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git
a/solr/modules/language-models/src/test/org/apache/solr/languagemodels/update/processor/TextToVectorUpdateProcessorTest.java
b/solr/modules/language-models/src/test/org/apache/solr/languagemodels/update/processor/TextToVectorUpdateProcessorTest.java
index 2b948786d2a..06711ae4919 100644
---
a/solr/modules/language-models/src/test/org/apache/solr/languagemodels/update/processor/TextToVectorUpdateProcessorTest.java
+++
b/solr/modules/language-models/src/test/org/apache/solr/languagemodels/update/processor/TextToVectorUpdateProcessorTest.java
@@ -77,6 +77,9 @@ public class TextToVectorUpdateProcessorTest extends
TestLanguageModelBase {
final SolrQuery query = new SolrQuery();
query.setQuery(solrQuery);
query.add("fl", "id,vector");
+ // *:* gives every doc the same score, so without an explicit sort the
order between
+ // tied docs is arbitrary (depends on segment/merge timing), which made
this flaky.
+ query.setSort("id", SolrQuery.ORDER.desc);
return query;
}
@@ -187,10 +190,10 @@ public class TextToVectorUpdateProcessorTest extends
TestLanguageModelBase {
assertJQ(
"/query" + query.toQueryString(),
"/response/numFound==2]",
- "/response/docs/[0]/id=='98'",
- "!/response/docs/[0]/vector==", // no vector field for document 98
- "/response/docs/[1]/id=='99'",
- "/response/docs/[1]/vector==[1.0, 2.0, 3.0, 4.0]");
+ "/response/docs/[0]/id=='99'",
+ "/response/docs/[0]/vector==[1.0, 2.0, 3.0, 4.0]",
+ "/response/docs/[1]/id=='98'",
+ "!/response/docs/[1]/vector=="); // no vector field for document 98
restTestHarness.delete(TextToVectorModelStore.REST_END_POINT + "/dummy-1");
}