libre.sh/keycloak-scim
3
0
Fork 0
Code Issues 15 Pull requests Releases 1 Packages Activity Actions

Merge pull request #3430 from stianst/KEYCLOAK-3819

Browse source 
KEYCLOAK-3819 Add support to change name of components
This commit is contained in:
Stian Thorgersen 2016-10-26 11:39:27 +02:00 committed by GitHub
commit 21459d0f43
2 changed files with 22 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
server-spi/src/main/java/org/keycloak/models/utils/RepresentationToModel.java
View file

@ -1746,6 +1746,10 @@ public class RepresentationToModel {
}
public static void updateComponent(KeycloakSession session, ComponentRepresentation rep, ComponentModel component, boolean internal) {
if (rep.getName() != null) {
component.setName(rep.getName());
}
if (rep.getParentId() != null) {
component.setParentId(rep.getParentId());
}

18
testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/admin/ComponentsTest.java
View file

@ -144,6 +144,24 @@ public class ComponentsTest extends AbstractAdminTest {
assertEquals(1, returned.getConfig().size());
}
@Test
public void testRename() {
ComponentRepresentation rep = createComponentRepresentation("mycomponent");
rep.getConfig().addFirst("required", "foo");
String id = createComponent(rep);
ComponentRepresentation returned = components.component(id).toRepresentation();
assertEquals("mycomponent", returned.getName());
rep.setName("myupdatedcomponent");
components.component(id).update(rep);
returned = components.component(id).toRepresentation();
assertEquals("myupdatedcomponent", returned.getName());
}
@Test
public void testSecretConfig() throws Exception {
ComponentRepresentation rep = createComponentRepresentation("mycomponent");