From 5ad05c93177882cf456439232803d4dca879625c Mon Sep 17 00:00:00 2001 From: Martin Reinhardt Date: Sat, 19 Oct 2019 04:50:32 +0200 Subject: [PATCH] [KEYCLOAK-6376] Directly create group --- .../account/custom/CustomAuthFlowOTPTest.java | 22 ++++++++----------- 1 file changed, 9 insertions(+), 13 deletions(-) diff --git a/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/account/custom/CustomAuthFlowOTPTest.java b/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/account/custom/CustomAuthFlowOTPTest.java index 28331871581..3ce1db6c161 100644 --- a/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/account/custom/CustomAuthFlowOTPTest.java +++ b/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/account/custom/CustomAuthFlowOTPTest.java @@ -372,19 +372,15 @@ public class CustomAuthFlowOTPTest extends AbstractCustomAccountManagementTest { } private GroupRepresentation getOrCreateOTPRoleInGroup() { - try { - return testRealmResource().groups().groups("otp_group",0,1).get(0); - } catch (NotFoundException | IndexOutOfBoundsException ex ) { - GroupRepresentation group = new GroupRepresentation(); - group.setName("otp_group"); - RoleRepresentation role = getOrCreateOTPRole(); - testRealmResource().groups().add(group); - //obtain id - GroupRepresentation groupRep = testRealmResource().groups().groups("otp_group",0,1).get(0); - testRealmResource().groups().group(groupRep.getId()).roles().realmLevel().add(Arrays.asList(role)); - //reread - return testRealmResource().groups().groups("otp_group",0,1).get(0); - } + GroupRepresentation group = new GroupRepresentation(); + group.setName("otp_group"); + RoleRepresentation role = getOrCreateOTPRole(); + testRealmResource().groups().add(group); + // obtain id + GroupRepresentation groupRep = testRealmResource().groups().groups("otp_group",0,1).get(0); + testRealmResource().groups().group(groupRep.getId()).roles().realmLevel().add(Arrays.asList(role)); + // reread + return testRealmResource().groups().groups("otp_group",0,1).get(0); } @Test