summaryrefslogtreecommitdiffstats
path: root/controller-api
diff options
context:
space:
mode:
authorJon Marius Venstad <jvenstad@yahoo-inc.com>2019-04-15 09:50:46 +0200
committerJon Marius Venstad <jvenstad@yahoo-inc.com>2019-04-15 10:16:26 +0200
commitc5c5015d4c4aeb1615f86a1b1aa3744bd3ff0722 (patch)
tree197ded919ddba283392b0f508317c8a4ca92a36f /controller-api
parentd0fe8b84ed98bf6cb294af8edda1f7d0bcd03e89 (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"));
}
}