inteligr8/SearchServices
1
0
Fork 0
mirror of https://github.com/Alfresco/SearchServices.git synced 2026-09-16 18:12:56 +00:00
Code Issues Packages Projects Releases Wiki Activity

SEARCH-1681 adding switch statement instead of if for shard params

Browse Source
This commit is contained in:
Keerat
2020-02-10 12:01:07 +00:00
parent 5eaf0e5292
commit 242b9bce28
1 changed files with 14 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files
e2e-test/src/test/java/org/alfresco/test/search/functional/searchServices/search/ShardInfoTest.java
+14 -16
View File
@@ -168,22 +168,20 @@ public class ShardInfoTest extends AbstractE2EFunctionalTest
assertEquals(instance.getState(), "ACTIVE");
// shardparams related checks
String shardParams = instance.getShardParams();
if(shardingMethod == "EXPLICIT_ID"){
assertNotNull(shardParams, "shard.key=");
}
if(shardingMethod == "DB_ID_RANGE"){
assertNotNull(shardParams, "");
}
if(shardingMethod == "DATE"){
assertNotNull(shardParams, "shard.key= "
+ "shard.date.grouping");
}
if(shardingMethod == "PROPERTY"){
assertNotNull(shardParams, "shard.key=");
String shardParams = instance.getShardParams();
switch (shardingMethod)
{
case "DB_ID_RANGE":
assertTrue(shardParams.contains("shard.key="));
case "PROPERTY":
assertTrue(shardParams.contains("shard.key="));
case "EXPLICIT_ID":
assertTrue(shardParams.contains("shard.key="));
case "DATE":
assertTrue(shardParams.contains("shard.key="));
break;
default:
throw new AssertionError("Not as expected: " + shardParams.toString());
}
}
}