Skip to content

Commit 13a5dde

Browse files
authored
[ESQL] Fix TopNSetTestCase test and unmute it (#129327)
Closes #129257
1 parent 32e6873 commit 13a5dde

File tree

2 files changed

+1
-4
lines changed

2 files changed

+1
-4
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -520,9 +520,6 @@ tests:
520520
- class: org.elasticsearch.xpack.esql.qa.single_node.GenerativeForkIT
521521
method: test {lookup-join.MultipleBatches*
522522
issue: https://github.com/elastic/elasticsearch/issues/129210
523-
- class: org.elasticsearch.compute.data.sort.LongTopNSetTests
524-
method: testCrankyBreaker
525-
issue: https://github.com/elastic/elasticsearch/issues/129257
526523

527524
# Examples:
528525
#

x-pack/plugin/esql/compute/src/test/java/org/elasticsearch/compute/data/sort/TopNSetTestCase.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,7 @@ public final void testCrankyBreaker() {
189189
collect(sort, value);
190190
}
191191

192-
assertResults(sort, sortOrder, limit - 1, values);
192+
assertResults(sort, sortOrder, limit, values);
193193
} catch (CircuitBreakingException e) {
194194
assertThat(e.getMessage(), equalTo(CrankyCircuitBreakerService.ERROR_MESSAGE));
195195
}

0 commit comments

Comments
 (0)