test: Fix expectation on windows (#11808) (#11816)

Windows receives a different error string so we need to check the target os when running the test. Fixes #11777 Co-authored-by: caalador <[email protected]>
vaadin · Sep 10, 2021 · 9180ccc · 9180ccc
1 parent 80746ee
commit 9180ccc
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/.../test-dev-mode/src/test/java/com/vaadin/flow/uitest/ui/ClientSideExceptionHandlingIT.java b/.../test-dev-mode/src/test/java/com/vaadin/flow/uitest/ui/ClientSideExceptionHandlingIT.java
@@ -21,12 +21,15 @@
 import org.junit.Ignore;
 import org.junit.Test;
 import org.openqa.selenium.By;
+import org.openqa.selenium.JavascriptExecutor;
 
 import com.vaadin.flow.testutil.ChromeBrowserTest;
 
 public class ClientSideExceptionHandlingIT extends ChromeBrowserTest {
 
     private static final By ERROR_LOCATOR = By.className("v-system-error");
+    public static final String UNIX_PATTERN = ".*TypeError.* property 'foo' of.*null.*";
+    public static final String WINDOWS_PATTERN = ".*TypeError.* : Cannot read properties of null .*reading 'foo'.*";
 
     @Test
     public void developmentModeExceptions() {
@@ -35,9 +38,14 @@ public void developmentModeExceptions() {
 
         String errorMessage = findElement(ERROR_LOCATOR).getText();
 
+        // Windows formats the error differently from unix
+        final boolean isWindows = (boolean) ((JavascriptExecutor) getDriver())
+                .executeScript(
+                        "return navigator.appVersion.indexOf(\"Win\")!=-1");
+        String testPattern = isWindows ? WINDOWS_PATTERN : UNIX_PATTERN;
+
         Assert.assertTrue("Unexpected error message: " + errorMessage,
-                Pattern.matches(".*TypeError.* property 'foo' of.*null.*",
-                        errorMessage));
+                Pattern.matches(testPattern, errorMessage));
     }
 
     @Test