Skip to content

Commit

Permalink
[test] Fix RecoverySourcePruneMergePolicyTests#testPruneSome (elastic…
Browse files Browse the repository at this point in the history
…#118944)

The `extra_source_size` field is set to a value between 10 and 10000
inclusive, so the assertion should be `greaterThanOrEqualTo(10)` rather
than `greaterThan(10)`.

See elastic#114618 Resolve elastic#118728
  • Loading branch information
arteam authored and rjernst committed Dec 18, 2024
1 parent c7fd73b commit 2d929e6
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@
import java.util.stream.Collectors;

import static org.hamcrest.Matchers.equalTo;
import static org.hamcrest.Matchers.greaterThan;
import static org.hamcrest.Matchers.greaterThanOrEqualTo;

public class RecoverySourcePruneMergePolicyTests extends ESTestCase {

Expand Down Expand Up @@ -191,7 +191,7 @@ public void testPruneSome() throws IOException {
}
assertEquals(i, extra_source.docID());
if (syntheticRecoverySource) {
assertThat(extra_source.longValue(), greaterThan(10L));
assertThat(extra_source.longValue(), greaterThanOrEqualTo(10L));
} else {
assertThat(extra_source.longValue(), equalTo(1L));
}
Expand Down

0 comments on commit 2d929e6

Please sign in to comment.