diff --git a/domiSMP-ui-tests/src/main/java/ddsl/StorePage.java b/domiSMP-ui-tests/src/main/java/ddsl/CommonCertificatePage.java similarity index 94% rename from domiSMP-ui-tests/src/main/java/ddsl/StorePage.java rename to domiSMP-ui-tests/src/main/java/ddsl/CommonCertificatePage.java index aaf37c84292428ba03a5411fc9f2d4ff2ea48e7b..3ca28b4b8a04506d25c96b8acd29a44d205788fa 100644 --- a/domiSMP-ui-tests/src/main/java/ddsl/StorePage.java +++ b/domiSMP-ui-tests/src/main/java/ddsl/CommonCertificatePage.java @@ -4,7 +4,7 @@ import org.openqa.selenium.WebDriver; import org.openqa.selenium.WebElement; import org.openqa.selenium.support.FindBy; -public class StorePage extends PageWithGrid { +public class CommonCertificatePage extends CommonPageWithGrid { /** * Common page used for Keystore and Truststore */ @@ -29,7 +29,7 @@ public class StorePage extends PageWithGrid { @FindBy(css = ".smp-warning-panel span") private WebElement smpWarningLbl; - public StorePage(WebDriver driver) { + public CommonCertificatePage(WebDriver driver) { super(driver); } diff --git a/domiSMP-ui-tests/src/main/java/ddsl/PageWithGrid.java b/domiSMP-ui-tests/src/main/java/ddsl/CommonPageWithGrid.java similarity index 90% rename from domiSMP-ui-tests/src/main/java/ddsl/PageWithGrid.java rename to domiSMP-ui-tests/src/main/java/ddsl/CommonPageWithGrid.java index 04fb19d0d31a62f80e15a9612f478e13b49aea5d..85e6da14a62a6609c5231aa40a01c4ec1bdc3ab7 100644 --- a/domiSMP-ui-tests/src/main/java/ddsl/PageWithGrid.java +++ b/domiSMP-ui-tests/src/main/java/ddsl/CommonPageWithGrid.java @@ -11,8 +11,8 @@ import org.slf4j.LoggerFactory; import java.util.List; -public class PageWithGrid extends DomiSMPPage { - private final static Logger LOG = LoggerFactory.getLogger(PageWithGrid.class); +public class CommonPageWithGrid extends DomiSMPPage { + private final static Logger LOG = LoggerFactory.getLogger(CommonPageWithGrid.class); /** * Generic page used for pages which have small grid in the right of the page. This element gives access to action buttons and elements of the page. @@ -29,7 +29,7 @@ public class PageWithGrid extends DomiSMPPage { @FindBy(css = "[role = \"tab\"]") private List<WebElement> tabList; - public PageWithGrid(WebDriver driver) { + public CommonPageWithGrid(WebDriver driver) { super(driver); PageFactory.initElements(new AjaxElementLocatorFactory(driver, data.getTIMEOUT()), this); } diff --git a/domiSMP-ui-tests/src/main/java/pages/administration/editDomainsPage/EditDomainsPage.java b/domiSMP-ui-tests/src/main/java/pages/administration/editDomainsPage/EditDomainsPage.java index 68438348958155203911ba6decd8e31fca2c1c13..b91b7f265b6e0748163d1222b5093459f779cd64 100644 --- a/domiSMP-ui-tests/src/main/java/pages/administration/editDomainsPage/EditDomainsPage.java +++ b/domiSMP-ui-tests/src/main/java/pages/administration/editDomainsPage/EditDomainsPage.java @@ -1,11 +1,11 @@ package pages.administration.editDomainsPage; -import ddsl.PageWithGrid; +import ddsl.CommonPageWithGrid; import org.openqa.selenium.WebDriver; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -public class EditDomainsPage extends PageWithGrid { +public class EditDomainsPage extends CommonPageWithGrid { /** * Page object for the Edit domains page. This contains the locators of the page and the methods for the behaviour of the page */ diff --git a/domiSMP-ui-tests/src/main/java/pages/systemSettings/TruststorePage.java b/domiSMP-ui-tests/src/main/java/pages/systemSettings/TruststorePage.java index 37565ccf084827da9c43338f58bc2c8298700f9c..9a47d96ccd3b77aadae318a2683fafd374027104 100644 --- a/domiSMP-ui-tests/src/main/java/pages/systemSettings/TruststorePage.java +++ b/domiSMP-ui-tests/src/main/java/pages/systemSettings/TruststorePage.java @@ -1,6 +1,6 @@ package pages.systemSettings; -import ddsl.StorePage; +import ddsl.CommonCertificatePage; import org.openqa.selenium.WebDriver; import org.openqa.selenium.WebElement; import org.openqa.selenium.support.FindBy; @@ -8,7 +8,7 @@ import org.openqa.selenium.support.FindBy; import java.util.regex.Matcher; import java.util.regex.Pattern; -public class TruststorePage extends StorePage { +public class TruststorePage extends CommonCertificatePage { /** * Page object for the Truststorepage. This contains the locators of the page and the methods for the behaviour of the page */ diff --git a/domiSMP-ui-tests/src/main/java/pages/systemSettings/UsersPage.java b/domiSMP-ui-tests/src/main/java/pages/systemSettings/UsersPage.java index e620cddba3e610ed47a0a561f20ae6626ca97516..44cdb562f90e2641710a80754166d28d3c207c9f 100644 --- a/domiSMP-ui-tests/src/main/java/pages/systemSettings/UsersPage.java +++ b/domiSMP-ui-tests/src/main/java/pages/systemSettings/UsersPage.java @@ -1,6 +1,6 @@ package pages.systemSettings; -import ddsl.PageWithGrid; +import ddsl.CommonPageWithGrid; import ddsl.dcomponents.commonComponents.UserDataCommonComponent; import ddsl.dobjects.DButton; import org.openqa.selenium.WebDriver; @@ -10,7 +10,7 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; import rest.models.UserModel; -public class UsersPage extends PageWithGrid { +public class UsersPage extends CommonPageWithGrid { /** * Page object for the Users page. This contains the locators of the page and the methods for the behaviour of the page */ diff --git a/domiSMP-ui-tests/src/main/java/pages/systemSettings/domainsPage/DomainsPage.java b/domiSMP-ui-tests/src/main/java/pages/systemSettings/domainsPage/DomainsPage.java index 45d32effe89dca9fcf84e2e9185d29fbbf9f58d1..9376628de4126cd9e692df22d32bfd028aa83f3e 100644 --- a/domiSMP-ui-tests/src/main/java/pages/systemSettings/domainsPage/DomainsPage.java +++ b/domiSMP-ui-tests/src/main/java/pages/systemSettings/domainsPage/DomainsPage.java @@ -1,6 +1,6 @@ package pages.systemSettings.domainsPage; -import ddsl.PageWithGrid; +import ddsl.CommonPageWithGrid; import ddsl.dobjects.DButton; import org.openqa.selenium.WebDriver; import org.openqa.selenium.WebElement; @@ -8,7 +8,7 @@ import org.openqa.selenium.support.FindBy; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -public class DomainsPage extends PageWithGrid { +public class DomainsPage extends CommonPageWithGrid { /** * Page object for the Users page. This contains the locators of the page and the methods for the behaviour of the page */ diff --git a/domiSMP-ui-tests/src/main/java/pages/systemSettings/keyStorePage/KeystorePage.java b/domiSMP-ui-tests/src/main/java/pages/systemSettings/keyStorePage/KeystorePage.java index fe709ba5523428f1524449663d9a58fcc8adedd7..1b4beb70c256476ce8e0d8787ebd1257f13e84ab 100644 --- a/domiSMP-ui-tests/src/main/java/pages/systemSettings/keyStorePage/KeystorePage.java +++ b/domiSMP-ui-tests/src/main/java/pages/systemSettings/keyStorePage/KeystorePage.java @@ -1,10 +1,10 @@ package pages.systemSettings.keyStorePage; -import ddsl.StorePage; +import ddsl.CommonCertificatePage; import org.openqa.selenium.WebDriver; -public class KeystorePage extends StorePage { +public class KeystorePage extends CommonCertificatePage { public KeystorePage(WebDriver driver) { super(driver); diff --git a/domiSMP-ui-tests/src/main/java/utils/TestRunData.java b/domiSMP-ui-tests/src/main/java/utils/TestRunData.java index ae5371c27f19615692a2dec4e73b0f631dd8a115..7ac5b309fdd7c4515f4ebf59f013c650f00f48e1 100644 --- a/domiSMP-ui-tests/src/main/java/utils/TestRunData.java +++ b/domiSMP-ui-tests/src/main/java/utils/TestRunData.java @@ -95,7 +95,7 @@ public class TestRunData { } public Duration getTIMEOUTinDuration() { - return Duration.ofMinutes(Long.parseLong((prop.getProperty("SHORT_TIMEOUT_SECONDS")))); + return Duration.ofSeconds(Long.parseLong((prop.getProperty("SHORT_TIMEOUT_SECONDS")))); } public Integer getLongWait() { @@ -103,7 +103,7 @@ public class TestRunData { } public Duration getLongWaitInDuration() { - return Duration.ofMinutes(Long.parseLong(prop.getProperty("LONG_TIMEOUT_SECONDS"))); + return Duration.ofSeconds(Long.parseLong(prop.getProperty("LONG_TIMEOUT_SECONDS"))); } diff --git a/domiSMP-ui-tests/src/test/java/domiSMPTests/ui/UsersPgTests.java b/domiSMP-ui-tests/src/test/java/domiSMPTests/ui/UsersPgTests.java index d200be36b918e99c193c4e296857a35b4cd13c24..1ff5d0ecf68fe59f66a90e55c2a326f4021363f9 100644 --- a/domiSMP-ui-tests/src/test/java/domiSMPTests/ui/UsersPgTests.java +++ b/domiSMP-ui-tests/src/test/java/domiSMPTests/ui/UsersPgTests.java @@ -4,18 +4,27 @@ import ddsl.DomiSMPPage; import ddsl.enums.Pages; import domiSMPTests.SeleniumTest; import org.openqa.selenium.WebElement; -import org.testng.Assert; +import org.testng.annotations.BeforeMethod; import org.testng.annotations.Test; +import org.testng.asserts.SoftAssert; import pages.LoginPage; import pages.systemSettings.UsersPage; import rest.models.UserModel; public class UsersPgTests extends SeleniumTest { + SoftAssert soft; + DomiSMPPage homePage; + LoginPage loginPage; + + @BeforeMethod(alwaysRun = true) + public void beforeTest() throws Exception { + soft = new SoftAssert(); + homePage = new DomiSMPPage(driver); + loginPage = homePage.goToLoginPage(); + } @Test(description = "USR-01 System admin is able to create new users") public void SystemAdminIsAbleToCreateNewUsers() throws Exception { - DomiSMPPage homePage = new DomiSMPPage(driver); - LoginPage loginPage = homePage.goToLoginPage(); loginPage.login(data.getAdminUser().get("username"), data.getAdminUser().get("password")); UsersPage usersPage = homePage.getSidebar().navigateTo(Pages.SYSTEM_SETTINGS_USERS); @@ -26,24 +35,25 @@ public class UsersPgTests extends SeleniumTest { usersPage.refreshPage(); // usersPage.filter(adminNewUserData.getUsername()); WebElement newUser = usersPage.getDataPanelGrid().searchAndGetElementInColumn("Username", adminNewUserData.getUsername()); - Assert.assertNotNull(newUser); + soft.assertNotNull(newUser); newUser.click(); - Assert.assertEquals(usersPage.getApplicationRoleValue(), adminNewUserData.getRole()); - Assert.assertEquals(usersPage.getFullNameValue(), adminNewUserData.getFullName()); - Assert.assertTrue(usersPage.isSelectedUserActive(), "User active status is true"); + soft.assertEquals(usersPage.getApplicationRoleValue(), adminNewUserData.getRole()); + soft.assertEquals(usersPage.getFullNameValue(), adminNewUserData.getFullName()); + soft.assertTrue(usersPage.isSelectedUserActive(), "User active status is true"); - Assert.assertEquals(usersPage.getEmailValue(), adminNewUserData.getEmailAddress()); - Assert.assertEquals(usersPage.getSelectedThemeValue(), adminNewUserData.getSmpTheme()); - Assert.assertEquals(usersPage.getSelectedLocaleValue(), adminNewUserData.getSmpLocale()); + soft.assertEquals(usersPage.getEmailValue(), adminNewUserData.getEmailAddress()); + soft.assertEquals(usersPage.getSelectedThemeValue(), adminNewUserData.getSmpTheme()); + soft.assertEquals(usersPage.getSelectedLocaleValue(), adminNewUserData.getSmpLocale()); + + soft.assertAll(); } @Test(description = "USR-02 USR-02 System admin is not able to create duplicated user") public void SystemAdminIsNotAbleToCreateDuplicatedUser() throws Exception { - DomiSMPPage homePage = new DomiSMPPage(driver); - LoginPage loginPage = homePage.goToLoginPage(); + loginPage.login(data.getAdminUser().get("username"), data.getAdminUser().get("password")); UsersPage usersPage = homePage.getSidebar().navigateTo(Pages.SYSTEM_SETTINGS_USERS); @@ -54,6 +64,7 @@ public class UsersPgTests extends SeleniumTest { usersPage.refreshPage(); usersPage.getCreateUserBtn().click(); String alertMessage = usersPage.fillNewUserDataAndSave(adminNewUserData); - Assert.assertEquals(alertMessage, "Invalid request [CreateUser]. Error: User with username [" + adminNewUserData.getUsername() + "] already exists!!"); + soft.assertEquals(alertMessage, "Invalid request [CreateUser]. Error: User with username [" + adminNewUserData.getUsername() + "] already exists!!"); + soft.assertAll(); } } \ No newline at end of file