keycloak-uncached
Changes
forms/common-themes/src/main/resources/theme/base/admin/resources/partials/client-scope-mappings.html 8(+4 -4)
forms/common-themes/src/main/resources/theme/base/admin/resources/partials/client-service-account-roles.html 8(+4 -4)
testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/clients/mappers/CreateClientMappersForm.java 2(+0 -2)
testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/clients/scope/ClientScope.java 21(+19 -2)
testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/clients/scope/ClientScopeForm.java 41(+41 -0)
testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/roles/RoleCompositeRoles.java 10(+10 -0)
Details
diff --git a/forms/common-themes/src/main/resources/theme/base/admin/resources/partials/client-scope-mappings.html b/forms/common-themes/src/main/resources/theme/base/admin/resources/partials/client-scope-mappings.html
index 6c48016..cd63f49 100755
--- a/forms/common-themes/src/main/resources/theme/base/admin/resources/partials/client-scope-mappings.html
+++ b/forms/common-themes/src/main/resources/theme/base/admin/resources/partials/client-scope-mappings.html
@@ -76,9 +76,9 @@
</div>
<div class="row" data-ng-show="targetClient">
<div class="col-md-3">
- <label class="control-label" for="client-available">{{:: 'available-roles' | translate}}</label>
+ <label class="control-label" for="available-client">{{:: 'available-roles' | translate}}</label>
<kc-tooltip>{{:: 'assign.available-roles.tooltip' | translate}}</kc-tooltip>
- <select id="client-available" class="form-control" multiple size="5"
+ <select id="available-client" class="form-control" multiple size="5"
ng-multiple="true"
ng-model="selectedClientRoles"
ng-options="r.name for r in clientRoles">
@@ -88,9 +88,9 @@
</button>
</div>
<div class="col-md-3">
- <label class="control-label" for="client-assigned">{{:: 'assigned-roles' | translate}}</label>
+ <label class="control-label" for="assigned-client">{{:: 'assigned-roles' | translate}}</label>
<kc-tooltip>{{:: 'client.assigned-roles.tooltip' | translate}}</kc-tooltip>
- <select id="client-assigned" class="form-control" multiple size=5
+ <select id="assigned-client" class="form-control" multiple size=5
ng-multiple="true"
ng-model="selectedClientMappings"
ng-options="r.name for r in clientMappings">
diff --git a/forms/common-themes/src/main/resources/theme/base/admin/resources/partials/client-service-account-roles.html b/forms/common-themes/src/main/resources/theme/base/admin/resources/partials/client-service-account-roles.html
index bb38c6a..bc552da 100644
--- a/forms/common-themes/src/main/resources/theme/base/admin/resources/partials/client-service-account-roles.html
+++ b/forms/common-themes/src/main/resources/theme/base/admin/resources/partials/client-service-account-roles.html
@@ -65,9 +65,9 @@
</div>
<div class="row" data-ng-show="targetClient">
<div class="col-md-3">
- <label class="control-label" for="client-available">{{:: 'available-roles' | translate}}</label>
+ <label class="control-label" for="available-client">{{:: 'available-roles' | translate}}</label>
<kc-tooltip>{{:: 'assign.available-roles.tooltip' | translate}}</kc-tooltip>
- <select id="client-available" class="form-control" multiple size="5"
+ <select id="available-client" class="form-control" multiple size="5"
ng-multiple="true"
ng-model="selectedClientRoles"
ng-options="r.name for r in clientRoles">
@@ -77,9 +77,9 @@
</button>
</div>
<div class="col-md-3">
- <label class="control-label" for="client-assigned">{{:: 'assigned-roles' | translate}}</label>
+ <label class="control-label" for="assigned-client">{{:: 'assigned-roles' | translate}}</label>
<kc-tooltip>{{:: 'client.assigned-roles.tooltip' | translate}}</kc-tooltip>
- <select id="client-assigned" class="form-control" multiple size=5
+ <select id="assigned-client" class="form-control" multiple size=5
ng-multiple="true"
ng-model="selectedClientMappings"
ng-options="r.name for r in clientMappings">
diff --git a/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/clients/mappers/CreateClientMappersForm.java b/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/clients/mappers/CreateClientMappersForm.java
index ef533d2..fb4dacf 100644
--- a/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/clients/mappers/CreateClientMappersForm.java
+++ b/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/clients/mappers/CreateClientMappersForm.java
@@ -9,8 +9,6 @@ import org.openqa.selenium.support.ui.Select;
/**
* @author Vaclav Muzikar <vmuzikar@redhat.com>
- *
- * TODO: SAML
*/
public class CreateClientMappersForm extends Form {
diff --git a/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/clients/scope/ClientScopeForm.java b/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/clients/scope/ClientScopeForm.java
new file mode 100644
index 0000000..e76ef14
--- /dev/null
+++ b/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/clients/scope/ClientScopeForm.java
@@ -0,0 +1,41 @@
+/*
+ * JBoss, Home of Professional Open Source.
+ * Copyright 2012, Red Hat, Inc., and individual contributors
+ * as indicated by the @author tags. See the copyright.txt file in the
+ * distribution for a full listing of individual contributors.
+ *
+ * This is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU Lesser General Public License as
+ * published by the Free Software Foundation; either version 2.1 of
+ * the License, or (at your option) any later version.
+ *
+ * This software is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this software; if not, write to the Free
+ * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
+ * 02110-1301 USA, or see the FSF site: http://www.fsf.org.
+ */
+package org.keycloak.testsuite.console.page.clients.scope;
+
+import org.keycloak.testsuite.console.page.fragment.OnOffSwitch;
+import org.keycloak.testsuite.page.Form;
+import org.openqa.selenium.support.FindBy;
+
+/**
+ *
+ * @author <a href="mailto:vramik@redhat.com">Vlastislav Ramik</a>
+ */
+public class ClientScopeForm extends Form {
+
+ @FindBy(className = "onoffswitch")
+ private OnOffSwitch fullScopeAllowedSwitch;
+
+
+ public void setFullScopeAllowed(boolean value) {
+ fullScopeAllowedSwitch.setOn(value);
+ }
+}
diff --git a/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/roles/RoleCompositeRoles.java b/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/roles/RoleCompositeRoles.java
index 876b45b..1cb01bc 100644
--- a/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/roles/RoleCompositeRoles.java
+++ b/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/roles/RoleCompositeRoles.java
@@ -1,7 +1,9 @@
package org.keycloak.testsuite.console.page.roles;
import java.util.ArrayList;
+import java.util.Arrays;
import java.util.Collection;
+import java.util.Collections;
import java.util.HashMap;
import java.util.HashSet;
import java.util.List;
@@ -88,6 +90,14 @@ public class RoleCompositeRoles extends Form {
button.click();
}
+ public void addRealmRole(String role) {
+ addMissingRoles(availableRealmRolesSelect, addSelectedRealmRolesButton, Arrays.asList(role));
+ }
+
+ public void addClientRole(String role) {
+ addMissingRoles(availableClientRolesSelect, addSelectedClientRolesButton, Arrays.asList(role));
+ }
+
protected void addMissingRoles(Select select, WebElement button, Collection<String> roles) {
select.deselectAll();
if (roles != null) { // if roles not provided, don't add any
diff --git a/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/console/clients/ClientCredentialsTest.java b/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/console/clients/ClientCredentialsTest.java
index a4c4698..337eeac 100644
--- a/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/console/clients/ClientCredentialsTest.java
+++ b/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/console/clients/ClientCredentialsTest.java
@@ -41,39 +41,39 @@ public class ClientCredentialsTest extends AbstractClientTest {
private ClientRepresentation newClient;
@Page
- private ClientCredentials clientCredentials;
+ private ClientCredentials clientCredentialsPage;
@Page
private ClientCredentialsGeneratePrivateKeys generatePrivateKeysPage;
@Before
- public void beforeClientsTest() {
+ public void before() {
newClient = createOidcClientRep(CONFIDENTIAL, TEST_CLIENT_ID, TEST_REDIRECT_URIS);
testRealmResource().clients().create(newClient).close();
ClientRepresentation found = findClientByClientId(TEST_CLIENT_ID);
assertNotNull("Client " + TEST_CLIENT_ID + " was not found.", found);
- clientCredentials.setId(found.getId());
- clientCredentials.navigateTo();
+ clientCredentialsPage.setId(found.getId());
+ clientCredentialsPage.navigateTo();
}
@Test
public void regenerateSecret() {
- clientCredentials.form().selectClientIdAndSecret();
- clientCredentials.form().regenerateSecret();
+ clientCredentialsPage.form().selectClientIdAndSecret();
+ clientCredentialsPage.form().regenerateSecret();
assertFlashMessageSuccess();
}
@Test
public void regenerateRegistrationAccessToken() {
- clientCredentials.form().regenerateRegistrationAccessToken();
+ clientCredentialsPage.form().regenerateRegistrationAccessToken();
assertFlashMessageSuccess();
}
@Test
public void generateNewKeysAndCert() {
- generatePrivateKeysPage.setId(clientCredentials.getId());
- clientCredentials.form().selectSignedJwt();
- clientCredentials.form().generateNewKeysAndCert();
+ generatePrivateKeysPage.setId(clientCredentialsPage.getId());
+ clientCredentialsPage.form().selectSignedJwt();
+ clientCredentialsPage.form().generateNewKeysAndCert();
assertCurrentUrlEquals(generatePrivateKeysPage);
generatePrivateKeysPage.generateForm().setKeyPassword("pass");
diff --git a/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/console/clients/ClientScopeTest.java b/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/console/clients/ClientScopeTest.java
new file mode 100644
index 0000000..94d0db6
--- /dev/null
+++ b/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/console/clients/ClientScopeTest.java
@@ -0,0 +1,102 @@
+/*
+ * JBoss, Home of Professional Open Source.
+ * Copyright 2012, Red Hat, Inc., and individual contributors
+ * as indicated by the @author tags. See the copyright.txt file in the
+ * distribution for a full listing of individual contributors.
+ *
+ * This is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU Lesser General Public License as
+ * published by the Free Software Foundation; either version 2.1 of
+ * the License, or (at your option) any later version.
+ *
+ * This software is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this software; if not, write to the Free
+ * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
+ * 02110-1301 USA, or see the FSF site: http://www.fsf.org.
+ */
+package org.keycloak.testsuite.console.clients;
+
+import java.util.List;
+import java.util.Map;
+import org.jboss.arquillian.graphene.page.Page;
+import static org.junit.Assert.*;
+import org.junit.Before;
+import org.junit.Test;
+import org.keycloak.representations.idm.ClientMappingsRepresentation;
+import org.keycloak.representations.idm.ClientRepresentation;
+import org.keycloak.representations.idm.MappingsRepresentation;
+import org.keycloak.representations.idm.RoleRepresentation;
+import static org.keycloak.testsuite.console.clients.AbstractClientTest.createOidcClientRep;
+import static org.keycloak.testsuite.console.page.clients.CreateClientForm.OidcAccessType.CONFIDENTIAL;
+import org.keycloak.testsuite.console.page.clients.credentials.ClientCredentialsGeneratePrivateKeys;
+import org.keycloak.testsuite.console.page.clients.scope.ClientScope;
+
+/**
+ *
+ * @author <a href="mailto:vramik@redhat.com">Vlastislav Ramik</a>
+ */
+public class ClientScopeTest extends AbstractClientTest {
+
+ private ClientRepresentation newClient;
+ private ClientRepresentation found;
+
+ @Page
+ private ClientScope clientScopePage;
+ @Page
+ private ClientCredentialsGeneratePrivateKeys generatePrivateKeysPage;
+
+ @Before
+ public void before() {
+ newClient = createOidcClientRep(CONFIDENTIAL, TEST_CLIENT_ID, TEST_REDIRECT_URIS);
+ testRealmResource().clients().create(newClient).close();
+
+ found = findClientByClientId(TEST_CLIENT_ID);
+ assertNotNull("Client " + TEST_CLIENT_ID + " was not found.", found);
+ clientScopePage.setId(found.getId());
+ clientScopePage.navigateTo();
+ }
+
+ @Test
+ public void clientScopeTest() {
+ assertTrue(found.isFullScopeAllowed());
+ clientScopePage.scopeForm().setFullScopeAllowed(false);
+ assertFlashMessageSuccess();
+
+ found = findClientByClientId(TEST_CLIENT_ID);
+ assertFalse(found.isFullScopeAllowed());
+ assertNull(getAllMappingsRepresentation().getRealmMappings());
+ assertNull(getAllMappingsRepresentation().getClientMappings());
+
+ clientScopePage.roleForm().addRealmRole("offline_access");
+ assertFlashMessageSuccess();
+
+ clientScopePage.roleForm().selectClientRole("account");
+ clientScopePage.roleForm().addClientRole("view-profile");
+ assertFlashMessageSuccess();
+
+ found = findClientByClientId(TEST_CLIENT_ID);
+ List<RoleRepresentation> realmMappings = getAllMappingsRepresentation().getRealmMappings();
+ assertEquals(1, realmMappings.size());
+ assertEquals("offline_access", realmMappings.get(0).getName());
+ Map<String, ClientMappingsRepresentation> clientMappings = getAllMappingsRepresentation().getClientMappings();
+ assertEquals(1, clientMappings.size());
+ assertEquals("view-profile", clientMappings.get("account").getMappings().get(0).getName());
+
+ clientScopePage.roleForm().removeAssignedRole("offline_access");
+ assertFlashMessageSuccess();
+ clientScopePage.roleForm().removeAssignedClientRole("view-profile");
+ assertFlashMessageSuccess();
+
+ assertNull(getAllMappingsRepresentation().getRealmMappings());
+ assertNull(getAllMappingsRepresentation().getClientMappings());
+ }
+
+ private MappingsRepresentation getAllMappingsRepresentation() {
+ return testRealmResource().clients().get(found.getId()).getScopeMappings().getAll();
+ }
+}