Fix testSetAdditionalRolesCanAddDeprecatedMasterRole() by removing th…

…e initial assertion Signed-off-by: Tianli Feng <ftianli@amazon.com>
opensearch-project · May 25, 2022 · 86cc9a0 · 86cc9a0
1 parent 5320b68
commit 86cc9a0
Showing 1 changed file with 0 additions and 3 deletions.
diff --git a/server/src/test/java/org/opensearch/cluster/node/DiscoveryNodeTests.java b/server/src/test/java/org/opensearch/cluster/node/DiscoveryNodeTests.java
@@ -179,9 +179,6 @@ public void testDiscoveryNodeIsRemoteClusterClientUnset() {
     // as a workaround for making the new CLUSTER_MANAGER_ROLE has got the same abbreviation 'm'.
     // The test validate this behavior.
     public void testSetAdditionalRolesCanAddDeprecatedMasterRole() {
-        // Validate MASTER_ROLE is not in DiscoveryNodeRole.BUILT_IN_ROLES
-        assertFalse(DiscoveryNode.getPossibleRoleNames().contains(DiscoveryNodeRole.MASTER_ROLE.roleName()));
-
         DiscoveryNode.setAdditionalRoles(Collections.emptySet());
         assertTrue(DiscoveryNode.getPossibleRoleNames().contains(DiscoveryNodeRole.MASTER_ROLE.roleName()));
     }