applied code review changes

This commit is contained in:
cagache
2018-04-12 11:18:17 +03:00
parent 25feb9d5bb
commit fb2e857d02

View File

@@ -162,7 +162,8 @@ public class RMRolesAndActionsAPI extends BaseAPI
doPutJsonRequest(adminUser, adminPassword, HttpStatus.SC_OK, requestBody, RM_ROLES_ROLE, roleName); doPutJsonRequest(adminUser, adminPassword, HttpStatus.SC_OK, requestBody, RM_ROLES_ROLE, roleName);
} }
/** Delete a created RM role. /**
* Delete a created RM role.
* *
* @param adminUser The username of the admin user. * @param adminUser The username of the admin user.
* @param adminPassword The password for the admin user. * @param adminPassword The password for the admin user.
@@ -170,7 +171,7 @@ public class RMRolesAndActionsAPI extends BaseAPI
*/ */
public void deleteRole(String adminUser, String adminPassword, String roleName) public void deleteRole(String adminUser, String adminPassword, String roleName)
{ {
doDeleteRequest(adminUser, adminPassword, MessageFormat.format(RM_ROLES_ROLE, "{0}",roleName)); doDeleteRequest(adminUser, adminPassword, MessageFormat.format(RM_ROLES_ROLE, "{0}", roleName));
boolean success = !getConfiguredRoles(adminUser, adminPassword).contains(roleName); boolean success = !getConfiguredRoles(adminUser, adminPassword).contains(roleName);
assertTrue("Failed to delete role " + roleName + " with " + adminUser, success); assertTrue("Failed to delete role " + roleName + " with " + adminUser, success);
} }