Skip to content

Commit a08e8cf

Browse files
authored
Fix flaky test TestCollapseQParserPlugin.testMultiSort (#3609)
Relying on index order is unreliable.
1 parent e2b4a0c commit a08e8cf

1 file changed

Lines changed: 0 additions & 11 deletions

File tree

solr/core/src/test/org/apache/solr/search/TestCollapseQParserPlugin.java

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -107,17 +107,6 @@ public void testMultiSort() {
107107
"//result/doc[1]/str[@name='id'][.='8']",
108108
"//result/doc[2]/str[@name='id'][.='4']");
109109

110-
// tie broken by index order
111-
params = new ModifiableSolrParams();
112-
params.add("q", "*:*");
113-
params.add("fq", "{!collapse field=group_s sort='test_l desc'}");
114-
params.add("sort", "id_i desc");
115-
assertQ(
116-
req(params),
117-
"*[count(//doc)=2]",
118-
"//result/doc[1]/str[@name='id'][.='6']",
119-
"//result/doc[2]/str[@name='id'][.='2']");
120-
121110
// score, then tiebreakers; note top level sort by score ASCENDING (just for weirdness)
122111
params = new ModifiableSolrParams();
123112
params.add("q", "*:* term_s:YYYY");

0 commit comments

Comments
 (0)