This is an automated email from the ASF dual-hosted git repository. dsmiley pushed a commit to branch branch_10x in repository https://gitbox.apache.org/repos/asf/solr.git
commit 68b226dfc18c529182442216f175e854e885b22d 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]> (cherry picked from commit 5c7840131265e233f09e40f149d300da04b6365c) --- .../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"); }
