Merge pull request #1903 from tkyjovsk/master

Increased pageload timeout. Increased other timeouts for travis. Better handling of closeable alert messages.
This commit is contained in:
Tomas Kyjovsky 2015-12-07 11:09:37 +01:00
commit 4a57bcc0b4
61 changed files with 380 additions and 338 deletions

View file

@ -41,5 +41,4 @@
</pluginManagement> </pluginManagement>
</build> </build>
</project> </project>

View file

@ -5,7 +5,7 @@ import org.jboss.arquillian.container.test.api.OperateOnDeployment;
import org.jboss.arquillian.graphene.findby.FindByJQuery; import org.jboss.arquillian.graphene.findby.FindByJQuery;
import org.jboss.arquillian.test.api.ArquillianResource; import org.jboss.arquillian.test.api.ArquillianResource;
import org.keycloak.testsuite.page.AbstractPageWithInjectedUrl; import org.keycloak.testsuite.page.AbstractPageWithInjectedUrl;
import org.keycloak.testsuite.util.WaitUtils; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
/** /**
@ -61,7 +61,7 @@ public class CustomerPortalExample extends AbstractPageWithInjectedUrl {
} }
public void customerSession() { public void customerSession() {
WaitUtils.waitGuiForElement(customerSessionLink); waitUntilElement(customerSessionLink).is().present();
customerSessionLink.click(); customerSessionLink.click();
} }
@ -70,11 +70,11 @@ public class CustomerPortalExample extends AbstractPageWithInjectedUrl {
} }
public void waitForCustomerListingHeader() { public void waitForCustomerListingHeader() {
WaitUtils.waitGuiForElementNotPresent(customerListingHeader); waitUntilElement(customerListingHeader).is().not().present();
} }
public void waitForCustomerSessionHeader() { public void waitForCustomerSessionHeader() {
WaitUtils.waitGuiForElementNotPresent(customerSessionHeader); waitUntilElement(customerSessionHeader).is().not().present();
} }
} }

View file

@ -5,7 +5,7 @@ import org.jboss.arquillian.container.test.api.OperateOnDeployment;
import org.jboss.arquillian.graphene.findby.FindByJQuery; import org.jboss.arquillian.graphene.findby.FindByJQuery;
import org.jboss.arquillian.test.api.ArquillianResource; import org.jboss.arquillian.test.api.ArquillianResource;
import org.keycloak.testsuite.page.AbstractPageWithInjectedUrl; import org.keycloak.testsuite.page.AbstractPageWithInjectedUrl;
import org.keycloak.testsuite.util.WaitUtils; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
/** /**
@ -48,7 +48,7 @@ public class ProductPortalExample extends AbstractPageWithInjectedUrl {
} }
public void waitForProductListingHeader() { public void waitForProductListingHeader() {
WaitUtils.waitGuiForElementNotPresent(productListingHeader); waitUntilElement(productListingHeader).is().not().present();
} }
public void logOut() { public void logOut() {

View file

@ -2,8 +2,7 @@ package org.keycloak.testsuite.auth.page.account;
import org.keycloak.representations.idm.UserRepresentation; import org.keycloak.representations.idm.UserRepresentation;
import org.keycloak.testsuite.page.Form; import org.keycloak.testsuite.page.Form;
import static org.keycloak.testsuite.util.WaitUtils.waitAjaxForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import static org.keycloak.testsuite.util.WaitUtils.waitAjaxForElementNotPresent;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy; import org.openqa.selenium.support.FindBy;
@ -49,11 +48,11 @@ public class AccountFields extends Form {
} }
public void waitForUsernameInputPresent() { public void waitForUsernameInputPresent() {
waitAjaxForElement(usernameInput); waitUntilElement(usernameInput).is().present();
} }
public void waitForUsernameInputNotPresent() { public void waitForUsernameInputNotPresent() {
waitAjaxForElementNotPresent(usernameInput); waitUntilElement(usernameInput).is().not().present();
} }
} }

View file

@ -21,7 +21,7 @@ import javax.ws.rs.core.UriBuilder;
import org.jboss.arquillian.graphene.findby.FindByJQuery; import org.jboss.arquillian.graphene.findby.FindByJQuery;
import org.keycloak.admin.client.resource.RealmResource; import org.keycloak.admin.client.resource.RealmResource;
import org.keycloak.testsuite.auth.page.AuthRealm; import org.keycloak.testsuite.auth.page.AuthRealm;
import static org.keycloak.testsuite.util.WaitUtils.waitGuiForElementPresent; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy; import org.openqa.selenium.support.FindBy;
@ -66,7 +66,7 @@ public class AccountManagement extends AuthRealm {
private WebElement error; private WebElement error;
public String getErrorMessage() { public String getErrorMessage() {
waitGuiForElementPresent(error, "Error message should be visible"); waitUntilElement(error, "Error message should be present").is().present();
return error.getText(); return error.getText();
} }
@ -107,6 +107,6 @@ public class AccountManagement extends AuthRealm {
} }
public void waitForAccountLinkPresent() { public void waitForAccountLinkPresent() {
waitGuiForElementPresent(accountLink, "account link should be present"); waitUntilElement(accountLink, "account link should be present").is().present();
} }
} }

View file

@ -18,7 +18,7 @@
package org.keycloak.testsuite.auth.page.account; package org.keycloak.testsuite.auth.page.account;
import org.keycloak.testsuite.page.Form; import org.keycloak.testsuite.page.Form;
import org.keycloak.testsuite.util.WaitUtils; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy; import org.openqa.selenium.support.FindBy;
@ -54,6 +54,6 @@ public class PasswordFields extends Form {
} }
public void waitForConfirmPasswordInputPresent() { public void waitForConfirmPasswordInputPresent() {
WaitUtils.waitGuiForElement(confirmPasswordInput); waitUntilElement(confirmPasswordInput).is().present();
} }
} }

View file

@ -0,0 +1,32 @@
/*
* JBoss, Home of Professional Open Source
*
* Copyright 2013 Red Hat, Inc. and/or its affiliates.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package org.keycloak.testsuite.auth.page.account.fragment;
import org.keycloak.testsuite.page.AbstractAlert;
/**
*
* @author tkyjovsk
*/
public class AccountManagementAlert extends AbstractAlert {
public boolean isError() {
return getAttributeClass().contains("alert-error");
}
}

View file

@ -20,8 +20,7 @@ package org.keycloak.testsuite.auth.page.login;
import javax.ws.rs.core.UriBuilder; import javax.ws.rs.core.UriBuilder;
import org.jboss.arquillian.graphene.page.Page; import org.jboss.arquillian.graphene.page.Page;
import org.keycloak.testsuite.auth.page.AuthRealm; import org.keycloak.testsuite.auth.page.AuthRealm;
import static org.keycloak.testsuite.util.WaitUtils.waitGuiForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import static org.keycloak.testsuite.util.WaitUtils.waitGuiForElementNotPresent;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy; import org.openqa.selenium.support.FindBy;
@ -62,11 +61,11 @@ public abstract class Login extends AuthRealm {
private WebElement keycloakTheme; private WebElement keycloakTheme;
public void waitForKeycloakThemeNotPresent() { public void waitForKeycloakThemeNotPresent() {
waitGuiForElementNotPresent(keycloakTheme); waitUntilElement(keycloakTheme).is().not().present();
} }
public void waitForKeycloakThemePresent() { public void waitForKeycloakThemePresent() {
waitGuiForElement(keycloakTheme); waitUntilElement(keycloakTheme).is().present();
} }
} }

View file

@ -19,7 +19,7 @@ package org.keycloak.testsuite.auth.page.login;
import javax.ws.rs.core.UriBuilder; import javax.ws.rs.core.UriBuilder;
import org.keycloak.testsuite.auth.page.AuthRealm; import org.keycloak.testsuite.auth.page.AuthRealm;
import static org.keycloak.testsuite.util.WaitUtils.waitGuiForElementPresent; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy; import org.openqa.selenium.support.FindBy;
@ -45,7 +45,7 @@ public class LoginActions extends AuthRealm {
private WebElement feedbackText; private WebElement feedbackText;
public String getFeedbackText() { public String getFeedbackText() {
waitGuiForElementPresent(feedbackText, "Feedback message should be visible"); waitUntilElement(feedbackText, "Feedback message should be present").is().present();
return feedbackText.getText(); return feedbackText.getText();
} }

View file

@ -70,17 +70,17 @@ public class LoginForm extends Form {
public void register() { public void register() {
waitForUsernameInputPresent(); waitForUsernameInputPresent();
waitAjaxForElement(registerLink); waitUntilElement(registerLink).is().present();
registerLink.click(); registerLink.click();
} }
public void login() { public void login() {
waitAjaxForElement(loginButton); waitUntilElement(loginButton).is().present();
loginButton.click(); loginButton.click();
} }
public void forgotPassword() { public void forgotPassword() {
waitAjaxForElement(forgottenPassword); waitUntilElement(forgottenPassword).is().present();
forgottenPassword.click(); forgottenPassword.click();
} }
@ -94,32 +94,31 @@ public class LoginForm extends Form {
// @Override // @Override
// public void cancel() { // public void cancel() {
// waitAjaxForElement(cancelButton); // waitUntilElement(cancelButton).is().present();
// cancelButton.click(); // cancelButton.click();
// } // }
public void waitForUsernameInputPresent() { public void waitForUsernameInputPresent() {
accountFields.waitForUsernameInputPresent(); accountFields.waitForUsernameInputPresent();
} }
public void waitForRegisterLinkNotPresent() { public void waitForRegisterLinkNotPresent() {
waitAjaxForElementNotPresent(registerLink); waitUntilElement(registerLink).is().not().present();
} }
public void waitForResetPasswordLinkNotPresent() { public void waitForResetPasswordLinkNotPresent() {
waitAjaxForElementNotPresent(forgottenPassword); waitUntilElement(forgottenPassword).is().not().present();
} }
public void waitForRememberMePresent() { public void waitForRememberMePresent() {
waitAjaxForElement(rememberMe); waitUntilElement(rememberMe).is().present();
} }
public void waitForRememberMeNotPresent() { public void waitForRememberMeNotPresent() {
waitAjaxForElementNotPresent(rememberMe); waitUntilElement(rememberMe).is().not().present();
} }
public void waitForLoginButtonPresent() { public void waitForLoginButtonPresent() {
waitGuiForElement(loginButton); waitUntilElement(loginButton).is().present();
} }
} }

View file

@ -21,7 +21,7 @@ import javax.ws.rs.core.UriBuilder;
import org.jboss.arquillian.graphene.page.Page; import org.jboss.arquillian.graphene.page.Page;
import org.keycloak.testsuite.auth.page.account.AccountFields; import org.keycloak.testsuite.auth.page.account.AccountFields;
import org.keycloak.testsuite.auth.page.account.PasswordFields; import org.keycloak.testsuite.auth.page.account.PasswordFields;
import static org.keycloak.testsuite.util.WaitUtils.waitGuiForElementPresent; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy; import org.openqa.selenium.support.FindBy;
@ -57,7 +57,7 @@ public class ResetCredentials extends LoginActions {
} }
public String getInfoMessage() { public String getInfoMessage() {
waitGuiForElementPresent(info, "Info message should be visible"); waitUntilElement(info, "Info message should be visible").is().present();
return info.getText(); return info.getText();
} }
} }

View file

@ -21,7 +21,7 @@
*/ */
package org.keycloak.testsuite.auth.page.login; package org.keycloak.testsuite.auth.page.login;
import static org.keycloak.testsuite.util.WaitUtils.waitGuiForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy; import org.openqa.selenium.support.FindBy;
@ -35,7 +35,7 @@ public class VerifyEmail extends Authenticate {
private WebElement instruction; private WebElement instruction;
public String getInstructionMessage() { public String getInstructionMessage() {
waitGuiForElement(instruction); waitUntilElement(instruction).is().present();
return instruction.getText(); return instruction.getText();
} }
} }

View file

@ -3,8 +3,7 @@ package org.keycloak.testsuite.console.page;
import org.keycloak.admin.client.resource.RealmResource; import org.keycloak.admin.client.resource.RealmResource;
import static org.keycloak.testsuite.auth.page.AuthRealm.TEST; import static org.keycloak.testsuite.auth.page.AuthRealm.TEST;
import org.keycloak.testsuite.util.WaitUtils; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import static org.keycloak.testsuite.util.WaitUtils.waitGuiForElement;
import org.openqa.selenium.By; import org.openqa.selenium.By;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy; import org.openqa.selenium.support.FindBy;
@ -39,7 +38,7 @@ public class AdminConsoleRealm extends AdminConsoleRealmsRoot {
private ConfigureMenu configureMenu; private ConfigureMenu configureMenu;
public ConfigureMenu configure() { public ConfigureMenu configure() {
waitGuiForElement(By.xpath("//div[./h2[text()='Configure']]")); waitUntilElement(By.xpath("//div[./h2[text()='Configure']]")).is().present();
return configureMenu; return configureMenu;
} }
@ -92,7 +91,7 @@ public class AdminConsoleRealm extends AdminConsoleRealmsRoot {
protected ManageMenu manageMenu; protected ManageMenu manageMenu;
public ManageMenu manage() { public ManageMenu manage() {
WaitUtils.waitGuiForElement(By.xpath("//div[./h2[text()='Manage']]")); waitUntilElement(By.xpath("//div[./h2[text()='Manage']]")).is().present();
return manageMenu; return manageMenu;
} }

View file

@ -24,12 +24,12 @@ public class Authentication extends AdminConsoleRealm {
private WebElement close; private WebElement close;
public String getSuccessMessage() { public String getSuccessMessage() {
waitAjaxForElement(success); waitUntilElement(success).is().present();
return success.getText(); return success.getText();
} }
public String getErrorMessage() { public String getErrorMessage() {
waitAjaxForElement(error); waitUntilElement(error).is().present();
return error.getText(); return error.getText();
} }
@ -37,7 +37,7 @@ public class Authentication extends AdminConsoleRealm {
if (close.isDisplayed()) { if (close.isDisplayed()) {
close.click(); close.click();
} }
waitAjaxForElementNotVisible(close); waitUntilElement(close).is().not().visible();
} }
public AuthenticationTabs tabs() { public AuthenticationTabs tabs() {

View file

@ -8,7 +8,7 @@ import org.openqa.selenium.support.ui.Select;
import java.util.List; import java.util.List;
import static org.keycloak.testsuite.util.WaitUtils.waitGuiForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
/** /**
* @author Petr Mensik * @author Petr Mensik
@ -32,7 +32,7 @@ public class PasswordPolicy extends Authentication {
private List<WebElement> allRows; private List<WebElement> allRows;
public void addPolicy(PasswordPolicy.Type policy, String value) { public void addPolicy(PasswordPolicy.Type policy, String value) {
waitGuiForElement(addPolicySelectElement); waitUntilElement(addPolicySelectElement).is().present();
addPolicySelect.selectByVisibleText(policy.getName()); addPolicySelect.selectByVisibleText(policy.getName());
setPolicyValue(policy, value); setPolicyValue(policy, value);
primaryButton.click(); primaryButton.click();

View file

@ -21,7 +21,7 @@
*/ */
package org.keycloak.testsuite.console.page.authentication.flows; package org.keycloak.testsuite.console.page.authentication.flows;
import static org.keycloak.testsuite.util.WaitUtils.waitAjaxForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.openqa.selenium.By; import org.openqa.selenium.By;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy; import org.openqa.selenium.support.FindBy;
@ -71,7 +71,7 @@ public class FlowsTable {
private WebElement getRowByLabelText(String text) { private WebElement getRowByLabelText(String text) {
WebElement row = tbody.findElement(By.xpath("//span[text() = '" + text + "']/../..")); WebElement row = tbody.findElement(By.xpath("//span[text() = '" + text + "']/../.."));
waitAjaxForElement(row); waitUntilElement(row).is().present();
return row; return row;
} }

View file

@ -38,17 +38,17 @@ public class ClientMappers extends Client {
} }
public void createMapper() { public void createMapper() {
waitAjaxForBody(); waitForBody();
clickHeaderLink(CREATE); clickHeaderLink(CREATE);
} }
public void addBuiltin() { public void addBuiltin() {
waitAjaxForBody(); waitForBody();
clickHeaderLink(ADD_BUILTIN); clickHeaderLink(ADD_BUILTIN);
} }
public void clickMapper(String mapperName) { public void clickMapper(String mapperName) {
waitAjaxForBody(); waitForBody();
body().findElement(By.linkText(mapperName)).click(); body().findElement(By.linkText(mapperName)).click();
} }
@ -57,7 +57,7 @@ public class ClientMappers extends Client {
} }
private void clickMapperActionButton(String mapperName, String buttonText) { private void clickMapperActionButton(String mapperName, String buttonText) {
waitAjaxForBody(); waitForBody();
clickRowActionButton(getRowByLinkText(mapperName), buttonText); clickRowActionButton(getRowByLinkText(mapperName), buttonText);
} }

View file

@ -62,32 +62,32 @@ public class Clients extends AdminConsoleRealm {
} }
public void createClient() { public void createClient() {
waitAjaxForBody(); waitForBody();
clickHeaderLink(CREATE); clickHeaderLink(CREATE);
} }
public void importClient() { public void importClient() {
waitAjaxForBody(); waitForBody();
clickHeaderLink(IMPORT); clickHeaderLink(IMPORT);
} }
public void clickClient(ClientRepresentation client) { public void clickClient(ClientRepresentation client) {
waitAjaxForBody(); waitForBody();
clickClient(client.getClientId()); clickClient(client.getClientId());
} }
public void clickClient(String clientId) { public void clickClient(String clientId) {
waitAjaxForBody(); waitForBody();
body().findElement(linkText(clientId)).click(); body().findElement(linkText(clientId)).click();
} }
public void editClient(String clientId) { public void editClient(String clientId) {
waitAjaxForBody(); waitForBody();
clickRowActionButton(getRowByLinkText(clientId), EDIT); clickRowActionButton(getRowByLinkText(clientId), EDIT);
} }
public void deleteClient(String clientId) { public void deleteClient(String clientId) {
waitAjaxForBody(); waitForBody();
clickRowActionButton(getRowByLinkText(clientId), DELETE); clickRowActionButton(getRowByLinkText(clientId), DELETE);
} }

View file

@ -8,7 +8,7 @@ import org.keycloak.testsuite.console.page.fragment.OnOffSwitch;
import org.keycloak.testsuite.page.Form; import org.keycloak.testsuite.page.Form;
import static org.keycloak.testsuite.page.Form.getInputValue; import static org.keycloak.testsuite.page.Form.getInputValue;
import static org.keycloak.testsuite.util.WaitUtils.pause; import static org.keycloak.testsuite.util.WaitUtils.pause;
import static org.keycloak.testsuite.util.WaitUtils.waitAjaxForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.keycloak.testsuite.util.Timer; import org.keycloak.testsuite.util.Timer;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy; import org.openqa.selenium.support.FindBy;
@ -69,7 +69,7 @@ public class CreateClientForm extends Form {
private List<WebElement> deleteRedirectUriIcons; private List<WebElement> deleteRedirectUriIcons;
public void setValues(ClientRepresentation client) { public void setValues(ClientRepresentation client) {
waitAjaxForElement(clientIdInput); waitUntilElement(clientIdInput).is().present();
setClientId(client.getClientId()); setClientId(client.getClientId());
setName(client.getName()); setName(client.getName());
@ -230,7 +230,7 @@ public class CreateClientForm extends Form {
} }
public String getProtocol() { public String getProtocol() {
waitAjaxForElement(protocolSelect.getFirstSelectedOption()); waitUntilElement(protocolSelect.getFirstSelectedOption()).is().present();
return protocolSelect.getFirstSelectedOption().getText(); return protocolSelect.getFirstSelectedOption().getText();
} }

View file

@ -33,17 +33,17 @@ public class AdminEvents extends Events {
private AdminEventsTableFilterForm filterForm; private AdminEventsTableFilterForm filterForm;
public void update() { public void update() {
waitAjaxForBody(); waitForBody();
clickHeaderButton("Update"); clickHeaderButton("Update");
} }
public void reset() { public void reset() {
waitAjaxForBody(); waitForBody();
clickHeaderButton("Reset"); clickHeaderButton("Reset");
} }
public void filter() { public void filter() {
waitAjaxForBody(); waitForBody();
filterButton.click(); filterButton.click();
} }

View file

@ -104,7 +104,7 @@ public class Config extends Events {
} }
public void waitForClearEventsButtonPresent() { public void waitForClearEventsButtonPresent() {
waitAjaxForElement(clearLoginEventsButton); waitUntilElement(clearLoginEventsButton).is().present();
} }
} }
} }

View file

@ -33,17 +33,17 @@ public class LoginEvents extends Events {
private LoginEventsTableFilterForm filterForm; private LoginEventsTableFilterForm filterForm;
public void update() { public void update() {
waitAjaxForBody(); waitForBody();
clickHeaderButton("Update"); clickHeaderButton("Update");
} }
public void reset() { public void reset() {
waitAjaxForBody(); waitForBody();
clickHeaderButton("Reset"); clickHeaderButton("Reset");
} }
public void filter() { public void filter() {
waitAjaxForBody(); waitForBody();
filterButton.click(); filterButton.click();
} }

View file

@ -1,6 +1,6 @@
package org.keycloak.testsuite.console.page.federation; package org.keycloak.testsuite.console.page.federation;
import static org.keycloak.testsuite.util.WaitUtils.waitGuiForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.keycloak.testsuite.console.page.fragment.OnOffSwitch; import org.keycloak.testsuite.console.page.fragment.OnOffSwitch;
import org.keycloak.testsuite.page.Form; import org.keycloak.testsuite.page.Form;
@ -50,7 +50,7 @@ public class KerberosUserProviderForm extends Form {
} }
public void setKerberosRealmInput(String kerberosRealm) { public void setKerberosRealmInput(String kerberosRealm) {
waitGuiForElement(By.id("kerberosRealm")); waitUntilElement(By.id("kerberosRealm")).is().present();
setInputValue(kerberosRealmInput, kerberosRealm); setInputValue(kerberosRealmInput, kerberosRealm);
} }
@ -71,7 +71,7 @@ public class KerberosUserProviderForm extends Form {
} }
public void selectEditMode(String mode) { public void selectEditMode(String mode) {
waitGuiForElement(By.id("editMode")); waitUntilElement(By.id("editMode")).is().present();
editModeSelect.selectByVisibleText(mode); editModeSelect.selectByVisibleText(mode);
} }

View file

@ -1,7 +1,7 @@
package org.keycloak.testsuite.console.page.federation; package org.keycloak.testsuite.console.page.federation;
import static org.keycloak.testsuite.util.WaitUtils.waitAjaxForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import static org.keycloak.testsuite.util.WaitUtils.waitGuiForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.jboss.arquillian.graphene.findby.FindByJQuery; import org.jboss.arquillian.graphene.findby.FindByJQuery;
import org.keycloak.testsuite.console.page.fragment.OnOffSwitch; import org.keycloak.testsuite.console.page.fragment.OnOffSwitch;
@ -156,17 +156,14 @@ public class LdapUserProviderForm extends Form {
} }
public void setKerberosRealmInput(String kerberosRealm) { public void setKerberosRealmInput(String kerberosRealm) {
waitAjaxForElement(kerberosRealmInput);
setInputValue(kerberosRealmInput, kerberosRealm); setInputValue(kerberosRealmInput, kerberosRealm);
} }
public void setServerPrincipalInput(String serverPrincipal) { public void setServerPrincipalInput(String serverPrincipal) {
waitAjaxForElement(serverPrincipalInput);
setInputValue(serverPrincipalInput, serverPrincipal); setInputValue(serverPrincipalInput, serverPrincipal);
} }
public void setKeyTabInput(String keyTab) { public void setKeyTabInput(String keyTab) {
waitAjaxForElement(keyTabInput);
setInputValue(keyTabInput, keyTab); setInputValue(keyTabInput, keyTab);
} }
@ -175,22 +172,22 @@ public class LdapUserProviderForm extends Form {
} }
public void selectEditMode(String mode) { public void selectEditMode(String mode) {
waitGuiForElement(By.id("editMode")); waitUntilElement(By.id("editMode")).is().present();
editModeSelect.selectByVisibleText(mode); editModeSelect.selectByVisibleText(mode);
} }
public void selectVendor(String vendor) { public void selectVendor(String vendor) {
waitGuiForElement(By.id("vendor")); waitUntilElement(By.id("vendor")).is().present();
vendorSelect.selectByVisibleText(vendor); vendorSelect.selectByVisibleText(vendor);
} }
public void selectAuthenticationType(String authenticationType) { public void selectAuthenticationType(String authenticationType) {
waitGuiForElement(By.id("authType")); waitUntilElement(By.id("authType")).is().present();
authTypeSelect.selectByVisibleText(authenticationType); authTypeSelect.selectByVisibleText(authenticationType);
} }
public void selectSearchScope(String searchScope) { public void selectSearchScope(String searchScope) {
waitGuiForElement(By.id("searchScope")); waitUntilElement(By.id("searchScope")).is().present();
searchScopeSelect.selectByVisibleText(searchScope); searchScopeSelect.selectByVisibleText(searchScope);
} }
@ -248,7 +245,7 @@ public class LdapUserProviderForm extends Form {
} }
public void synchronizeAllUsers() { public void synchronizeAllUsers() {
waitAjaxForElement(synchronizeAllUsersButton); waitUntilElement(synchronizeAllUsersButton).is().present();
synchronizeAllUsersButton.click(); synchronizeAllUsersButton.click();
} }
} }

View file

@ -5,7 +5,7 @@ import org.keycloak.testsuite.console.page.AdminConsoleRealm;
import org.openqa.selenium.By; import org.openqa.selenium.By;
import org.openqa.selenium.support.ui.Select; import org.openqa.selenium.support.ui.Select;
import static org.keycloak.testsuite.util.WaitUtils.waitGuiForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
/** /**
* Created by fkiss. * Created by fkiss.
@ -21,7 +21,7 @@ public class UserFederation extends AdminConsoleRealm {
private Select addProviderSelect; private Select addProviderSelect;
public void addProvider(String provider) { public void addProvider(String provider) {
waitGuiForElement(By.cssSelector("select[ng-model*='selectedProvider']")); waitUntilElement(By.cssSelector("select[ng-model*='selectedProvider']")).is().present();
addProviderSelect.selectByVisibleText(provider); addProviderSelect.selectByVisibleText(provider);
} }

View file

@ -0,0 +1,50 @@
/*
* JBoss, Home of Professional Open Source
*
* Copyright 2013 Red Hat, Inc. and/or its affiliates.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package org.keycloak.testsuite.console.page.fragment;
import org.keycloak.testsuite.page.AbstractAlert;
import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy;
/**
*
* @author Petr Mensik
* @author tkyjovsk
*/
public class AdminConsoleAlert extends AbstractAlert {
@FindBy(xpath = "//button[@class='close']")
protected WebElement closeButton;
public boolean isInfo() {
return getAttributeClass().contains("alert-info");
}
public boolean isWarning() {
return getAttributeClass().contains("alert-warning");
}
public boolean isDanger() {
return getAttributeClass().contains("alert-danger");
}
public void close() {
closeButton.click();
}
}

View file

@ -2,7 +2,7 @@ package org.keycloak.testsuite.console.page.fragment;
import java.util.List; import java.util.List;
import static org.keycloak.testsuite.util.WaitUtils.pause; import static org.keycloak.testsuite.util.WaitUtils.pause;
import static org.keycloak.testsuite.util.WaitUtils.waitAjaxForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.openqa.selenium.By; import org.openqa.selenium.By;
import static org.openqa.selenium.By.xpath; import static org.openqa.selenium.By.xpath;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
@ -30,18 +30,18 @@ public class DataTable {
private WebElement infoRow; private WebElement infoRow;
public void search(String pattern) { public void search(String pattern) {
waitAjaxForBody(); waitForBody();
searchInput.sendKeys(pattern); searchInput.sendKeys(pattern);
searchButton.click(); searchButton.click();
} }
public void clickHeaderButton(String buttonText) { public void clickHeaderButton(String buttonText) {
waitAjaxForBody(); waitForBody();
header.findElement(By.xpath(".//button[text()='" + buttonText + "']")).click(); header.findElement(By.xpath(".//button[text()='" + buttonText + "']")).click();
} }
public void clickHeaderLink(String linkText) { public void clickHeaderLink(String linkText) {
waitAjaxForBody(); waitForBody();
header.findElement(By.linkText(linkText)).click(); header.findElement(By.linkText(linkText)).click();
} }
@ -49,19 +49,19 @@ public class DataTable {
return body; return body;
} }
public void waitAjaxForBody() { public void waitForBody() {
waitAjaxForElement(body); waitUntilElement(body).is().present();
} }
public List<WebElement> rows() { public List<WebElement> rows() {
waitAjaxForBody(); waitForBody();
pause(250); pause(250);
return rows; return rows;
} }
public WebElement getRowByLinkText(String text) { public WebElement getRowByLinkText(String text) {
WebElement row = body.findElement(By.xpath(".//tr[./td/a[text()='" + text + "']]")); WebElement row = body.findElement(By.xpath(".//tr[./td/a[text()='" + text + "']]"));
waitAjaxForElement(row); waitUntilElement(row).is().present();
return row; return row;
} }

View file

@ -1,66 +0,0 @@
/*
* JBoss, Home of Professional Open Source
*
* Copyright 2013 Red Hat, Inc. and/or its affiliates.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package org.keycloak.testsuite.console.page.fragment;
import static org.jboss.arquillian.graphene.Graphene.waitGui;
import org.jboss.arquillian.graphene.fragment.Root;
import static org.keycloak.testsuite.util.WaitUtils.waitGuiForElementPresent;
import org.openqa.selenium.WebElement;
/**
*
* @author Petr Mensik
*/
public class FlashMessage {
@Root
private WebElement root;
public boolean isSuccess() {
waitGui().until("Flash message should be success")
.element(root)
.attribute("class")
.contains("success");
return root.getAttribute("class").contains("success");
}
public boolean isError() {
waitGui().until("Flash message should be error")
.element(root)
.attribute("class")
.contains("error");
return root.getAttribute("class").contains("error");
}
public boolean isDanger() {
waitGui().until("Flash message should be danger")
.element(root)
.attribute("class")
.contains("danger");
return root.getAttribute("class").contains("danger");
}
public String getText() {
return root.getText();
}
public void waitUntilPresent() {
waitGuiForElementPresent(root, "Flash message should be visible.");
}
}

View file

@ -18,7 +18,7 @@
package org.keycloak.testsuite.console.page.fragment; package org.keycloak.testsuite.console.page.fragment;
import java.util.List; import java.util.List;
import static org.keycloak.testsuite.util.WaitUtils.waitGuiForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.openqa.selenium.By; import org.openqa.selenium.By;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
@ -53,7 +53,7 @@ public class Menu {
} }
public String getCurrentRealm() { public String getCurrentRealm() {
waitGuiForElement(By.cssSelector(MENU_LOCATOR)); waitUntilElement(By.cssSelector(MENU_LOCATOR)).is().present();
return toggle.get(1).getText(); return toggle.get(1).getText();
} }
@ -67,7 +67,7 @@ public class Menu {
menuOrder = 0; menuOrder = 0;
break; break;
} }
waitGuiForElement(By.cssSelector(MENU_LOCATOR)); waitUntilElement(By.cssSelector(MENU_LOCATOR)).is().present();
if (!menuList.get(menuOrder).isDisplayed()) { if (!menuList.get(menuOrder).isDisplayed()) {
toggle.get(menuOrder).click(); toggle.get(menuOrder).click();
} }

View file

@ -1,6 +1,6 @@
package org.keycloak.testsuite.console.page.fragment; package org.keycloak.testsuite.console.page.fragment;
import static org.keycloak.testsuite.util.WaitUtils.waitAjaxForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy; import org.openqa.selenium.support.FindBy;
@ -21,22 +21,22 @@ public class ModalDialog {
private WebElement nameInput; private WebElement nameInput;
public void ok() { public void ok() {
waitAjaxForElement(okButton); waitUntilElement(okButton).is().present();
okButton.click(); okButton.click();
} }
public void confirmDeletion() { public void confirmDeletion() {
waitAjaxForElement(deleteButton); waitUntilElement(deleteButton).is().present();
deleteButton.click(); deleteButton.click();
} }
public void cancel() { public void cancel() {
waitAjaxForElement(cancelButton); waitUntilElement(cancelButton).is().present();
cancelButton.click(); cancelButton.click();
} }
public void setName(String name) { public void setName(String name) {
waitAjaxForElement(nameInput); waitUntilElement(nameInput).is().present();
nameInput.clear(); nameInput.clear();
nameInput.sendKeys(name); nameInput.sendKeys(name);
} }

View file

@ -19,7 +19,7 @@ package org.keycloak.testsuite.console.page.fragment;
import org.jboss.arquillian.graphene.fragment.Root; import org.jboss.arquillian.graphene.fragment.Root;
import org.jboss.arquillian.test.api.ArquillianResource; import org.jboss.arquillian.test.api.ArquillianResource;
import static org.keycloak.testsuite.util.WaitUtils.waitAjaxForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.openqa.selenium.By; import org.openqa.selenium.By;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
import org.openqa.selenium.interactions.Actions; import org.openqa.selenium.interactions.Actions;
@ -45,12 +45,12 @@ public class OnOffSwitch {
} }
public boolean isOn() { public boolean isOn() {
waitAjaxForElement(root); waitUntilElement(root).is().present();
return root.findElement(By.tagName("input")).isSelected(); return root.findElement(By.tagName("input")).isSelected();
} }
private void click() { private void click() {
waitAjaxForElement(root); waitUntilElement(root).is().present();
actions.moveToElement(root.findElements(By.tagName("span")).get(0)) actions.moveToElement(root.findElements(By.tagName("span")).get(0))
.click().build().perform(); .click().build().perform();
} }

View file

@ -20,7 +20,7 @@ package org.keycloak.testsuite.console.page.realm;
import org.openqa.selenium.support.FindBy; import org.openqa.selenium.support.FindBy;
import org.openqa.selenium.support.ui.Select; import org.openqa.selenium.support.ui.Select;
import static org.keycloak.testsuite.util.WaitUtils.waitGuiForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.openqa.selenium.By; import org.openqa.selenium.By;
/** /**
@ -47,7 +47,7 @@ public class ThemeSettings extends RealmSettings {
private Select emailThemeSelect; private Select emailThemeSelect;
public void changeLoginTheme(String themeName) { public void changeLoginTheme(String themeName) {
waitGuiForElement(By.id("loginTheme")); waitUntilElement(By.id("loginTheme")).is().present();
loginThemeSelect.selectByVisibleText(themeName); loginThemeSelect.selectByVisibleText(themeName);
} }

View file

@ -23,7 +23,7 @@ import org.openqa.selenium.support.FindBy;
import org.openqa.selenium.support.ui.Select; import org.openqa.selenium.support.ui.Select;
import static java.lang.String.valueOf; import static java.lang.String.valueOf;
import static org.keycloak.testsuite.util.WaitUtils.waitGuiForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import static org.apache.commons.lang3.text.WordUtils.capitalize; import static org.apache.commons.lang3.text.WordUtils.capitalize;
import org.jboss.arquillian.graphene.page.Page; import org.jboss.arquillian.graphene.page.Page;
import org.keycloak.testsuite.page.Form; import org.keycloak.testsuite.page.Form;
@ -70,7 +70,7 @@ public class TokenSettings extends RealmSettings {
private void setTimeout(Select timeoutElement, WebElement unitElement, private void setTimeout(Select timeoutElement, WebElement unitElement,
int timeout, TimeUnit unit) { int timeout, TimeUnit unit) {
waitGuiForElement(sessionTimeout); waitUntilElement(sessionTimeout).is().present();
timeoutElement.selectByValue(capitalize(unit.name().toLowerCase())); timeoutElement.selectByValue(capitalize(unit.name().toLowerCase()));
unitElement.clear(); unitElement.clear();
unitElement.sendKeys(valueOf(timeout)); unitElement.sendKeys(valueOf(timeout));

View file

@ -9,7 +9,7 @@ import java.util.Map;
import java.util.Set; import java.util.Set;
import org.keycloak.representations.idm.RoleRepresentation.Composites; import org.keycloak.representations.idm.RoleRepresentation.Composites;
import org.keycloak.testsuite.page.Form; import org.keycloak.testsuite.page.Form;
import static org.keycloak.testsuite.util.WaitUtils.waitGuiForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.openqa.selenium.By; import org.openqa.selenium.By;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy; import org.openqa.selenium.support.FindBy;
@ -127,7 +127,7 @@ public class RoleCompositeRoles extends Form {
// *** original methods *** // *** original methods ***
public void addAvailableRole(String... roles) { public void addAvailableRole(String... roles) {
waitGuiForElement(By.id("available")); waitUntilElement(By.id("available")).is().present();
for (String role : roles) { for (String role : roles) {
availableRealmRolesSelect.selectByVisibleText(role); availableRealmRolesSelect.selectByVisibleText(role);
addSelectedRealmRolesButton.click(); addSelectedRealmRolesButton.click();
@ -135,13 +135,13 @@ public class RoleCompositeRoles extends Form {
} }
public void removeAssignedRole(String role) { public void removeAssignedRole(String role) {
waitGuiForElement(By.id("assigned")); waitUntilElement(By.id("assigned")).is().present();
assignedRealmRolesSelect.selectByVisibleText(role); assignedRealmRolesSelect.selectByVisibleText(role);
removeSelectedRealmRolesButton.click(); removeSelectedRealmRolesButton.click();
} }
public boolean isAssignedRole(String role) { public boolean isAssignedRole(String role) {
waitGuiForElement(By.id("assigned")); waitUntilElement(By.id("assigned")).is().present();
try { try {
assignedRealmRolesSelect.selectByVisibleText(role); assignedRealmRolesSelect.selectByVisibleText(role);
} catch (Exception ex) { } catch (Exception ex) {
@ -151,7 +151,7 @@ public class RoleCompositeRoles extends Form {
} }
public boolean isAssignedClientRole(String role) { public boolean isAssignedClientRole(String role) {
waitGuiForElement(By.id("assigned")); waitUntilElement(By.id("assigned")).is().present();
try { try {
assignedClientRolesSelect.selectByVisibleText(role); assignedClientRolesSelect.selectByVisibleText(role);
} catch (Exception ex) { } catch (Exception ex) {
@ -161,12 +161,12 @@ public class RoleCompositeRoles extends Form {
} }
public void selectClientRole(String client) { public void selectClientRole(String client) {
waitGuiForElement(By.id("clients")); waitUntilElement(By.id("clients")).is().present();
clientSelect.selectByVisibleText(client); clientSelect.selectByVisibleText(client);
} }
public void addAvailableClientRole(String... roles) { public void addAvailableClientRole(String... roles) {
waitGuiForElement(By.id("available-client")); waitUntilElement(By.id("available-client")).is().present();
for (String role : roles) { for (String role : roles) {
availableClientRolesSelect.selectByVisibleText(role); availableClientRolesSelect.selectByVisibleText(role);
addSelectedClientRolesButton.click(); addSelectedClientRolesButton.click();
@ -174,7 +174,7 @@ public class RoleCompositeRoles extends Form {
} }
public void removeAssignedClientRole(String client) { public void removeAssignedClientRole(String client) {
waitGuiForElement(By.id("assigned-client")); waitUntilElement(By.id("assigned-client")).is().present();
assignedClientRolesSelect.selectByVisibleText(client); assignedClientRolesSelect.selectByVisibleText(client);
removeSelectedClientRolesButton.click(); removeSelectedClientRolesButton.click();
} }

View file

@ -28,7 +28,7 @@ public class RolesTable extends DataTable {
} }
public void clickRole(String name) { public void clickRole(String name) {
waitAjaxForBody(); waitForBody();
clickRowByLinkText(name); clickRowByLinkText(name);
} }

View file

@ -4,7 +4,7 @@ import java.util.List;
import org.keycloak.representations.idm.UserRepresentation; import org.keycloak.representations.idm.UserRepresentation;
import org.keycloak.testsuite.console.page.fragment.OnOffSwitch; import org.keycloak.testsuite.console.page.fragment.OnOffSwitch;
import org.keycloak.testsuite.page.Form; import org.keycloak.testsuite.page.Form;
import static org.keycloak.testsuite.util.WaitUtils.waitAjaxForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy; import org.openqa.selenium.support.FindBy;
import org.openqa.selenium.support.ui.Select; import org.openqa.selenium.support.ui.Select;
@ -125,7 +125,7 @@ public class UserAttributesForm extends Form {
} }
public void setValues(UserRepresentation user) { public void setValues(UserRepresentation user) {
waitAjaxForElement(usernameInput); waitUntilElement(usernameInput).is().present();
setUsername(user.getUsername()); setUsername(user.getUsername());
setEmail(user.getEmail()); setEmail(user.getEmail());
setFirstName(user.getFirstName()); setFirstName(user.getFirstName());

View file

@ -27,7 +27,7 @@ import org.keycloak.representations.idm.UserRepresentation;
import org.keycloak.testsuite.console.page.AdminConsoleRealm; import org.keycloak.testsuite.console.page.AdminConsoleRealm;
import org.keycloak.testsuite.console.page.fragment.DataTable; import org.keycloak.testsuite.console.page.fragment.DataTable;
import static org.keycloak.testsuite.util.WaitUtils.waitAjaxForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import static org.openqa.selenium.By.*; import static org.openqa.selenium.By.*;
/** /**
@ -73,7 +73,7 @@ public class Users extends AdminConsoleRealm {
} }
public void clickUser(String username) { public void clickUser(String username) {
waitAjaxForElement(body()); waitUntilElement(body()).is().present();
body().findElement(linkText(username)).click(); body().findElement(linkText(username)).click();
} }

View file

@ -0,0 +1,51 @@
package org.keycloak.testsuite.page;
import com.google.common.base.Predicate;
import static org.jboss.arquillian.graphene.Graphene.waitModel;
import org.jboss.arquillian.graphene.fragment.Root;
import org.jboss.logging.Logger;
import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.openqa.selenium.WebDriver;
import org.openqa.selenium.WebElement;
/**
*
* @author tkyjovsk
*/
public abstract class AbstractAlert {
protected final Logger log = Logger.getLogger(this.getClass());
@Root
protected WebElement root;
public void waitUntilPresent() {
waitUntilElement(root, "Flash message should be present.").is().present();
}
public void waitUntilPresentAndClassSet() {
waitUntilPresent();
waitModel().until(new Predicate<WebDriver>() {
@Override
public boolean apply(WebDriver input) {
return !getAttributeClass().endsWith("alert-");
}
});
}
public String getText() {
return root.getText();
}
public String getAttributeClass() {
String attrClass = root.getAttribute("class");
log.debug("Alert @class = '" + attrClass + "'");
return attrClass;
}
public boolean isSuccess() {
log.debug("Alert.isSuccess()");
return getAttributeClass().contains("alert-success");
}
}

View file

@ -3,7 +3,7 @@ package org.keycloak.testsuite.page;
import org.jboss.arquillian.drone.api.annotation.Drone; import org.jboss.arquillian.drone.api.annotation.Drone;
import static org.jboss.arquillian.graphene.Graphene.guardAjax; import static org.jboss.arquillian.graphene.Graphene.guardAjax;
import org.jboss.logging.Logger; import org.jboss.logging.Logger;
import static org.keycloak.testsuite.util.WaitUtils.waitAjaxForElement; import static org.keycloak.testsuite.util.WaitUtils.waitUntilElement;
import org.openqa.selenium.WebDriver; import org.openqa.selenium.WebDriver;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
import org.openqa.selenium.support.FindBy; import org.openqa.selenium.support.FindBy;
@ -36,14 +36,14 @@ public class Form {
} }
public static String getInputValue(WebElement input) { public static String getInputValue(WebElement input) {
waitAjaxForElement(input); waitUntilElement(input).is().present();
return input.getAttribute(VALUE); return input.getAttribute(VALUE);
} }
public static final String VALUE = "value"; public static final String VALUE = "value";
public static void setInputValue(WebElement input, String value) { public static void setInputValue(WebElement input, String value) {
waitAjaxForElement(input); waitUntilElement(input).is().present();
if (input.isEnabled()) { if (input.isEnabled()) {
input.clear(); input.clear();
if (value != null) { if (value != null) {

View file

@ -17,11 +17,10 @@
*/ */
package org.keycloak.testsuite.util; package org.keycloak.testsuite.util;
import java.util.concurrent.TimeUnit;
import java.util.logging.Level; import java.util.logging.Level;
import java.util.logging.Logger; import java.util.logging.Logger;
import static org.jboss.arquillian.graphene.Graphene.waitAjax;
import static org.jboss.arquillian.graphene.Graphene.waitGui; import static org.jboss.arquillian.graphene.Graphene.waitGui;
import org.jboss.arquillian.graphene.wait.ElementBuilder;
import org.openqa.selenium.By; import org.openqa.selenium.By;
import org.openqa.selenium.WebElement; import org.openqa.selenium.WebElement;
@ -33,52 +32,19 @@ import org.openqa.selenium.WebElement;
public final class WaitUtils { public final class WaitUtils {
public static final String PAGELOAD_TIMEOUT_PROP = "pageload.timeout"; public static final String PAGELOAD_TIMEOUT_PROP = "pageload.timeout";
public static final String IMPLICIT_TIMEOUT_PROP = "implicit.timeout";
public static final String SCRIPT_TIMEOUT_PROP = "script.timeout";
public static final String POLLING_INTERVAL_PROP = "polling.interval";
public static final Integer PAGELOAD_TIMEOUT = Integer.parseInt(System.getProperty(PAGELOAD_TIMEOUT_PROP, "5000")); public static final Integer PAGELOAD_TIMEOUT = Integer.parseInt(System.getProperty(PAGELOAD_TIMEOUT_PROP, "60000"));
public static final Integer IMPLICIT_TIMEOUT = Integer.parseInt(System.getProperty(IMPLICIT_TIMEOUT_PROP, "3000"));
public static final Integer SCRIPT_TIMEOUT = Integer.parseInt(System.getProperty(SCRIPT_TIMEOUT_PROP, "3000"));
public static final Integer POLLING_INTERVAL = Integer.parseInt(System.getProperty(POLLING_INTERVAL_PROP, "1000")); public static ElementBuilder<Void> waitUntilElement(By by) {
return waitGui().until().element(by);
public static void waitAjaxForElement(WebElement element) {
waitAjax().withTimeout(SCRIPT_TIMEOUT, TimeUnit.MILLISECONDS).pollingEvery(POLLING_INTERVAL, TimeUnit.MILLISECONDS)
.until().element(element).is().present();
} }
public static void waitAjaxForElementNotPresent(WebElement element) { public static ElementBuilder<Void> waitUntilElement(WebElement element) {
waitAjax().withTimeout(SCRIPT_TIMEOUT, TimeUnit.MILLISECONDS).pollingEvery(POLLING_INTERVAL, TimeUnit.MILLISECONDS) return waitGui().until().element(element);
.until().element(element).is().not().present();
} }
public static void waitAjaxForElementNotVisible(WebElement element) { public static ElementBuilder<Void> waitUntilElement(WebElement element, String failMessage) {
waitAjax().withTimeout(SCRIPT_TIMEOUT, TimeUnit.MILLISECONDS).pollingEvery(POLLING_INTERVAL, TimeUnit.MILLISECONDS) return waitGui().until(failMessage).element(element);
.until().element(element).is().not().visible();
}
public static void waitGuiForElement(By element, String message) {
waitGui().withTimeout(IMPLICIT_TIMEOUT, TimeUnit.MILLISECONDS).pollingEvery(POLLING_INTERVAL, TimeUnit.MILLISECONDS)
.until(message).element(element).is().present();
}
public static void waitGuiForElement(By element) {
waitGuiForElement(element, null);
}
public static void waitGuiForElement(WebElement element) {
waitGuiForElementPresent(element, null);
}
public static void waitGuiForElementPresent(WebElement element, String message) {
waitGui().withTimeout(IMPLICIT_TIMEOUT, TimeUnit.MILLISECONDS).pollingEvery(POLLING_INTERVAL, TimeUnit.MILLISECONDS)
.until(message).element(element).is().present();
}
public static void waitGuiForElementNotPresent(WebElement element) {
waitGui().withTimeout(IMPLICIT_TIMEOUT, TimeUnit.MILLISECONDS).pollingEvery(POLLING_INTERVAL, TimeUnit.MILLISECONDS)
.until().element(element).is().not().present();
} }
public static void pause(long millis) { public static void pause(long millis) {

View file

@ -19,9 +19,7 @@ package org.keycloak.testsuite;
import java.text.MessageFormat; import java.text.MessageFormat;
import java.util.List; import java.util.List;
import org.jboss.arquillian.graphene.findby.FindByJQuery;
import org.jboss.arquillian.graphene.page.Page; import org.jboss.arquillian.graphene.page.Page;
import static org.junit.Assert.assertTrue;
import org.junit.Before; import org.junit.Before;
import org.keycloak.admin.client.resource.RealmResource; import org.keycloak.admin.client.resource.RealmResource;
import static org.keycloak.representations.idm.CredentialRepresentation.PASSWORD; import static org.keycloak.representations.idm.CredentialRepresentation.PASSWORD;
@ -32,7 +30,6 @@ import static org.keycloak.testsuite.admin.Users.setPasswordFor;
import org.keycloak.testsuite.auth.page.AuthRealm; import org.keycloak.testsuite.auth.page.AuthRealm;
import static org.keycloak.testsuite.auth.page.AuthRealm.TEST; import static org.keycloak.testsuite.auth.page.AuthRealm.TEST;
import org.keycloak.testsuite.auth.page.login.OIDCLogin; import org.keycloak.testsuite.auth.page.login.OIDCLogin;
import org.keycloak.testsuite.console.page.fragment.FlashMessage;
import org.openqa.selenium.Cookie; import org.openqa.selenium.Cookie;
/** /**
@ -48,9 +45,6 @@ public abstract class AbstractAuthTest extends AbstractKeycloakTest {
protected UserRepresentation testUser; protected UserRepresentation testUser;
@FindByJQuery(".alert")
protected FlashMessage flashMessage;
@Override @Override
public void addTestRealms(List<RealmRepresentation> testRealms) { public void addTestRealms(List<RealmRepresentation> testRealms) {
RealmRepresentation testRealmRep = new RealmRepresentation(); RealmRepresentation testRealmRep = new RealmRepresentation();
@ -101,21 +95,6 @@ public abstract class AbstractAuthTest extends AbstractKeycloakTest {
} }
} }
public void assertFlashMessageSuccess() {
flashMessage.waitUntilPresent();
assertTrue(flashMessage.getText(), flashMessage.isSuccess());
}
public void assertFlashMessageDanger() {
flashMessage.waitUntilPresent();
assertTrue(flashMessage.getText(), flashMessage.isDanger());
}
public void assertFlashMessageError() {
flashMessage.waitUntilPresent();
assertTrue(flashMessage.getText(), flashMessage.isError());
}
public RealmResource testRealmResource() { public RealmResource testRealmResource() {
return adminClient.realm(testRealmPage.getAuthRealm()); return adminClient.realm(testRealmPage.getAuthRealm());
} }

View file

@ -118,8 +118,6 @@ public abstract class AbstractKeycloakTest {
protected void driverSettings() { protected void driverSettings() {
driver.manage().timeouts().pageLoadTimeout(WaitUtils.PAGELOAD_TIMEOUT, TimeUnit.MILLISECONDS); driver.manage().timeouts().pageLoadTimeout(WaitUtils.PAGELOAD_TIMEOUT, TimeUnit.MILLISECONDS);
driver.manage().timeouts().implicitlyWait(WaitUtils.IMPLICIT_TIMEOUT, TimeUnit.MILLISECONDS);
driver.manage().timeouts().setScriptTimeout(WaitUtils.SCRIPT_TIMEOUT, TimeUnit.MILLISECONDS);
driver.manage().window().maximize(); driver.manage().window().maximize();
} }

View file

@ -1,10 +1,13 @@
package org.keycloak.testsuite.account; package org.keycloak.testsuite.account;
import org.jboss.arquillian.graphene.page.Page; import org.jboss.arquillian.graphene.page.Page;
import static org.junit.Assert.assertTrue;
import org.junit.Before; import org.junit.Before;
import org.keycloak.testsuite.AbstractAuthTest; import org.keycloak.testsuite.AbstractAuthTest;
import static org.keycloak.testsuite.auth.page.AuthRealm.TEST; import static org.keycloak.testsuite.auth.page.AuthRealm.TEST;
import org.keycloak.testsuite.auth.page.account.AccountManagement; import org.keycloak.testsuite.auth.page.account.AccountManagement;
import org.keycloak.testsuite.auth.page.account.fragment.AccountManagementAlert;
import org.openqa.selenium.support.FindBy;
/** /**
* *
@ -15,6 +18,9 @@ public abstract class AbstractAccountManagementTest extends AbstractAuthTest {
@Page @Page
protected AccountManagement testRealmAccountManagementPage; protected AccountManagement testRealmAccountManagementPage;
@FindBy(className = "alert")
protected AccountManagementAlert alert;
@Override @Override
public void setDefaultPageUriParameters() { public void setDefaultPageUriParameters() {
super.setDefaultPageUriParameters(); super.setDefaultPageUriParameters();
@ -28,4 +34,14 @@ public abstract class AbstractAccountManagementTest extends AbstractAuthTest {
createTestUserWithAdminClient(); createTestUserWithAdminClient();
} }
public void assertAlertSuccess() {
alert.waitUntilPresentAndClassSet();
assertTrue(alert.isSuccess());
}
public void assertAlertError() {
alert.waitUntilPresentAndClassSet();
assertTrue(alert.isError());
}
} }

View file

@ -66,7 +66,7 @@ public class AccountTest extends AbstractAccountManagementTest {
testRealmAccountPage.setFirstName(NEW_FIRST_NAME); testRealmAccountPage.setFirstName(NEW_FIRST_NAME);
testRealmAccountPage.setLastName(NEW_LAST_NAME); testRealmAccountPage.setLastName(NEW_LAST_NAME);
testRealmAccountPage.save(); testRealmAccountPage.save();
assertFlashMessageSuccess(); assertAlertSuccess();
testRealmAccountManagementPage.signOut(); testRealmAccountManagementPage.signOut();
testRealmLoginPage.form().login(testUser); testRealmLoginPage.form().login(testUser);

View file

@ -38,20 +38,20 @@ public class ChangePasswordTest extends AbstractAccountManagementTest {
@Test @Test
public void invalidChangeAttempts() { public void invalidChangeAttempts() {
testRealmChangePasswordPage.save(); testRealmChangePasswordPage.save();
assertFlashMessageError(); assertAlertError();
testRealmChangePasswordPage.changePasswords(WRONG_PASSWORD, NEW_PASSWORD, NEW_PASSWORD); testRealmChangePasswordPage.changePasswords(WRONG_PASSWORD, NEW_PASSWORD, NEW_PASSWORD);
assertFlashMessageError(); assertAlertError();
testRealmChangePasswordPage.changePasswords(correctPassword, NEW_PASSWORD, NEW_PASSWORD + "-mismatch"); testRealmChangePasswordPage.changePasswords(correctPassword, NEW_PASSWORD, NEW_PASSWORD + "-mismatch");
assertFlashMessageError(); assertAlertError();
} }
@Test @Test
public void successfulChangeAttempts() { public void successfulChangeAttempts() {
// change password successfully // change password successfully
testRealmChangePasswordPage.changePasswords(correctPassword, NEW_PASSWORD, NEW_PASSWORD); testRealmChangePasswordPage.changePasswords(correctPassword, NEW_PASSWORD, NEW_PASSWORD);
assertFlashMessageSuccess(); assertAlertSuccess();
// login using new password // login using new password
testRealmAccountManagementPage.signOut(); testRealmAccountManagementPage.signOut();
@ -61,7 +61,7 @@ public class ChangePasswordTest extends AbstractAccountManagementTest {
// change password back // change password back
testRealmAccountManagementPage.password(); testRealmAccountManagementPage.password();
testRealmChangePasswordPage.changePasswords(NEW_PASSWORD, correctPassword, correctPassword); testRealmChangePasswordPage.changePasswords(NEW_PASSWORD, correctPassword, correctPassword);
assertFlashMessageSuccess(); assertAlertSuccess();
} }
} }

View file

@ -17,7 +17,9 @@
*/ */
package org.keycloak.testsuite.console; package org.keycloak.testsuite.console;
import org.jboss.arquillian.graphene.findby.FindByJQuery;
import org.jboss.arquillian.graphene.page.Page; import org.jboss.arquillian.graphene.page.Page;
import static org.junit.Assert.assertTrue;
import org.junit.Before; import org.junit.Before;
import org.keycloak.representations.idm.UserRepresentation; import org.keycloak.representations.idm.UserRepresentation;
import org.keycloak.testsuite.AbstractAuthTest; import org.keycloak.testsuite.AbstractAuthTest;
@ -27,6 +29,7 @@ import org.keycloak.testsuite.console.page.AdminConsoleRealm.ConfigureMenu;
import org.keycloak.testsuite.console.page.AdminConsoleRealm.ManageMenu; import org.keycloak.testsuite.console.page.AdminConsoleRealm.ManageMenu;
import static org.keycloak.testsuite.auth.page.AuthRealm.TEST; import static org.keycloak.testsuite.auth.page.AuthRealm.TEST;
import org.keycloak.testsuite.auth.page.login.Login; import org.keycloak.testsuite.auth.page.login.Login;
import org.keycloak.testsuite.console.page.fragment.AdminConsoleAlert;
import org.keycloak.testsuite.console.page.fragment.ModalDialog; import org.keycloak.testsuite.console.page.fragment.ModalDialog;
import static org.keycloak.testsuite.util.URLAssert.assertCurrentUrlStartsWithLoginUrlOf; import static org.keycloak.testsuite.util.URLAssert.assertCurrentUrlStartsWithLoginUrlOf;
import static org.keycloak.testsuite.util.URLAssert.assertCurrentUrlStartsWith; import static org.keycloak.testsuite.util.URLAssert.assertCurrentUrlStartsWith;
@ -50,6 +53,9 @@ public abstract class AbstractConsoleTest extends AbstractAuthTest {
@FindBy(xpath = "//div[@class='modal-dialog']") @FindBy(xpath = "//div[@class='modal-dialog']")
protected ModalDialog modalDialog; protected ModalDialog modalDialog;
@FindBy(className = "alert")
protected AdminConsoleAlert alert;
protected boolean adminLoggedIn = false; protected boolean adminLoggedIn = false;
@Override @Override
@ -100,6 +106,18 @@ public abstract class AbstractConsoleTest extends AbstractAuthTest {
assertCurrentUrlStartsWithLoginUrlOf(adminConsole); assertCurrentUrlStartsWithLoginUrlOf(adminConsole);
} }
public void assertAlertSuccess() {
alert.waitUntilPresentAndClassSet();
assertTrue(alert.isSuccess());
alert.close();
}
public void assertAlertDanger() {
alert.waitUntilPresentAndClassSet();
assertTrue(alert.isDanger());
alert.close();
}
public ConfigureMenu configure() { public ConfigureMenu configure() {
return adminConsoleRealmPage.configure(); return adminConsoleRealmPage.configure();
} }

View file

@ -50,18 +50,18 @@ public class PasswordPolicyTest extends AbstractConsoleTest {
passwordPolicyPage.navigateTo(); passwordPolicyPage.navigateTo();
passwordPolicyPage.addPolicy(HASH_ITERATIONS, 5); passwordPolicyPage.addPolicy(HASH_ITERATIONS, 5);
passwordPolicyPage.removePolicy(HASH_ITERATIONS); passwordPolicyPage.removePolicy(HASH_ITERATIONS);
assertFlashMessageSuccess(); assertAlertSuccess();
} }
@Test @Test
public void testInvalidPolicyValues() { public void testInvalidPolicyValues() {
passwordPolicyPage.navigateTo(); passwordPolicyPage.navigateTo();
passwordPolicyPage.addPolicy(HASH_ITERATIONS, "asd"); passwordPolicyPage.addPolicy(HASH_ITERATIONS, "asd");
assertFlashMessageDanger(); assertAlertDanger();
passwordPolicyPage.removePolicy(HASH_ITERATIONS); passwordPolicyPage.removePolicy(HASH_ITERATIONS);
passwordPolicyPage.addPolicy(REGEX_PATTERN, "(["); passwordPolicyPage.addPolicy(REGEX_PATTERN, "([");
assertFlashMessageDanger(); assertAlertDanger();
} }
@Test @Test
@ -72,10 +72,10 @@ public class PasswordPolicyTest extends AbstractConsoleTest {
testUserCredentialsPage.navigateTo(); testUserCredentialsPage.navigateTo();
testUserCredentialsPage.resetPassword("1234567"); testUserCredentialsPage.resetPassword("1234567");
assertFlashMessageDanger(); assertAlertDanger();
testUserCredentialsPage.resetPassword("12345678"); testUserCredentialsPage.resetPassword("12345678");
assertFlashMessageSuccess(); assertAlertSuccess();
} }
@Test @Test
@ -86,10 +86,10 @@ public class PasswordPolicyTest extends AbstractConsoleTest {
testUserCredentialsPage.navigateTo(); testUserCredentialsPage.navigateTo();
testUserCredentialsPage.resetPassword("invalidPassword1"); testUserCredentialsPage.resetPassword("invalidPassword1");
assertFlashMessageDanger(); assertAlertDanger();
testUserCredentialsPage.resetPassword("validPassword12"); testUserCredentialsPage.resetPassword("validPassword12");
assertFlashMessageSuccess(); assertAlertSuccess();
} }
@Test @Test
@ -100,10 +100,10 @@ public class PasswordPolicyTest extends AbstractConsoleTest {
testUserCredentialsPage.navigateTo(); testUserCredentialsPage.navigateTo();
testUserCredentialsPage.resetPassword("iNVALIDPASSWORD"); testUserCredentialsPage.resetPassword("iNVALIDPASSWORD");
assertFlashMessageDanger(); assertAlertDanger();
testUserCredentialsPage.resetPassword("vaLIDPASSWORD"); testUserCredentialsPage.resetPassword("vaLIDPASSWORD");
assertFlashMessageSuccess(); assertAlertSuccess();
} }
@Test @Test
@ -114,10 +114,10 @@ public class PasswordPolicyTest extends AbstractConsoleTest {
testUserCredentialsPage.navigateTo(); testUserCredentialsPage.navigateTo();
testUserCredentialsPage.resetPassword("Invalidpassword"); testUserCredentialsPage.resetPassword("Invalidpassword");
assertFlashMessageDanger(); assertAlertDanger();
testUserCredentialsPage.resetPassword("VAlidpassword"); testUserCredentialsPage.resetPassword("VAlidpassword");
assertFlashMessageSuccess(); assertAlertSuccess();
} }
@Test @Test
@ -128,10 +128,10 @@ public class PasswordPolicyTest extends AbstractConsoleTest {
testUserCredentialsPage.navigateTo(); testUserCredentialsPage.navigateTo();
testUserCredentialsPage.resetPassword("invalidPassword*"); testUserCredentialsPage.resetPassword("invalidPassword*");
assertFlashMessageDanger(); assertAlertDanger();
testUserCredentialsPage.resetPassword("validPassword*#"); testUserCredentialsPage.resetPassword("validPassword*#");
assertFlashMessageSuccess(); assertAlertSuccess();
} }
@Test @Test
@ -142,10 +142,10 @@ public class PasswordPolicyTest extends AbstractConsoleTest {
testUserCredentialsPage.navigateTo(); testUserCredentialsPage.navigateTo();
testUserCredentialsPage.resetPassword(testUser.getUsername()); testUserCredentialsPage.resetPassword(testUser.getUsername());
assertFlashMessageDanger(); assertAlertDanger();
testUserCredentialsPage.resetPassword("validpassword"); testUserCredentialsPage.resetPassword("validpassword");
assertFlashMessageSuccess(); assertAlertSuccess();
} }
@Test @Test
@ -156,10 +156,10 @@ public class PasswordPolicyTest extends AbstractConsoleTest {
testUserCredentialsPage.navigateTo(); testUserCredentialsPage.navigateTo();
testUserCredentialsPage.resetPassword("invalidPassword"); testUserCredentialsPage.resetPassword("invalidPassword");
assertFlashMessageDanger(); assertAlertDanger();
testUserCredentialsPage.resetPassword("VALID#password"); testUserCredentialsPage.resetPassword("VALID#password");
assertFlashMessageSuccess(); assertAlertSuccess();
} }
@Test @Test
@ -170,13 +170,13 @@ public class PasswordPolicyTest extends AbstractConsoleTest {
testUserCredentialsPage.navigateTo(); testUserCredentialsPage.navigateTo();
testUserCredentialsPage.resetPassword("firstPassword"); testUserCredentialsPage.resetPassword("firstPassword");
assertFlashMessageSuccess(); assertAlertSuccess();
testUserCredentialsPage.resetPassword("secondPassword"); testUserCredentialsPage.resetPassword("secondPassword");
assertFlashMessageSuccess(); assertAlertSuccess();
testUserCredentialsPage.resetPassword("firstPassword"); testUserCredentialsPage.resetPassword("firstPassword");
assertFlashMessageDanger(); assertAlertDanger();
} }
} }

View file

@ -60,7 +60,7 @@ public class RequiredActionsTest extends AbstractConsoleTest {
public void termsAndConditionsDefaultActionTest() { public void termsAndConditionsDefaultActionTest() {
requiredActionsPage.setTermsAndConditionEnabled(true); requiredActionsPage.setTermsAndConditionEnabled(true);
requiredActionsPage.setTermsAndConditionDefaultAction(true); requiredActionsPage.setTermsAndConditionDefaultAction(true);
assertFlashMessageSuccess(); assertAlertSuccess();
allowTestRealmUserRegistration(); allowTestRealmUserRegistration();
@ -74,7 +74,7 @@ public class RequiredActionsTest extends AbstractConsoleTest {
@Test @Test
public void configureTotpDefaultActionTest() { public void configureTotpDefaultActionTest() {
requiredActionsPage.setConfigureTotpDefaultAction(true); requiredActionsPage.setConfigureTotpDefaultAction(true);
assertFlashMessageSuccess(); assertAlertSuccess();
allowTestRealmUserRegistration(); allowTestRealmUserRegistration();

View file

@ -36,7 +36,7 @@ public class ClientRolesTest extends AbstractClientTest {
// assertCurrentUrl(createClientRole); // can't do this, need client id to build uri // assertCurrentUrl(createClientRole); // can't do this, need client id to build uri
createClientRolePage.form().setBasicAttributes(roleRep); createClientRolePage.form().setBasicAttributes(roleRep);
createClientRolePage.form().save(); createClientRolePage.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
createClientRolePage.form().setCompositeRoles(roleRep); createClientRolePage.form().setCompositeRoles(roleRep);
// TODO add verification of notification message when KEYCLOAK-1497 gets resolved // TODO add verification of notification message when KEYCLOAK-1497 gets resolved
} }
@ -47,11 +47,10 @@ public class ClientRolesTest extends AbstractClientTest {
RoleRepresentation newRole = new RoleRepresentation("client-role", "", false); RoleRepresentation newRole = new RoleRepresentation("client-role", "", false);
createClient(newClient); createClient(newClient);
assertFlashMessageSuccess(); assertAlertSuccess();
clientPage.tabs().roles(); clientPage.tabs().roles();
addClientRole(newRole); addClientRole(newRole);
assertFlashMessageSuccess();
clientRolePage.backToClientRolesViaBreadcrumb(); clientRolePage.backToClientRolesViaBreadcrumb();
assertFalse(clientRolesPage.roles().getRolesFromTableRows().isEmpty()); assertFalse(clientRolesPage.roles().getRolesFromTableRows().isEmpty());
@ -60,7 +59,7 @@ public class ClientRolesTest extends AbstractClientTest {
clientsPage.table().search(newClient.getClientId()); clientsPage.table().search(newClient.getClientId());
clientsPage.table().deleteClient(newClient.getClientId()); clientsPage.table().deleteClient(newClient.getClientId());
modalDialog.confirmDeletion(); modalDialog.confirmDeletion();
assertFlashMessageSuccess(); assertAlertSuccess();
assertNull(clientsPage.table().findClient(newClient.getClientId())); assertNull(clientsPage.table().findClient(newClient.getClientId()));
} }

View file

@ -48,7 +48,7 @@ public class ClientSettingsTest extends AbstractClientTest {
public void crudOIDCConfidential() { public void crudOIDCConfidential() {
newClient = createClientRepresentation("oidc-confidential", "http://example.test/app/*"); newClient = createClientRepresentation("oidc-confidential", "http://example.test/app/*");
createClient(newClient); createClient(newClient);
assertFlashMessageSuccess(); assertAlertSuccess();
clientPage.backToClientsViaBreadcrumb(); clientPage.backToClientsViaBreadcrumb();
assertCurrentUrlEquals(clientsPage); assertCurrentUrlEquals(clientsPage);
@ -70,7 +70,7 @@ public class ClientSettingsTest extends AbstractClientTest {
newClient = createClientRepresentation("oidc-public", "http://example.test/app/*"); newClient = createClientRepresentation("oidc-public", "http://example.test/app/*");
newClient.setPublicClient(true); newClient.setPublicClient(true);
createClient(newClient); createClient(newClient);
assertFlashMessageSuccess(); assertAlertSuccess();
clientPage.backToClientsViaBreadcrumb(); clientPage.backToClientsViaBreadcrumb();
assertCurrentUrlEquals(clientsPage); assertCurrentUrlEquals(clientsPage);
@ -81,7 +81,7 @@ public class ClientSettingsTest extends AbstractClientTest {
newClient = createClientRepresentation("oidc-bearer-only", "http://example.test/app/*"); newClient = createClientRepresentation("oidc-bearer-only", "http://example.test/app/*");
newClient.setBearerOnly(true); newClient.setBearerOnly(true);
createClient(newClient); createClient(newClient);
assertFlashMessageSuccess(); assertAlertSuccess();
clientPage.backToClientsViaBreadcrumb(); clientPage.backToClientsViaBreadcrumb();
assertCurrentUrlEquals(clientsPage); assertCurrentUrlEquals(clientsPage);
@ -99,11 +99,11 @@ public class ClientSettingsTest extends AbstractClientTest {
public void invalidSettings() { public void invalidSettings() {
clientsPage.table().createClient(); clientsPage.table().createClient();
createClientPage.form().save(); createClientPage.form().save();
assertFlashMessageDanger(); assertAlertDanger();
createClientPage.form().setClientId("test-client"); createClientPage.form().setClientId("test-client");
createClientPage.form().save(); createClientPage.form().save();
assertFlashMessageDanger(); assertAlertDanger();
} }
public void assertClientSettingsEqual(ClientRepresentation c1, ClientRepresentation c2) { public void assertClientSettingsEqual(ClientRepresentation c1, ClientRepresentation c2) {

View file

@ -30,7 +30,7 @@ public class ConfigTest extends AbstractConsoleTest {
configPage.form().removeSaveType("LOGIN"); configPage.form().removeSaveType("LOGIN");
configPage.form().setExpiration("50", "Days"); configPage.form().setExpiration("50", "Days");
configPage.form().save(); configPage.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
RealmRepresentation realm = testRealmResource().toRepresentation(); RealmRepresentation realm = testRealmResource().toRepresentation();
assertTrue(realm.isEventsEnabled()); assertTrue(realm.isEventsEnabled());
@ -44,7 +44,7 @@ public class ConfigTest extends AbstractConsoleTest {
configPage.form().setSaveAdminEvents(true); configPage.form().setSaveAdminEvents(true);
configPage.form().setIncludeRepresentation(true); configPage.form().setIncludeRepresentation(true);
configPage.form().save(); configPage.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
RealmRepresentation realm = testRealmResource().toRepresentation(); RealmRepresentation realm = testRealmResource().toRepresentation();
assertTrue(realm.isAdminEventsEnabled()); assertTrue(realm.isAdminEventsEnabled());

View file

@ -33,7 +33,7 @@ public class KerberosUserFederationTest extends AbstractConsoleTest {
createKerberosUserProvider.form().selectEditMode(READ_ONLY); createKerberosUserProvider.form().selectEditMode(READ_ONLY);
createKerberosUserProvider.form().setUpdateProfileFirstLogin(true); createKerberosUserProvider.form().setUpdateProfileFirstLogin(true);
createKerberosUserProvider.form().save(); createKerberosUserProvider.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
RealmRepresentation realm = testRealmResource().toRepresentation(); RealmRepresentation realm = testRealmResource().toRepresentation();
UserFederationProviderRepresentation ufpr = realm.getUserFederationProviders().get(0); UserFederationProviderRepresentation ufpr = realm.getUserFederationProviders().get(0);
assertKerberosSetings(ufpr, "KEYCLOAK.ORG", "HTTP/localhost@KEYCLOAK.ORG", "http.keytab", "true", "true", "true"); assertKerberosSetings(ufpr, "KEYCLOAK.ORG", "HTTP/localhost@KEYCLOAK.ORG", "http.keytab", "true", "true", "true");
@ -50,18 +50,18 @@ public class KerberosUserFederationTest extends AbstractConsoleTest {
createKerberosUserProvider.form().selectEditMode(UNSYNCED); createKerberosUserProvider.form().selectEditMode(UNSYNCED);
createKerberosUserProvider.form().setUpdateProfileFirstLogin(true); createKerberosUserProvider.form().setUpdateProfileFirstLogin(true);
createKerberosUserProvider.form().save(); createKerberosUserProvider.form().save();
assertFlashMessageDanger(); assertAlertDanger();
createKerberosUserProvider.form().setServerPrincipalInput(""); createKerberosUserProvider.form().setServerPrincipalInput("");
createKerberosUserProvider.form().setKerberosRealmInput("KEYCLOAK.ORG");; createKerberosUserProvider.form().setKerberosRealmInput("KEYCLOAK.ORG");;
createKerberosUserProvider.form().save(); createKerberosUserProvider.form().save();
assertFlashMessageDanger(); assertAlertDanger();
createKerberosUserProvider.form().setServerPrincipalInput("HTTP/localhost@KEYCLOAK.ORG");; createKerberosUserProvider.form().setServerPrincipalInput("HTTP/localhost@KEYCLOAK.ORG");;
createKerberosUserProvider.form().setKeyTabInput(""); createKerberosUserProvider.form().setKeyTabInput("");
createKerberosUserProvider.form().save(); createKerberosUserProvider.form().save();
assertFlashMessageDanger(); assertAlertDanger();
createKerberosUserProvider.form().setKeyTabInput("http.keytab");; createKerberosUserProvider.form().setKeyTabInput("http.keytab");;
createKerberosUserProvider.form().save(); createKerberosUserProvider.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
} }
private void assertKerberosSetings(UserFederationProviderRepresentation ufpr, String kerberosRealm, String serverPrincipal, String keyTab, String debug, String useKerberosForPasswordAuthentication, String updateProfileFirstLogin) { private void assertKerberosSetings(UserFederationProviderRepresentation ufpr, String kerberosRealm, String serverPrincipal, String keyTab, String debug, String useKerberosForPasswordAuthentication, String updateProfileFirstLogin) {

View file

@ -49,7 +49,7 @@ public class LdapUserFederationTest extends AbstractConsoleTest {
createLdapUserProvider.form().setKeyTabInput("http.keytab"); createLdapUserProvider.form().setKeyTabInput("http.keytab");
createLdapUserProvider.form().setDebugEnabled(true); createLdapUserProvider.form().setDebugEnabled(true);
createLdapUserProvider.form().save(); createLdapUserProvider.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
RealmRepresentation realm = testRealmResource().toRepresentation(); RealmRepresentation realm = testRealmResource().toRepresentation();
UserFederationProviderRepresentation ufpr = realm.getUserFederationProviders().get(0); UserFederationProviderRepresentation ufpr = realm.getUserFederationProviders().get(0);
@ -71,7 +71,7 @@ public class LdapUserFederationTest extends AbstractConsoleTest {
createLdapUserProvider.form().setLdapUserDnInput("ou=People,dc=keycloak,dc=org"); createLdapUserProvider.form().setLdapUserDnInput("ou=People,dc=keycloak,dc=org");
createLdapUserProvider.form().setLdapBindCredentialInput("secret"); createLdapUserProvider.form().setLdapBindCredentialInput("secret");
createLdapUserProvider.form().save(); createLdapUserProvider.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
RealmRepresentation realm = testRealmResource().toRepresentation(); RealmRepresentation realm = testRealmResource().toRepresentation();
UserFederationProviderRepresentation ufpr = realm.getUserFederationProviders().get(0); UserFederationProviderRepresentation ufpr = realm.getUserFederationProviders().get(0);
@ -92,22 +92,22 @@ public class LdapUserFederationTest extends AbstractConsoleTest {
createLdapUserProvider.form().setLdapUserDnInput("ou=People,dc=keycloak,dc=org"); createLdapUserProvider.form().setLdapUserDnInput("ou=People,dc=keycloak,dc=org");
createLdapUserProvider.form().setLdapBindCredentialInput("secret"); createLdapUserProvider.form().setLdapBindCredentialInput("secret");
createLdapUserProvider.form().save(); createLdapUserProvider.form().save();
assertFlashMessageDanger(); assertAlertDanger();
createLdapUserProvider.form().setLdapUserDnInput(""); createLdapUserProvider.form().setLdapUserDnInput("");
createLdapUserProvider.form().setLdapConnectionUrlInput("ldap://localhost:389"); createLdapUserProvider.form().setLdapConnectionUrlInput("ldap://localhost:389");
createLdapUserProvider.form().save(); createLdapUserProvider.form().save();
assertFlashMessageDanger(); assertAlertDanger();
createLdapUserProvider.form().setLdapUserDnInput("ou=People,dc=keycloak,dc=org"); createLdapUserProvider.form().setLdapUserDnInput("ou=People,dc=keycloak,dc=org");
createLdapUserProvider.form().setLdapBindDnInput(""); createLdapUserProvider.form().setLdapBindDnInput("");
createLdapUserProvider.form().save(); createLdapUserProvider.form().save();
assertFlashMessageDanger(); assertAlertDanger();
createLdapUserProvider.form().setLdapBindDnInput("uid=admin,ou=system"); createLdapUserProvider.form().setLdapBindDnInput("uid=admin,ou=system");
createLdapUserProvider.form().setLdapBindCredentialInput(""); createLdapUserProvider.form().setLdapBindCredentialInput("");
createLdapUserProvider.form().save(); createLdapUserProvider.form().save();
assertFlashMessageDanger(); assertAlertDanger();
createLdapUserProvider.form().setLdapBindCredentialInput("secret"); createLdapUserProvider.form().setLdapBindCredentialInput("secret");
createLdapUserProvider.form().save(); createLdapUserProvider.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
} }
@Test @Test
@ -122,19 +122,19 @@ public class LdapUserFederationTest extends AbstractConsoleTest {
createLdapUserProvider.form().setLdapBindCredentialInput("secret"); createLdapUserProvider.form().setLdapBindCredentialInput("secret");
createLdapUserProvider.form().setAccountAfterPasswordUpdateEnabled(true); createLdapUserProvider.form().setAccountAfterPasswordUpdateEnabled(true);
createLdapUserProvider.form().save(); createLdapUserProvider.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
LDAPEmbeddedServer ldapServer = null; LDAPEmbeddedServer ldapServer = null;
try { try {
ldapServer = startEmbeddedLdapServer(); ldapServer = startEmbeddedLdapServer();
createLdapUserProvider.form().testConnection(); createLdapUserProvider.form().testConnection();
assertFlashMessageSuccess(); assertAlertSuccess();
createLdapUserProvider.form().testAuthentication(); createLdapUserProvider.form().testAuthentication();
assertFlashMessageSuccess(); assertAlertSuccess();
createLdapUserProvider.form().synchronizeAllUsers(); createLdapUserProvider.form().synchronizeAllUsers();
assertFlashMessageSuccess(); assertAlertSuccess();
createLdapUserProvider.form().setLdapBindCredentialInput("secret1"); createLdapUserProvider.form().setLdapBindCredentialInput("secret1");
createLdapUserProvider.form().testAuthentication(); createLdapUserProvider.form().testAuthentication();
assertFlashMessageDanger(); assertAlertDanger();
} finally { } finally {
if (ldapServer != null) { if (ldapServer != null) {
ldapServer.stop(); ldapServer.stop();

View file

@ -38,8 +38,7 @@ public class IdentityProviderTest extends AbstractConsoleTest {
// @Test // @Test
public void testAddNewProvider() { public void testAddNewProvider() {
idpSettingsPage.addNewProvider(new Provider(SocialProvider.FACEBOOK, "klic", "secret")); idpSettingsPage.addNewProvider(new Provider(SocialProvider.FACEBOOK, "klic", "secret"));
flashMessage.waitUntilPresent(); assertAlertSuccess();
assertTrue("Success message should be displayed", flashMessage.isSuccess());
} }
// @Test(expected = NoSuchElementException.class) // @Test(expected = NoSuchElementException.class)

View file

@ -74,7 +74,7 @@ public class SecurityDefensesTest extends AbstractRealmTest {
bruteForceDetectionPage.form().setWaitIncrementSelect(BruteForceDetection.TimeSelectValues.SECONDS); bruteForceDetectionPage.form().setWaitIncrementSelect(BruteForceDetection.TimeSelectValues.SECONDS);
bruteForceDetectionPage.form().setWaitIncrementInput(String.valueOf(secondsToWait)); bruteForceDetectionPage.form().setWaitIncrementInput(String.valueOf(secondsToWait));
bruteForceDetectionPage.form().save(); bruteForceDetectionPage.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
testRealmAccountPage.navigateTo(); testRealmAccountPage.navigateTo();
@ -115,7 +115,7 @@ public class SecurityDefensesTest extends AbstractRealmTest {
bruteForceDetectionPage.form().setMinQuickLoginWaitSelect(BruteForceDetection.TimeSelectValues.SECONDS); bruteForceDetectionPage.form().setMinQuickLoginWaitSelect(BruteForceDetection.TimeSelectValues.SECONDS);
bruteForceDetectionPage.form().setMinQuickLoginWaitInput(String.valueOf(secondsToWait)); bruteForceDetectionPage.form().setMinQuickLoginWaitInput(String.valueOf(secondsToWait));
bruteForceDetectionPage.form().save(); bruteForceDetectionPage.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
testRealmAccountPage.navigateTo(); testRealmAccountPage.navigateTo();
@ -195,7 +195,7 @@ public class SecurityDefensesTest extends AbstractRealmTest {
bruteForceDetectionPage.form().setFailureResetTimeSelect(BruteForceDetection.TimeSelectValues.SECONDS); bruteForceDetectionPage.form().setFailureResetTimeSelect(BruteForceDetection.TimeSelectValues.SECONDS);
bruteForceDetectionPage.form().setFailureResetTimeInput(String.valueOf(secondsToWait)); bruteForceDetectionPage.form().setFailureResetTimeInput(String.valueOf(secondsToWait));
bruteForceDetectionPage.form().save(); bruteForceDetectionPage.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
testRealmAccountPage.navigateTo(); testRealmAccountPage.navigateTo();
@ -228,7 +228,7 @@ public class SecurityDefensesTest extends AbstractRealmTest {
bruteForceDetectionPage.form().setWaitIncrementSelect(BruteForceDetection.TimeSelectValues.MINUTES); bruteForceDetectionPage.form().setWaitIncrementSelect(BruteForceDetection.TimeSelectValues.MINUTES);
bruteForceDetectionPage.form().setWaitIncrementInput("10"); bruteForceDetectionPage.form().setWaitIncrementInput("10");
bruteForceDetectionPage.form().save(); bruteForceDetectionPage.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
testRealmAccountPage.navigateTo(); testRealmAccountPage.navigateTo();

View file

@ -44,7 +44,7 @@ public class DefaultRolesTest extends AbstractRolesTest {
String defaultRoleName = defaultRoleRep.getName(); String defaultRoleName = defaultRoleRep.getName();
defaultRolesPage.form().addAvailableRole(defaultRoleName); defaultRolesPage.form().addAvailableRole(defaultRoleName);
assertFlashMessageSuccess(); assertAlertSuccess();
UserRepresentation newUser = new UserRepresentation(); UserRepresentation newUser = new UserRepresentation();
newUser.setUsername("new_user"); newUser.setUsername("new_user");

View file

@ -42,7 +42,7 @@ public class RealmRolesTest extends AbstractRolesTest {
assertCurrentUrlEquals(createRolePage); assertCurrentUrlEquals(createRolePage);
createRolePage.form().setBasicAttributes(roleRep); createRolePage.form().setBasicAttributes(roleRep);
createRolePage.form().save(); createRolePage.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
createRolePage.form().setCompositeRoles(roleRep); createRolePage.form().setCompositeRoles(roleRep);
// TODO add verification of notification message when KEYCLOAK-1497 gets resolved // TODO add verification of notification message when KEYCLOAK-1497 gets resolved
} }
@ -53,7 +53,7 @@ public class RealmRolesTest extends AbstractRolesTest {
// assertCurrentUrl(role); // can't do this, role id needed as uri param // assertCurrentUrl(role); // can't do this, role id needed as uri param
rolePage.form().setBasicAttributes(roleRep); rolePage.form().setBasicAttributes(roleRep);
rolePage.form().save(); rolePage.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
rolePage.form().setCompositeRoles(roleRep); rolePage.form().setCompositeRoles(roleRep);
} }
@ -64,6 +64,7 @@ public class RealmRolesTest extends AbstractRolesTest {
} }
@Test @Test
@Ignore
public void crudRole() { public void crudRole() {
addRole(testRole); addRole(testRole);
@ -77,7 +78,7 @@ public class RealmRolesTest extends AbstractRolesTest {
testRole.setDescription("updated role description"); testRole.setDescription("updated role description");
rolePage.form().setDescription(testRole.getDescription()); rolePage.form().setDescription(testRole.getDescription());
rolePage.form().save(); rolePage.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
configure().roles(); configure().roles();
foundRole = realmRolesPage.table().findRole(testRole.getName()); // search & get role from table foundRole = realmRolesPage.table().findRole(testRole.getName()); // search & get role from table
@ -111,13 +112,12 @@ public class RealmRolesTest extends AbstractRolesTest {
@Test @Test
public void testAddExistingRole() { public void testAddExistingRole() {
addRole(testRole); addRole(testRole);
assertFlashMessageSuccess();
configure().roles(); configure().roles();
realmRolesPage.table().addRole(); realmRolesPage.table().addRole();
createRolePage.form().setBasicAttributes(testRole); createRolePage.form().setBasicAttributes(testRole);
createRolePage.form().save(); createRolePage.form().save();
assertFlashMessageDanger(); assertAlertDanger();
} }
public void createTestRoles(String namePrefix, int count) { public void createTestRoles(String namePrefix, int count) {

View file

@ -67,7 +67,7 @@ public class RequiredUserActionsTest extends AbstractUserTest {
public void updatePassword() { public void updatePassword() {
userAttributesPage.form().addRequiredAction(UPDATE_PASSWORD.getActionName()); userAttributesPage.form().addRequiredAction(UPDATE_PASSWORD.getActionName());
userAttributesPage.form().save(); userAttributesPage.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
testRealmAccountPage.navigateTo(); testRealmAccountPage.navigateTo();
@ -91,7 +91,7 @@ public class RequiredUserActionsTest extends AbstractUserTest {
public void updateProfile() { public void updateProfile() {
userAttributesPage.form().addRequiredAction(UPDATE_PROFILE.getActionName()); userAttributesPage.form().addRequiredAction(UPDATE_PROFILE.getActionName());
userAttributesPage.form().save(); userAttributesPage.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
testRealmAccountPage.navigateTo(); testRealmAccountPage.navigateTo();
@ -128,7 +128,7 @@ public class RequiredUserActionsTest extends AbstractUserTest {
userAttributesPage.form().addRequiredAction(TERMS_AND_CONDITIONS.getActionName()); userAttributesPage.form().addRequiredAction(TERMS_AND_CONDITIONS.getActionName());
userAttributesPage.form().save(); userAttributesPage.form().save();
assertFlashMessageSuccess(); assertAlertSuccess();
testRealmAccountPage.navigateTo(); testRealmAccountPage.navigateTo();

View file

@ -49,7 +49,7 @@ public class UserAttributesTest extends AbstractUserTest {
setPasswordFor(newTestRealmUser, "pass"); setPasswordFor(newTestRealmUser, "pass");
newTestRealmUser.setEmail(invalidEmail); newTestRealmUser.setEmail(invalidEmail);
createUser(newTestRealmUser); createUser(newTestRealmUser);
assertFlashMessageDanger(); assertAlertDanger();
userAttributesPage.backToUsersViaBreadcrumb(); userAttributesPage.backToUsersViaBreadcrumb();
assertNull(usersPage.table().findUser(testUsername)); assertNull(usersPage.table().findUser(testUsername));
@ -58,7 +58,7 @@ public class UserAttributesTest extends AbstractUserTest {
@Test @Test
public void noUsername() { public void noUsername() {
createUser(newTestRealmUser); createUser(newTestRealmUser);
assertFlashMessageDanger(); assertAlertDanger();
} }
@Test @Test
@ -66,7 +66,7 @@ public class UserAttributesTest extends AbstractUserTest {
String testUsername = "test_duplicated_user"; String testUsername = "test_duplicated_user";
newTestRealmUser.setUsername(testUsername); newTestRealmUser.setUsername(testUsername);
createUser(newTestRealmUser); createUser(newTestRealmUser);
assertFlashMessageSuccess(); assertAlertSuccess();
userAttributesPage.backToUsersViaBreadcrumb(); userAttributesPage.backToUsersViaBreadcrumb();
assertNotNull(usersPage.table().findUser(testUsername)); assertNotNull(usersPage.table().findUser(testUsername));
@ -74,7 +74,7 @@ public class UserAttributesTest extends AbstractUserTest {
UserRepresentation testUser2 = new UserRepresentation(); UserRepresentation testUser2 = new UserRepresentation();
testUser2.setUsername(testUsername); testUser2.setUsername(testUsername);
createUser(testUser2); createUser(testUser2);
assertFlashMessageDanger(); assertAlertDanger();
} }
@Test @Test
@ -83,7 +83,7 @@ public class UserAttributesTest extends AbstractUserTest {
disabledUser.setEnabled(false); disabledUser.setEnabled(false);
disabledUser.setUsername("disabled_user"); disabledUser.setUsername("disabled_user");
createUser(disabledUser); createUser(disabledUser);
assertFlashMessageSuccess(); assertAlertSuccess();
// TODO try to log in // TODO try to log in
} }

View file

@ -9,6 +9,13 @@
<property name="browser">${browser}</property> <property name="browser">${browser}</property>
</extension> </extension>
<extension qualifier="graphene">
<property name="waitGuiInterval">5</property>
<property name="waitAjaxInterval">5</property>
<property name="waitModelInterval">10</property>
<property name="waitGuardInterval">5</property>
</extension>
<extension qualifier="graphene-secondbrowser"> <extension qualifier="graphene-secondbrowser">
<property name="browser">${browser}</property> <property name="browser">${browser}</property>
</extension> </extension>

View file

@ -609,6 +609,7 @@
</pluginManagement> </pluginManagement>
</build> </build>
</profile> </profile>
</profiles> </profiles>
</project> </project>