Chore: Change Jetty version to 9.4.38.v20210224 (#12252)

* Update Jetty version to 9.4.38.v20210224

* Update tests

diff --git a/pom.xml b/pom.xml
index efa44bb220272b537249e4f87bdc6b2cbb05e45c..d84e1b8ce52ebd476ee2fe5ef26945699e179080 100644 (file)
--- a/pom.xml
+++ b/pom.xml
@@ -47,8 +47,7 @@
         <!-- Dependency unpack directory -->
         <dependency.unpack.directory>${project.build.directory}/dependency-unpack</dependency.unpack.directory>
 
-        <!-- Cannot use 9.3 or 9.4 because they sometimes reset the connection too early in case of errors -->
-        <jetty.version>9.4.14.v20181114</jetty.version>
+        <jetty.version>9.4.38.v20210224</jetty.version>
         <phantomjs.version>2.1.1</phantomjs.version>
     </properties>
 

diff --git a/test/pom.xml b/test/pom.xml
index bf36456e2ab5ae19807eb45714328625a88c7543..c0f7c6f297a109e9edfe2bad544e856036f69fd8 100644 (file)
--- a/test/pom.xml
+++ b/test/pom.xml
@@ -11,7 +11,7 @@
         <maven.compiler.source>1.6</maven.compiler.source>
         <maven.compiler.target>1.6</maven.compiler.target>
         <failOnMissingWebXml>false</failOnMissingWebXml>
-        <jetty.version>9.3.7.v20160115</jetty.version>
+        <jetty.version>9.4.38.v20210224</jetty.version>
         <phantomjs.version>2.1.1</phantomjs.version>
         <testbench.version>4.2.2</testbench.version>
         <testbench.api.version>7.7.14</testbench.api.version>

diff --git a/test/servlet-containers/jetty9/pom.xml b/test/servlet-containers/jetty9/pom.xml
index d46b826786ace53ef058d024feccf680bf450040..4c420bd3c90f16ceef3c98d6cf1f2d53b9f61b64 100644 (file)
--- a/test/servlet-containers/jetty9/pom.xml
+++ b/test/servlet-containers/jetty9/pom.xml
@@ -12,7 +12,7 @@
     <packaging>war</packaging>
     <properties>
         <jetty.skip>false</jetty.skip>
-        <jetty.version>9.3.21.v20170918</jetty.version>
+        <jetty.version>9.4.38.v20210224</jetty.version>
         <server.name>jetty9</server.name>
     </properties>
 

diff --git a/uitest/src/test/java/com/vaadin/tests/applicationservlet/NoApplicationClassTest.java b/uitest/src/test/java/com/vaadin/tests/applicationservlet/NoApplicationClassTest.java
index 546be2988568aa10a79763d2739eded8995718ee..82737e40baa560dc9bb23ff479c7095ff68c0432 100644 (file)
--- a/uitest/src/test/java/com/vaadin/tests/applicationservlet/NoApplicationClassTest.java
+++ b/uitest/src/test/java/com/vaadin/tests/applicationservlet/NoApplicationClassTest.java
@@ -11,7 +11,7 @@ public class NoApplicationClassTest extends SingleBrowserTest {
     @Test
     public void testInvalidApplicationClass() {
         openTestURL();
-        String exceptionMessage = getDriver().findElement(By.xpath("//pre[2]"))
+        String exceptionMessage = getDriver().findElement(By.xpath("//pre[1]"))
                 .getText();
         String expected = "ServletException: java.lang.ClassNotFoundException: ClassThatIsNotPresent";
         Assert.assertTrue(

diff --git a/uitest/src/test/java/com/vaadin/tests/components/ui/InvalidViewportTest.java b/uitest/src/test/java/com/vaadin/tests/components/ui/InvalidViewportTest.java
index b33cfbf31930a0afbd67f55a17dabe90d7dabdf7..c28fbfa15d2fd2775dad86f860adc90719da58cb 100644 (file)
--- a/uitest/src/test/java/com/vaadin/tests/components/ui/InvalidViewportTest.java
+++ b/uitest/src/test/java/com/vaadin/tests/components/ui/InvalidViewportTest.java
@@ -15,7 +15,8 @@ public class InvalidViewportTest extends SingleBrowserTest {
 
         WebElement heading = findElement(By.tagName("h2"));
 
-        Assert.assertEquals("HTTP ERROR 500", heading.getText());
+        Assert.assertTrue("Unexpected heading: " + heading.getText(),
+                heading.getText().startsWith("HTTP ERROR 500"));
     }
 
 }

diff --git a/uitest/src/test/java/com/vaadin/tests/requesthandlers/AppResource404Test.java b/uitest/src/test/java/com/vaadin/tests/requesthandlers/AppResource404Test.java
index 99594290b4404289586d9ad6fc6b22aaecf98af2..8e40c75fed8cdef5e0fb3cc17f72aace2218b639 100644 (file)
--- a/uitest/src/test/java/com/vaadin/tests/requesthandlers/AppResource404Test.java
+++ b/uitest/src/test/java/com/vaadin/tests/requesthandlers/AppResource404Test.java
@@ -32,12 +32,14 @@ public class AppResource404Test extends MultiBrowserTest {
         openTestURL();
         $(LinkElement.class).get(2).click(5, 5);
         disableWaitingAndWait();
-        Assert.assertTrue("Page does not contain the given text",
+        String errorText = "Page does not contain the given text";
+        Assert.assertTrue(errorText,
                 driver.getPageSource().contains("HTTP ERROR 404"));
-        if (!BrowserUtil.isIE(getDesiredCapabilities())) {
-               Assert.assertTrue("Page does not contain the given text",
-                driver.getPageSource().contains("Problem accessing /run/APP/"));
-        }
+        Assert.assertTrue(errorText,
+                driver.getPageSource().contains("/run/APP/"));
+        Assert.assertTrue(errorText,
+                driver.getPageSource().contains(
+                        "Request was not handled by any registered handler"));
     }
 
     @Test