summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHenri Sara <hesara@vaadin.com>2017-08-31 09:08:10 +0300
committerHenri Sara <henri.sara@gmail.com>2017-08-31 09:50:13 +0300
commit514f3a88a24dd69da2fd3a8ce305f21f50b91919 (patch)
tree9319a4e5a372602683ddc99e456ec9bdd1650162
parent7641b63854057cb34c87a3bffe618123b6760db3 (diff)
downloadvaadin-framework-514f3a88a24dd69da2fd3a8ce305f21f50b91919.tar.gz
vaadin-framework-514f3a88a24dd69da2fd3a8ce305f21f50b91919.zip
Make BaseThemeTest more stable on IE11
Try to make the test more stable by using a different way of detecting the presence of a notification. This approach is used e.g. in GridComponentsTest.
-rw-r--r--uitest/src/test/java/com/vaadin/tests/components/uitest/ThemeTest.java3
1 files changed, 2 insertions, 1 deletions
diff --git a/uitest/src/test/java/com/vaadin/tests/components/uitest/ThemeTest.java b/uitest/src/test/java/com/vaadin/tests/components/uitest/ThemeTest.java
index f6f88baafe..39ee947fb6 100644
--- a/uitest/src/test/java/com/vaadin/tests/components/uitest/ThemeTest.java
+++ b/uitest/src/test/java/com/vaadin/tests/components/uitest/ThemeTest.java
@@ -129,7 +129,8 @@ public abstract class ThemeTest extends MultiBrowserTest {
private void testNotification(int id, String identifier)
throws IOException {
$(ButtonElement.class).id("notifButt" + id).click();
- waitForElementPresent(By.className("v-Notification"));
+ // For some reason, this seems to be more reliable on IE11
+ waitUntil(driver -> isElementPresent(NotificationElement.class), 10);
compareScreen(identifier);
$(NotificationElement.class).first().close();
}