honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix ClusterHealthResponsesTests condition (#49360) 

Currently the condtion that is supposed to test creation of test instances with
multiple indices is never true because it compares Strings with an enum. This
changes it so the condition uses the enum constants instead.
This commit is contained in:
Christoph Büscher 2019-11-21 17:14:01 +01:00
parent 779b4bd92b
commit 138d16ab9e
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
server/src/test/java/org/elasticsearch/action/admin/cluster/health/ClusterHealthResponsesTests.java
View File

@ -113,7 +113,7 @@ public class ClusterHealthResponsesTests extends AbstractSerializingTestCase<Clu
protected ClusterHealthResponse createTestInstance() { protected ClusterHealthResponse createTestInstance() {
int indicesSize = randomInt(20); int indicesSize = randomInt(20);
Map<String, ClusterIndexHealth> indices = new HashMap<>(indicesSize); Map<String, ClusterIndexHealth> indices = new HashMap<>(indicesSize);
if ("indices".equals(level) || "shards".equals(level)) { if (ClusterHealthRequest.Level.INDICES.equals(level) || ClusterHealthRequest.Level.SHARDS.equals(level)) {
for (int i = 0; i < indicesSize; i++) { for (int i = 0; i < indicesSize; i++) {
String indexName = randomAlphaOfLengthBetween(1, 5) + i; String indexName = randomAlphaOfLengthBetween(1, 5) + i;
indices.put(indexName, ClusterIndexHealthTests.randomIndexHealth(indexName, level)); indices.put(indexName, ClusterIndexHealthTests.randomIndexHealth(indexName, level));