diff options
author | Jon Marius Venstad <jvenstad@yahoo-inc.com> | 2019-04-15 09:50:46 +0200 |
---|---|---|
committer | Jon Marius Venstad <jvenstad@yahoo-inc.com> | 2019-04-15 10:16:26 +0200 |
commit | c5c5015d4c4aeb1615f86a1b1aa3744bd3ff0722 (patch) | |
tree | 197ded919ddba283392b0f508317c8a4ca92a36f /controller-api | |
parent | d0fe8b84ed98bf6cb294af8edda1f7d0bcd03e89 (diff) |
UserRoles -> Roles
Diffstat (limited to 'controller-api')
-rw-r--r-- | controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/user/Roles.java (renamed from controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/user/UserRoles.java) | 4 | ||||
-rw-r--r-- | controller-api/src/test/java/com/yahoo/vespa/hosted/controller/api/integration/user/RolesTest.java (renamed from controller-api/src/test/java/com/yahoo/vespa/hosted/controller/api/integration/user/UserRolesTest.java) | 30 |
2 files changed, 17 insertions, 17 deletions
diff --git a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/user/UserRoles.java b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/user/Roles.java index 239d7216491..5f63ad1f185 100644 --- a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/user/UserRoles.java +++ b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/user/Roles.java @@ -14,10 +14,10 @@ import java.util.List; * * @author jonmv */ -public class UserRoles { +public class Roles { /** Creates a new UserRoles which can be used for serialisation and listing of bound user roles. */ - public UserRoles() { } + public Roles() { } /** Returns the list of {@link TenantRole}s a {@link UserId} may be a member of. */ public List<TenantRole> tenantRoles(TenantName tenant) { diff --git a/controller-api/src/test/java/com/yahoo/vespa/hosted/controller/api/integration/user/UserRolesTest.java b/controller-api/src/test/java/com/yahoo/vespa/hosted/controller/api/integration/user/RolesTest.java index c8e3d1987c9..d43793462ad 100644 --- a/controller-api/src/test/java/com/yahoo/vespa/hosted/controller/api/integration/user/UserRolesTest.java +++ b/controller-api/src/test/java/com/yahoo/vespa/hosted/controller/api/integration/user/RolesTest.java @@ -12,59 +12,59 @@ import static org.junit.Assert.assertEquals; /** * @author jonmv */ -public class UserRolesTest { +public class RolesTest { - private static final UserRoles userRoles = new UserRoles(); + private static final Roles roles = new Roles(); @Test public void testSerialization() { TenantName tenant = TenantName.from("my-tenant"); - for (TenantRole role : userRoles.tenantRoles(tenant)) - assertEquals(role, userRoles.toRole(UserRoles.valueOf(role))); + for (TenantRole role : roles.tenantRoles(tenant)) + assertEquals(role, roles.toRole(Roles.valueOf(role))); ApplicationName application = ApplicationName.from("my-application"); - for (ApplicationRole role : userRoles.applicationRoles(tenant, application)) - assertEquals(role, userRoles.toRole(UserRoles.valueOf(role))); + for (ApplicationRole role : roles.applicationRoles(tenant, application)) + assertEquals(role, roles.toRole(Roles.valueOf(role))); assertEquals(Role.tenantOperator(tenant), - userRoles.toRole("my-tenant.tenantOperator")); + roles.toRole("my-tenant.tenantOperator")); assertEquals(Role.applicationReader(tenant, application), - userRoles.toRole("my-tenant.my-application.applicationReader")); + roles.toRole("my-tenant.my-application.applicationReader")); } @Test(expected = IllegalArgumentException.class) public void illegalTenantName() { - UserRoles.valueOf(Role.tenantAdmin(TenantName.from("my.tenant"))); + Roles.valueOf(Role.tenantAdmin(TenantName.from("my.tenant"))); } @Test(expected = IllegalArgumentException.class) public void illegalApplicationName() { - UserRoles.valueOf(Role.applicationOperator(TenantName.from("my-tenant"), ApplicationName.from("my.app"))); + Roles.valueOf(Role.applicationOperator(TenantName.from("my-tenant"), ApplicationName.from("my.app"))); } @Test(expected = IllegalArgumentException.class) public void illegalRole() { - UserRoles.valueOf(Role.tenantPipeline(TenantName.from("my-tenant"), ApplicationName.from("my-app"))); + Roles.valueOf(Role.tenantPipeline(TenantName.from("my-tenant"), ApplicationName.from("my-app"))); } @Test(expected = IllegalArgumentException.class) public void illegalRoleValue() { - userRoles.toRole("my-tenant.awesomePerson"); + roles.toRole("my-tenant.awesomePerson"); } @Test(expected = IllegalArgumentException.class) public void illegalCombination() { - userRoles.toRole("my-tenant.my-application.tenantOwner"); + roles.toRole("my-tenant.my-application.tenantOwner"); } @Test(expected = IllegalArgumentException.class) public void illegalValue() { - userRoles.toRole("everyone"); + roles.toRole("everyone"); } @Test public void allowHostedOperator() { - assertEquals(Role.hostedOperator(), userRoles.toRole("hostedOperator")); + assertEquals(Role.hostedOperator(), roles.toRole("hostedOperator")); } } |