diff options
author | Sauli Tähkäpää <sauli@vaadin.com> | 2014-09-27 13:03:02 +0300 |
---|---|---|
committer | Vaadin Code Review <review@vaadin.com> | 2014-09-27 10:40:33 +0000 |
commit | 9b061b61608acbd1ca8732cc0cfa6aff4cb26d41 (patch) | |
tree | f69ac58860cb9248ce45680ae23c722adec65c63 | |
parent | 99b1c1a7936f250885e647983d7e30fd3a7f1018 (diff) | |
download | vaadin-framework-9b061b61608acbd1ca8732cc0cfa6aff4cb26d41.tar.gz vaadin-framework-9b061b61608acbd1ca8732cc0cfa6aff4cb26d41.zip |
Fix ThemeChangeOnTheFlyTest to work with autogenerated style. (#14670)
Change-Id: Iac7800a97e7263b51032579bd6ab0de9a95d59f8
-rw-r--r-- | uitest/src/com/vaadin/tests/themes/ThemeChangeOnTheFlyTest.java | 23 |
1 files changed, 10 insertions, 13 deletions
diff --git a/uitest/src/com/vaadin/tests/themes/ThemeChangeOnTheFlyTest.java b/uitest/src/com/vaadin/tests/themes/ThemeChangeOnTheFlyTest.java index eb010e82ee..a5657c4eec 100644 --- a/uitest/src/com/vaadin/tests/themes/ThemeChangeOnTheFlyTest.java +++ b/uitest/src/com/vaadin/tests/themes/ThemeChangeOnTheFlyTest.java @@ -15,10 +15,8 @@ */ package com.vaadin.tests.themes; -import java.io.IOException; -import java.util.List; - -import org.junit.Assert; +import com.vaadin.testbench.elements.ButtonElement; +import com.vaadin.tests.tb3.MultiBrowserTest; import org.junit.Test; import org.openqa.selenium.By; import org.openqa.selenium.WebDriver; @@ -26,8 +24,11 @@ import org.openqa.selenium.WebElement; import org.openqa.selenium.remote.DesiredCapabilities; import org.openqa.selenium.support.ui.ExpectedCondition; -import com.vaadin.testbench.elements.ButtonElement; -import com.vaadin.tests.tb3.MultiBrowserTest; +import java.io.IOException; +import java.util.List; + +import static org.hamcrest.CoreMatchers.containsString; +import static org.hamcrest.MatcherAssert.assertThat; public class ThemeChangeOnTheFlyTest extends MultiBrowserTest { @@ -100,9 +101,8 @@ public class ThemeChangeOnTheFlyTest extends MultiBrowserTest { @Override public Boolean apply(WebDriver input) { String rootClass = rootDiv.getAttribute("class").trim(); - String expected = "v-app " + theme; - expected = expected.trim(); - return rootClass.equals(expected); + + return rootClass.contains(theme); } }, 30); } @@ -110,12 +110,9 @@ public class ThemeChangeOnTheFlyTest extends MultiBrowserTest { private void assertOverlayTheme(String theme) { final WebElement overlayContainerDiv = findElement(By .xpath("//div[contains(@class,'v-overlay-container')]")); - String expected = "v-app v-overlay-container " + theme; - expected = expected.trim(); - String overlayClass = overlayContainerDiv.getAttribute("class").trim(); - Assert.assertEquals(expected, overlayClass); + assertThat(overlayClass, containsString(theme)); } } |