From fa496b3326c7ff709498fdbd39008043ae8be5ff Mon Sep 17 00:00:00 2001 From: Armin Braun Date: Tue, 9 Feb 2021 12:52:09 +0100 Subject: [PATCH] Fix off by one Mistake in SparseFileTrackerTests This was caused by #68709 which turned the edge case of a range with start > end into a tripped assertion. --- .../elasticsearch/index/store/cache/SparseFileTrackerTests.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/x-pack/plugin/searchable-snapshots/src/test/java/org/elasticsearch/index/store/cache/SparseFileTrackerTests.java b/x-pack/plugin/searchable-snapshots/src/test/java/org/elasticsearch/index/store/cache/SparseFileTrackerTests.java index 2651fb8f645c9..54b43a9ad86a1 100644 --- a/x-pack/plugin/searchable-snapshots/src/test/java/org/elasticsearch/index/store/cache/SparseFileTrackerTests.java +++ b/x-pack/plugin/searchable-snapshots/src/test/java/org/elasticsearch/index/store/cache/SparseFileTrackerTests.java @@ -71,7 +71,7 @@ public void testInvalidRange() { if (length > 1L) { e = expectThrows(IllegalArgumentException.class, () -> { long start = randomLongBetween(1L, Math.max(1L, length - 1L)); - long end = randomLongBetween(length, length + 1000L); + long end = randomLongBetween(length + 1, length + 1000L); sparseFileTracker.waitForRange(ByteRange.of(start, end), null, listener); }); assertThat("end must not be greater than length", e.getMessage(), containsString("invalid range"));