Skip to content

Commit 2b67f16

Browse files
committed
Fix off by one error in test
./gradlew ':x-pack:plugin:searchable-snapshots:qa:rest:integTestRunner' \ --tests "org.elasticsearch.xpack.searchablesnapshots.rest.FsSearchableSnapshotsIT.testSearchResults" \ -Dtests.seed=5B626133268C101A \ -Dtests.security.manager=true \ -Dtests.locale=ro-RO \ -Dtests.timezone=America/Port-au-Prince \ -Dcompiler.java=13
1 parent d9b8249 commit 2b67f16

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

x-pack/plugin/searchable-snapshots/src/test/java/org/elasticsearch/xpack/searchablesnapshots/AbstractSearchableSnapshotsRestTestCase.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -206,12 +206,11 @@ public void testClearCache() throws Exception {
206206
}
207207

208208
public void assertSearchResults(String indexName, int numDocs, Boolean ignoreThrottled) throws IOException {
209-
final int randomTieBreaker = randomIntBetween(1, numDocs - 1);
209+
final int randomTieBreaker = randomIntBetween(0, numDocs - 1);
210210
Map<String, Object> searchResults;
211211
switch (randomInt(3)) {
212212
case 0:
213-
searchResults
214-
= search(indexName, QueryBuilders.termQuery("field", String.valueOf(randomTieBreaker)), ignoreThrottled);
213+
searchResults = search(indexName, QueryBuilders.termQuery("field", String.valueOf(randomTieBreaker)), ignoreThrottled);
215214
assertThat(extractValue(searchResults, "hits.total.value"), equalTo(1));
216215
@SuppressWarnings("unchecked")
217216
Map<String, Object> searchHit = (Map<String, Object>) ((List<?>) extractValue(searchResults, "hits.hits")).get(0);

0 commit comments

Comments
 (0)