keycloak-memoizeit
Changes
testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/auth/page/login/LoginActions.java 4(+2 -2)
testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/auth/page/login/LoginForm.java 2(+1 -1)
testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/fragment/Dropdown.java 11(+8 -3)
testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/page/AbstractPage.java 4(+4 -0)
testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/account/RegistrationTest.java 11(+5 -6)
testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/util/URLAssert.java 2(+1 -1)
testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/clients/credentials/ClientCredentialsForm.java 18(+11 -7)
testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/clients/scope/ClientScopeForm.java 2(+1 -1)
testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/realm/RealmSettings.java 10(+10 -0)
testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/users/User.java 7(+7 -0)
testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/users/Users.java 24(+18 -6)
testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/AbstractConsoleTest.java 4(+2 -2)
testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/clients/AbstractClientTest.java 6(+5 -1)
testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/federation/LdapUserFederationTest.java 6(+3 -3)
testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/realm/InternationalizationTest.java 3(+3 -0)
Details
diff --git a/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/auth/page/login/LoginActions.java b/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/auth/page/login/LoginActions.java
index 46ccb7d..484072c 100644
--- a/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/auth/page/login/LoginActions.java
+++ b/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/auth/page/login/LoginActions.java
@@ -40,11 +40,11 @@ public class LoginActions extends AuthRealm {
@FindBy(css = "div[id='kc-form-options'] span a")
private WebElement backToLoginForm;
- @FindBy(css = "span.kc-feedback-text")
+ @FindBy(xpath = "//span[@class='kc-feedback-text' and string-length(text())>1]")
private WebElement feedbackText;
public String getFeedbackText() {
- waitUntilElement(feedbackText, "Feedback message should be present").is().present();
+ waitUntilElement(feedbackText, "Feedback message should be present").is().visible();
return feedbackText.getText();
}
diff --git a/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/auth/page/login/LoginForm.java b/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/auth/page/login/LoginForm.java
index d29be72..85ad4b6 100644
--- a/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/auth/page/login/LoginForm.java
+++ b/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/auth/page/login/LoginForm.java
@@ -42,7 +42,7 @@ public class LoginForm extends Form {
// @FindBy(name = "cancel")
// private WebElement cancelButton;
- @FindBy(linkText = "Register")
+ @FindBy(xpath = "//div[@id='kc-registration']/span/a")
private WebElement registerLink;
@FindBy(linkText = "Forgot Password?")
private WebElement forgottenPassword;
diff --git a/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/fragment/Dropdown.java b/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/fragment/Dropdown.java
index 8661a04..070e6c7 100644
--- a/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/fragment/Dropdown.java
+++ b/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/console/page/fragment/Dropdown.java
@@ -6,6 +6,8 @@ import org.openqa.selenium.By;
import org.openqa.selenium.WebDriver;
import org.openqa.selenium.WebElement;
+import static org.keycloak.testsuite.util.WaitUtils.*;
+
/**
* @author Vaclav Muzikar <vmuzikar@redhat.com>
*/
@@ -17,11 +19,14 @@ public class Dropdown {
private WebDriver driver;
public String getSelected() {
- return dropDownRoot.findElement(By.xpath("./a")).getText();
+ waitUntilElement(dropDownRoot).is().present();
+ WebElement element = dropDownRoot.findElement(By.xpath("./a"));
+ return element.getText();
}
public void selectByText(String text) {
- String href = dropDownRoot.findElement(By.xpath("./ul/li/a[text()='" + text + "']")).getAttribute("href");
- driver.navigate().to(href);
+ waitUntilElement(dropDownRoot).is().present();
+ WebElement element = dropDownRoot.findElement(By.xpath("./ul/li/a[text()='" + text + "']"));
+ driver.navigate().to(element.getAttribute("href"));
}
}
diff --git a/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/page/AbstractPage.java b/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/page/AbstractPage.java
index 707f1f7..7af2375 100644
--- a/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/page/AbstractPage.java
+++ b/testsuite/integration-arquillian/tests/base/src/main/java/org/keycloak/testsuite/page/AbstractPage.java
@@ -24,6 +24,9 @@ import javax.ws.rs.core.UriBuilder;
import org.jboss.arquillian.drone.api.annotation.Drone;
import org.jboss.logging.Logger;
import static org.keycloak.testsuite.util.WaitUtils.pause;
+import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
+
+import org.openqa.selenium.By;
import org.openqa.selenium.WebDriver;
/**
@@ -90,6 +93,7 @@ public abstract class AbstractPage {
log.info("navigating to " + uri);
driver.navigate().to(uri);
pause(300); // this is needed for FF for some reason
+ waitUntilElement(By.tagName("body")).is().visible();
log.info("current URL: " + driver.getCurrentUrl());
}
diff --git a/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/account/RegistrationTest.java b/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/account/RegistrationTest.java
index f23741e..a49d823 100644
--- a/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/account/RegistrationTest.java
+++ b/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/account/RegistrationTest.java
@@ -70,8 +70,9 @@ public class RegistrationTest extends AbstractAccountManagementTest {
}
public void assertMessageAttributeMissing(String attributeName) {
- assertTrue(testRealmRegistrationPage.getFeedbackText()
- .contains("Please specify " + attributeName + "."));
+ String feedbackTest = testRealmRegistrationPage.getFeedbackText();
+ String contains = "Please specify " + attributeName + ".";
+ assertTrue("'" + feedbackTest + "' doesn't contain '" + contains + "'", feedbackTest.contains(contains));
}
@Test
@@ -84,8 +85,7 @@ public class RegistrationTest extends AbstractAccountManagementTest {
public void invalidEmail() {
newUser.setEmail("invalid.email.value");
testRealmRegistrationPage.register(newUser);
- assertTrue(testRealmRegistrationPage.getFeedbackText()
- .equals("Invalid email address."));
+ assertEquals("Invalid email address.", testRealmRegistrationPage.getFeedbackText());
assertUserDoesntExistWithAdminClient(newUser);
}
@@ -121,8 +121,7 @@ public class RegistrationTest extends AbstractAccountManagementTest {
public void notMatchingPasswords() {
testRealmRegistrationPage.setValues(newUser, "not-matching-password");
testRealmRegistrationPage.submit();
- assertTrue(testRealmRegistrationPage.getFeedbackText()
- .equals("Password confirmation doesn't match."));
+ assertEquals("Password confirmation doesn't match.", testRealmRegistrationPage.getFeedbackText());
testRealmRegistrationPage.register(newUser);
assertUserExistsWithAdminClient(newUser);
diff --git a/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/util/URLAssert.java b/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/util/URLAssert.java
index 0ff9fc6..c66fd70 100644
--- a/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/util/URLAssert.java
+++ b/testsuite/integration-arquillian/tests/base/src/test/java/org/keycloak/testsuite/util/URLAssert.java
@@ -47,7 +47,7 @@ public class URLAssert {
// }
// };
// wait.until(urlStartsWith);
- assertEqualsNormalized(driver.getCurrentUrl(), page.toString());
+ assertEqualsNormalized(page.toString(), driver.getCurrentUrl());
}
public static void assertCurrentUrlStartsWith(AbstractPage page) {
diff --git a/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/clients/credentials/ClientCredentialsForm.java b/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/clients/credentials/ClientCredentialsForm.java
index 991b175..51d1b24 100644
--- a/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/clients/credentials/ClientCredentialsForm.java
+++ b/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/clients/credentials/ClientCredentialsForm.java
@@ -25,6 +25,7 @@ import org.keycloak.testsuite.page.Form;
import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy;
import org.openqa.selenium.support.ui.Select;
+import static org.keycloak.testsuite.util.WaitUtils.*;
/**
*
@@ -34,15 +35,15 @@ public class ClientCredentialsForm extends Form {
@FindBy(id = "clientAuthenticatorType")
private Select clientAuthenticatorType;
-
- @FindBy(xpath = "//button[text()='Regenerate Secret']")
- private WebElement regenerateSecretButton;
- @FindBy(xpath = "//button[text()='Generate new keys and certificate']")
- private WebElement generateNewKeysAndCert;
+ @FindBy(xpath = "//button[@data-ng-click='changePassword()']")
+ private WebElement regenerateSecretButton; // Regenerate Secret
+
+ @FindBy(xpath = "//button[@data-ng-click='generateSigningKey()']")
+ private WebElement generateNewKeysAndCert; // Generate new keys and certificate
- @FindBy(xpath = "//button[text()='Regenerate registration access token']")
- private WebElement regenerateRegistrationAccessTokenButton;
+ @FindBy(xpath = "//button[@data-ng-click='regenerateRegistrationAccessToken()']")
+ private WebElement regenerateRegistrationAccessTokenButton; // Regenerate registration access token
public void selectClientIdAndSecret() {
clientAuthenticatorType.selectByVisibleText("Client Id and Secret");
@@ -53,14 +54,17 @@ public class ClientCredentialsForm extends Form {
}
public void regenerateSecret() {
+ waitUntilElement(regenerateSecretButton).is().visible();
regenerateSecretButton.click();
}
public void regenerateRegistrationAccessToken() {
+ waitUntilElement(regenerateRegistrationAccessTokenButton).is().visible();
regenerateRegistrationAccessTokenButton.click();
}
public void generateNewKeysAndCert() {
+ waitUntilElement(generateNewKeysAndCert).is().visible();
generateNewKeysAndCert.click();
}
}
diff --git a/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/clients/scope/ClientScopeForm.java b/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/clients/scope/ClientScopeForm.java
index e76ef14..588d78e 100644
--- a/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/clients/scope/ClientScopeForm.java
+++ b/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/clients/scope/ClientScopeForm.java
@@ -31,7 +31,7 @@ import org.openqa.selenium.support.FindBy;
*/
public class ClientScopeForm extends Form {
- @FindBy(className = "onoffswitch")
+ @FindBy(xpath = ".//div[@class='onoffswitch' and ./input[@id='fullScopeAllowed']]")
private OnOffSwitch fullScopeAllowedSwitch;
diff --git a/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/realm/RealmSettings.java b/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/realm/RealmSettings.java
index 347108d..2080962 100644
--- a/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/realm/RealmSettings.java
+++ b/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/realm/RealmSettings.java
@@ -4,6 +4,8 @@ import org.keycloak.testsuite.console.page.AdminConsoleRealm;
import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy;
+import static org.keycloak.testsuite.util.WaitUtils.*;
+
/**
*
* @author tkyjovsk
@@ -37,34 +39,42 @@ public class RealmSettings extends AdminConsoleRealm {
private WebElement defenseTab;
public void general() {
+ waitUntilElement(generalSettingsTab).is().present();
generalSettingsTab.click();
}
public void login() {
+ waitUntilElement(loginSettingsTab).is().present();
loginSettingsTab.click();
}
public void keys() {
+ waitUntilElement(keysSettingsTab).is().present();
keysSettingsTab.click();
}
public void email() {
+ waitUntilElement(emailSettingsTab).is().present();
emailSettingsTab.click();
}
public void themes() {
+ waitUntilElement(themeSettingsTab).is().present();
themeSettingsTab.click();
}
public void cache() {
+ waitUntilElement(cacheSettingsTab).is().present();
cacheSettingsTab.click();
}
public void tokens() {
+ waitUntilElement(tokenSettingsTab).is().present();
tokenSettingsTab.click();
}
public void securityDefenses() {
+ waitUntilElement(defenseTab).is().present();
defenseTab.click();
}
diff --git a/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/users/User.java b/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/users/User.java
index 7cea05c..3b73ea1 100644
--- a/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/users/User.java
+++ b/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/users/User.java
@@ -5,6 +5,8 @@ import static org.keycloak.testsuite.console.page.fragment.Breadcrumb.BREADCRUMB
import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy;
+import static org.keycloak.testsuite.util.WaitUtils.*;
+
/**
*
* @author tkyjovsk
@@ -54,22 +56,27 @@ public class User extends Users {
private WebElement sessionsLink;
public void attributes() {
+ waitUntilElement(attributesLink).is().present();
attributesLink.click();
}
public void credentials() {
+ waitUntilElement(consentsLink).is().present();
credentialsLink.click();
}
public void roleMappings() {
+ waitUntilElement(roleMappingsLink).is().present();
roleMappingsLink.click();
}
public void consents() {
+ waitUntilElement(consentsLink).is().present();
consentsLink.click();
}
public void sessions() {
+ waitUntilElement(sessionsLink).is().present();
sessionsLink.click();
}
diff --git a/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/users/Users.java b/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/users/Users.java
index 9e083a7..82a4906 100644
--- a/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/users/Users.java
+++ b/testsuite/integration-arquillian/tests/other/console/src/main/java/org/keycloak/testsuite/console/page/users/Users.java
@@ -17,6 +17,7 @@
*/
package org.keycloak.testsuite.console.page.users;
+import org.openqa.selenium.By;
import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy;
@@ -42,8 +43,8 @@ public class Users extends AdminConsoleRealm {
}
public static final String VIEW_ALL_USERS = "View all users";
- public static final String UNLOCK_USERS = "Unlock Users";
- public static final String ADD_USER = "Add User";
+ public static final String UNLOCK_USERS = "Unlock users";
+ public static final String ADD_USER = "Add user";
public static final String EDIT = "Edit";
public static final String IMPERSONATE = "Impersonate";
@@ -73,19 +74,22 @@ public class Users extends AdminConsoleRealm {
public void clickUser(String username) {
waitUntilElement(body()).is().present();
- body().findElement(linkText(username)).click();
+ WebElement link = body().findElement(
+ By.xpath(".//tr/td[./following::td[text()='" + username + "']]/a")
+ );
+ link.click();
}
public void editUser(String username) {
- clickRowActionButton(getRowByLinkText(username), EDIT);
+ clickRowActionButton(getRowByUsername(username), EDIT);
}
public void impersonateUser(String username) {
- clickRowActionButton(getRowByLinkText(username), IMPERSONATE);
+ clickRowActionButton(getRowByUsername(username), IMPERSONATE);
}
public void deleteUser(String username) {
- clickRowActionButton(getRowByLinkText(username), DELETE);
+ clickRowActionButton(getRowByUsername(username), DELETE);
modalDialog.confirmDeletion();
}
@@ -132,6 +136,14 @@ public class Users extends AdminConsoleRealm {
return users;
}
+ protected WebElement getRowByUsername(String userName) {
+ WebElement row = body().findElement(
+ By.xpath(".//tr[./td/following::td[text()='" + userName + "']]")
+ );
+ waitUntilElement(row).is().present();
+ return row;
+ }
+
}
}
diff --git a/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/AbstractConsoleTest.java b/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/AbstractConsoleTest.java
index 633b550..d061210 100644
--- a/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/AbstractConsoleTest.java
+++ b/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/AbstractConsoleTest.java
@@ -107,13 +107,13 @@ public abstract class AbstractConsoleTest extends AbstractAuthTest {
public void assertAlertSuccess() {
alert.waitUntilPresentAndClassSet();
- assertTrue(alert.isSuccess());
+ assertTrue("Is not success; @class=" + alert.getAttributeClass(), alert.isSuccess());
alert.close();
}
public void assertAlertDanger() {
alert.waitUntilPresentAndClassSet();
- assertTrue(alert.isDanger());
+ assertTrue("Is not danger; @class=" + alert.getAttributeClass(), alert.isDanger());
alert.close();
}
diff --git a/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/clients/AbstractClientTest.java b/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/clients/AbstractClientTest.java
index f646675..0f68266 100644
--- a/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/clients/AbstractClientTest.java
+++ b/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/clients/AbstractClientTest.java
@@ -43,6 +43,7 @@ import static org.keycloak.testsuite.util.AttributesAssert.assertEqualsBooleanAt
import static org.keycloak.testsuite.util.AttributesAssert.assertEqualsListAttributes;
import static org.keycloak.testsuite.util.AttributesAssert.assertEqualsStringAttributes;
import static org.keycloak.testsuite.util.URLAssert.assertCurrentUrlEquals;
+import static org.keycloak.testsuite.util.WaitUtils.pause;
/**
*
@@ -69,11 +70,14 @@ public abstract class AbstractClientTest extends AbstractConsoleTest {
}
public void createClient(ClientRepresentation client) {
- WaitUtils.waitUntilElement(By.tagName("body"));
+ WaitUtils.waitUntilElement(By.tagName("body")).is().present();
assertCurrentUrlEquals(clientsPage);
clientsPage.table().createClient();
createClientPage.form().setValues(client);
createClientPage.form().save();
+ assertAlertSuccess();
+
+ pause(500); // To ensure that the form will be loaded completely
clientSettingsPage.form().setValues(client);
if (SAML.equals(client.getProtocol())) {
diff --git a/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/federation/LdapUserFederationTest.java b/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/federation/LdapUserFederationTest.java
index be50411..a127e41 100644
--- a/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/federation/LdapUserFederationTest.java
+++ b/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/federation/LdapUserFederationTest.java
@@ -41,7 +41,7 @@ public class LdapUserFederationTest extends AbstractConsoleTest {
createLdapUserProvider.form().setLdapBindDnInput("KEYCLOAK/Administrator");
createLdapUserProvider.form().setLdapUserDnInput("ou=People,dc=keycloak,dc=org");
createLdapUserProvider.form().setLdapBindCredentialInput("secret");
- createLdapUserProvider.form().setAccountAfterPasswordUpdateEnabled(false);
+// createLdapUserProvider.form().setAccountAfterPasswordUpdateEnabled(false);
// enable kerberos
createLdapUserProvider.form().setAllowKerberosAuthEnabled(true);
createLdapUserProvider.form().setKerberosRealmInput("KEYCLOAK.ORG");
@@ -119,7 +119,7 @@ public class LdapUserFederationTest extends AbstractConsoleTest {
createLdapUserProvider.form().setLdapBindDnInput("uid=admin,ou=system");
createLdapUserProvider.form().setLdapUserDnInput("ou=People,dc=keycloak,dc=org");
createLdapUserProvider.form().setLdapBindCredentialInput("secret");
- createLdapUserProvider.form().setAccountAfterPasswordUpdateEnabled(true);
+// createLdapUserProvider.form().setAccountAfterPasswordUpdateEnabled(true);
createLdapUserProvider.form().save();
assertAlertSuccess();
LDAPEmbeddedServer ldapServer = null;
@@ -152,7 +152,7 @@ public class LdapUserFederationTest extends AbstractConsoleTest {
assertEquals(searchScope, ufpr.getConfig().get("searchScope"));
assertEquals(connectionPooling, ufpr.getConfig().get("connectionPooling"));
assertEquals(pagination, ufpr.getConfig().get("pagination"));
- assertEquals(enableAccountAfterPasswordUpdate, ufpr.getConfig().get("userAccountControlsAfterPasswordUpdate"));
+// assertEquals(enableAccountAfterPasswordUpdate, ufpr.getConfig().get("userAccountControlsAfterPasswordUpdate"));
}
private void assertLdapBasicMapping(UserFederationProviderRepresentation ufpr, String usernameLdapAttribute,
diff --git a/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/realm/InternationalizationTest.java b/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/realm/InternationalizationTest.java
index 7cb21a6..2dec996 100644
--- a/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/realm/InternationalizationTest.java
+++ b/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/realm/InternationalizationTest.java
@@ -29,6 +29,7 @@ public class InternationalizationTest extends AbstractRealmTest {
tabs().themes();
themeSettingsPage.setInternatEnabled(true);
themeSettingsPage.saveTheme();
+ assertAlertSuccess();
realmSettingsPage.setAdminRealm(AuthRealm.TEST);
accountPage.setAuthRealm(testRealmPage);
deleteAllCookiesForTestRealm();
@@ -76,11 +77,13 @@ public class InternationalizationTest extends AbstractRealmTest {
}
private void assertConsoleLocale(String expected) {
+ pause(500);
assertCurrentUrlEquals(realmSettingsPage);
assertLocale(".//a[contains(@href,'/theme-settings')]", expected); // Themes
}
private void assertAccountLocale(String expected) {
+ pause(500);
assertCurrentUrlEquals(accountPage);
assertLocale(".//div[contains(@class,'bs-sidebar')]/ul/li", expected); // Account
}
diff --git a/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/realm/SecurityDefensesTest.java b/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/realm/SecurityDefensesTest.java
index 72bcd7f..5792a73 100644
--- a/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/realm/SecurityDefensesTest.java
+++ b/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/realm/SecurityDefensesTest.java
@@ -173,7 +173,7 @@ public class SecurityDefensesTest extends AbstractRealmTest {
}
private void assertFeedbackText(String text) {
- waitUntilElement(feedbackTextElement);
+ waitUntilElement(feedbackTextElement).is().present();
assertEquals(text, feedbackTextElement.getText());
}
diff --git a/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/realm/TokensTest.java b/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/realm/TokensTest.java
index 975a619..9121d92 100644
--- a/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/realm/TokensTest.java
+++ b/testsuite/integration-arquillian/tests/other/console/src/test/java/org/keycloak/testsuite/console/realm/TokensTest.java
@@ -36,8 +36,8 @@ public class TokensTest extends AbstractRealmTest {
@Page
private TokenSettings tokenSettingsPage;
- private static final int TIMEOUT = 4;
- private static final TimeUnit TIME_UNIT = TimeUnit.SECONDS;
+ private static final int TIMEOUT = 1;
+ private static final TimeUnit TIME_UNIT = TimeUnit.MINUTES;
@Before
public void beforeTokensTest() {
@@ -52,7 +52,7 @@ public class TokensTest extends AbstractRealmTest {
tokenSettingsPage.form().save();
loginToTestRealmConsoleAs(testUser);
- TIME_UNIT.sleep(TIMEOUT + 2);
+ waitForTimeout(TIMEOUT + 2);
driver.navigate().refresh();
@@ -66,16 +66,22 @@ public class TokensTest extends AbstractRealmTest {
tokenSettingsPage.form().save();
loginToTestRealmConsoleAs(testUser);
- TIME_UNIT.sleep(TIMEOUT / 2);
+ waitForTimeout(TIMEOUT / 2);
driver.navigate().refresh();
assertCurrentUrlStartsWith(testRealmAdminConsolePage); // assert still logged in (within lifespan)
- TIME_UNIT.sleep(TIMEOUT / 2 + 2);
+ waitForTimeout(TIMEOUT / 2 + 2);
driver.navigate().refresh();
log.debug(driver.getCurrentUrl());
assertCurrentUrlStartsWithLoginUrlOf(testRealmPage); // assert logged out (lifespan exceeded)
}
+ private void waitForTimeout (int timeout) throws InterruptedException {
+ log.info("Wait for timeout: " + timeout + " " + TIME_UNIT);
+ TIME_UNIT.sleep(timeout);
+ log.info("Timeout reached");
+ }
+
}