Skip to content

Commit

Permalink
Fix LocalIndicesCleanerTests with hidden index (#53066)
Browse files Browse the repository at this point in the history
The LocalIndicesCleanerTests#testHandlesWatcherHistory creates random
watch history indices with a version of 1-20. When the version is 11,
this matches the existing watch history template that has the
index.hidden setting set to true and the check for indices in this test
did not include hidden indices. The change here updates the indices
options to include hidden indices.

Closes #53025
  • Loading branch information
jaymode authored Mar 4, 2020
1 parent 657809d commit 2930b32
Showing 1 changed file with 1 addition and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -57,12 +57,11 @@ protected void assertIndicesCount(int count) throws Exception {
//so when es core gets the request with the explicit index name, it throws an index not found exception as that index
//doesn't exist anymore. If we ignore unavailable instead no error will be thrown.
GetSettingsResponse getSettingsResponse = client().admin().indices().prepareGetSettings()
.setIndicesOptions(IndicesOptions.fromOptions(true, true, true, true)).get();
.setIndicesOptions(IndicesOptions.fromOptions(true, true, true, true, true)).get();
assertThat(getSettingsResponse.getIndexToSettings().size(), equalTo(count));
});
}

@AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/53025")
public void testHandlesWatcherHistory() throws Exception {
internalCluster().startNode();

Expand Down

0 comments on commit 2930b32

Please sign in to comment.