Fix CriticalNotificationsTest on Chrome

Without this fix, a mismatch in screenshots causes a non-screenshot
test failure.

Change-Id: I6fb157a764ef1ec594836cbeceacd898820e7e11

diff --git a/uitest/src/test/java/com/vaadin/tests/application/CriticalNotificationsTest.java b/uitest/src/test/java/com/vaadin/tests/application/CriticalNotificationsTest.java
index 6229edc1ee329ec564573572be10df182560840a..51fe3dfd06a18312fe3cfc7a37b396e88dbb824e 100644 (file)
--- a/uitest/src/test/java/com/vaadin/tests/application/CriticalNotificationsTest.java
+++ b/uitest/src/test/java/com/vaadin/tests/application/CriticalNotificationsTest.java
@@ -72,6 +72,12 @@ public class CriticalNotificationsTest extends MultiBrowserThemeTest {
             click($(CheckBoxElement.class).caption("Include details").first());
         }
         $(ButtonElement.class).caption(buttonCaption).first().click();
+
+        // some critical notifications invalidate the session, and if a
+        // screenshot does not match, waitForVaadin would cause the screenshot
+        // comparison to crash because of a missing session
+        testBench().disableWaitForVaadin();
+
         // Give the notification some time to animate
         sleep(1000);
         compareScreen($(NotificationElement.class).first(),