diff options
author | Henri Sara <hesara@vaadin.com> | 2017-08-31 09:08:10 +0300 |
---|---|---|
committer | Henri Sara <henri.sara@gmail.com> | 2017-08-31 09:50:13 +0300 |
commit | 514f3a88a24dd69da2fd3a8ce305f21f50b91919 (patch) | |
tree | 9319a4e5a372602683ddc99e456ec9bdd1650162 | |
parent | 7641b63854057cb34c87a3bffe618123b6760db3 (diff) | |
download | vaadin-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.java | 3 |
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(); } |