Skip to content
Snippets Groups Projects
Commit adc3e49d authored by Robert Goldmann's avatar Robert Goldmann
Browse files

Refactoring: use hasSize() with assertJ

parent 413bb315
No related branches found
No related tags found
No related merge requests found
...@@ -177,7 +177,7 @@ public class IntegrationTestHelper ...@@ -177,7 +177,7 @@ public class IntegrationTestHelper
.isEqualTo(Localization.getString("info.title.database.import.dialog")); .isEqualTo(Localization.getString("info.title.database.import.dialog"));
List<WebElement> tableRows = driver.findElements(By.cssSelector(".container form table tr")); List<WebElement> tableRows = driver.findElements(By.cssSelector(".container form table tr"));
assertThat(tableRows.size()).isEqualTo(destinationAccounts.size()); assertThat(tableRows).hasSize(destinationAccounts.size());
for(int i = 0; i < destinationAccounts.size(); i++) for(int i = 0; i < destinationAccounts.size(); i++)
{ {
......
...@@ -61,7 +61,7 @@ class SearchTest extends SeleniumTestBase ...@@ -61,7 +61,7 @@ class SearchTest extends SeleniumTestBase
// results // results
List<WebElement> results = driver.findElements(By.cssSelector(".search-container .card-panel")); List<WebElement> results = driver.findElements(By.cssSelector(".search-container .card-panel"));
assertThat(results.size()).isEqualTo(10); assertThat(results).hasSize(10);
} }
@Test @Test
...@@ -80,13 +80,13 @@ class SearchTest extends SeleniumTestBase ...@@ -80,13 +80,13 @@ class SearchTest extends SeleniumTestBase
// validate results // validate results
List<WebElement> results = driver.findElements(By.cssSelector(".search-container .card-panel")); List<WebElement> results = driver.findElements(By.cssSelector(".search-container .card-panel"));
assertThat(results.size()).isEqualTo(10); assertThat(results).hasSize(10);
// === PAGE 1 === // === PAGE 1 ===
pages.get(3).click(); pages.get(3).click();
pages = driver.findElements(By.cssSelector(".pagination-position-top .pagination li")); pages = driver.findElements(By.cssSelector(".pagination-position-top .pagination li"));
assertThat(pages.size()).isEqualTo(5); assertThat(results).hasSize(5);
// previous button should be enabled // previous button should be enabled
assertThat(pages.get(0).getAttribute("class")).doesNotContain("disabled"); assertThat(pages.get(0).getAttribute("class")).doesNotContain("disabled");
...@@ -101,7 +101,7 @@ class SearchTest extends SeleniumTestBase ...@@ -101,7 +101,7 @@ class SearchTest extends SeleniumTestBase
// validate // validate
results = driver.findElements(By.cssSelector(".search-container .card-panel")); results = driver.findElements(By.cssSelector(".search-container .card-panel"));
assertThat(results.size()).isEqualTo(4); assertThat(results).hasSize(4);
} }
@Test @Test
...@@ -124,7 +124,7 @@ class SearchTest extends SeleniumTestBase ...@@ -124,7 +124,7 @@ class SearchTest extends SeleniumTestBase
// results // results
List<WebElement> results = driver.findElements(By.cssSelector(".search-container .card-panel")); List<WebElement> results = driver.findElements(By.cssSelector(".search-container .card-panel"));
assertThat(results.size()).isEqualTo(2); assertThat(results).hasSize(2);
} }
@Test @Test
...@@ -135,7 +135,7 @@ class SearchTest extends SeleniumTestBase ...@@ -135,7 +135,7 @@ class SearchTest extends SeleniumTestBase
assertThat(driver.findElement(By.cssSelector(".headline-date")).getText()).isEqualTo("May 2019"); assertThat(driver.findElement(By.cssSelector(".headline-date")).getText()).isEqualTo("May 2019");
List<WebElement> transactionsRows = driver.findElements(By.cssSelector(".transaction-container .hide-on-med-and-down.transaction-row-top")); List<WebElement> transactionsRows = driver.findElements(By.cssSelector(".transaction-container .hide-on-med-and-down.transaction-row-top"));
assertThat(transactionsRows.size()).isEqualTo(25); assertThat(transactionsRows).hasSize(25);
assertThat(transactionsRows.get(0).getAttribute("class")).contains("background-blue-light"); assertThat(transactionsRows.get(0).getAttribute("class")).contains("background-blue-light");
for(int i = 1; i < transactionsRows.size(); i++) for(int i = 1; i < transactionsRows.size(); i++)
{ {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment