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

KEYCLOAK-4392 Copy component id from representation to model

Browse source
This commit is contained in:
Anders Båtstrand 2017-02-10 13:37:16 +01:00
parent 75909a0add
commit 3af9f2f989
2 changed files with 21 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -1727,6 +1727,7 @@ public class RepresentationToModel {
public static ComponentModel toModel(KeycloakSession session, ComponentRepresentation rep) { public static ComponentModel toModel(KeycloakSession session, ComponentRepresentation rep) {
ComponentModel model = new ComponentModel(); ComponentModel model = new ComponentModel();
model.setId(rep.getId());
model.setParentId(rep.getParentId()); model.setParentId(rep.getParentId());
model.setProviderType(rep.getProviderType()); model.setProviderType(rep.getProviderType());
model.setProviderId(rep.getProviderId()); model.setProviderId(rep.getProviderId());

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

@ -98,6 +98,26 @@ public class ComponentsTest extends AbstractAdminTest {
assertTrue(returned.getConfig().containsKey("required")); assertTrue(returned.getConfig().containsKey("required"));
} }
@Test
public void testCreateWithoutGivenId() {
ComponentRepresentation rep = createComponentRepresentation("mycomponent");
rep.getConfig().addFirst("required", "foo");
rep.setId(null);
String id = createComponent(rep);
assertNotNull(id);
}
@Test
public void testCreateWithGivenId() {
ComponentRepresentation rep = createComponentRepresentation("mycomponent");
rep.getConfig().addFirst("required", "foo");
rep.setId("fixed-id");
String id = createComponent(rep);
assertEquals("fixed-id", id);
}
@Test @Test
public void testUpdate() { public void testUpdate() {
ComponentRepresentation rep = createComponentRepresentation("mycomponent"); ComponentRepresentation rep = createComponentRepresentation("mycomponent");