Skip to content

Commit 5099780

Browse files
author
Tianli Feng
authored
Fix testSetAdditionalRolesCanAddDeprecatedMasterRole() by removing the initial assertion (#3441)
Signed-off-by: Tianli Feng <ftianli@amazon.com>
1 parent 5320b68 commit 5099780

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

server/src/test/java/org/opensearch/cluster/node/DiscoveryNodeTests.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -179,9 +179,6 @@ public void testDiscoveryNodeIsRemoteClusterClientUnset() {
179179
// as a workaround for making the new CLUSTER_MANAGER_ROLE has got the same abbreviation 'm'.
180180
// The test validate this behavior.
181181
public void testSetAdditionalRolesCanAddDeprecatedMasterRole() {
182-
// Validate MASTER_ROLE is not in DiscoveryNodeRole.BUILT_IN_ROLES
183-
assertFalse(DiscoveryNode.getPossibleRoleNames().contains(DiscoveryNodeRole.MASTER_ROLE.roleName()));
184-
185182
DiscoveryNode.setAdditionalRoles(Collections.emptySet());
186183
assertTrue(DiscoveryNode.getPossibleRoleNames().contains(DiscoveryNodeRole.MASTER_ROLE.roleName()));
187184
}

0 commit comments

Comments
 (0)