(solr) branch main updated: Fix flaky test TestCollapseQParserPlugin.testMultiSort (#3609)

[dsmiley]

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 a08e8cf1d06 Fix flaky test TestCollapseQParserPlugin.testMultiSort 
(#3609)
a08e8cf1d06 is described below

commit a08e8cf1d0647339cdf5c6db5c349b2aaaee2b0a
Author: David Smiley <dsmi...@apache.org>
AuthorDate: Mon Sep 8 21:58:26 2025 -0400

    Fix flaky test TestCollapseQParserPlugin.testMultiSort (#3609)
    
    Relying on index order is unreliable.
---
 .../org/apache/solr/search/TestCollapseQParserPlugin.java     | 11 -----------
 1 file changed, 11 deletions(-)

diff --git 
a/solr/core/src/test/org/apache/solr/search/TestCollapseQParserPlugin.java 
b/solr/core/src/test/org/apache/solr/search/TestCollapseQParserPlugin.java
index 962186fd8b2..716a4a129bc 100644
--- a/solr/core/src/test/org/apache/solr/search/TestCollapseQParserPlugin.java
+++ b/solr/core/src/test/org/apache/solr/search/TestCollapseQParserPlugin.java
@@ -107,17 +107,6 @@ public class TestCollapseQParserPlugin extends 
SolrTestCaseJ4 {
         "//result/doc[1]/str[@name='id'][.='8']",
         "//result/doc[2]/str[@name='id'][.='4']");
 
-    // tie broken by index order
-    params = new ModifiableSolrParams();
-    params.add("q", "*:*");
-    params.add("fq", "{!collapse field=group_s sort='test_l desc'}");
-    params.add("sort", "id_i desc");
-    assertQ(
-        req(params),
-        "*[count(//doc)=2]",
-        "//result/doc[1]/str[@name='id'][.='6']",
-        "//result/doc[2]/str[@name='id'][.='2']");
-
     // score, then tiebreakers; note top level sort by score ASCENDING (just 
for weirdness)
     params = new ModifiableSolrParams();
     params.add("q", "*:* term_s:YYYY");