fail faster

Author: rbri

src/test/java/org/htmlunit/CookieManager4Test.java

@@ -1151,4 +1151,15 @@ public void sameSiteIFrameFromSubDomain() throws Exception {
             assertEquals("unknown", mgr.getCookie("c5").getSameSite());
         }
     }
+
+    @Override
+    protected final WebDriver getWebDriver() {
+        final WebDriver driver = super.getWebDriver();
+        if (driver instanceof HtmlUnitDriver) {
+            // set timeout to fail fast when the url not mapped
+            ((HtmlUnitDriver) driver).getWebClient().getOptions().setTimeout(1000);
+        }
+
+        return driver;
+    }
 }

src/test/java/org/htmlunit/CookieManager5Test.java

@@ -172,16 +172,25 @@ public void updateCookieExpires() throws Exception {
         servlets.put(SetCookieExpires1000Servlet.URL, SetCookieExpires1000Servlet.class);
         startWebServer("./", null, servlets);
 
-        final WebClient webClient = getWebClient();
-        webClient.getPage("http://localhost:" + PORT + SetCookieExpires10Servlet.URL);
-        assertEquals(1, webClient.getCookieManager().getCookies().size());
-        Cookie cookie = webClient.getCookieManager().getCookies().iterator().next();
-        assertFalse("" + cookie.getExpires(), cookie.getExpires().after(date));
-
-        webClient.getPage("http://localhost:" + PORT + SetCookieExpires1000Servlet.URL);
-        assertEquals(1, webClient.getCookieManager().getCookies().size());
-        cookie = webClient.getCookieManager().getCookies().iterator().next();
-        assertTrue("" + cookie.getExpires(), cookie.getExpires().after(date));
+        try (WebClient webClient = getWebClient()) {
+            webClient.getPage("http://localhost:" + PORT + SetCookieExpires10Servlet.URL);
+            assertEquals(1, webClient.getCookieManager().getCookies().size());
+            Cookie cookie = webClient.getCookieManager().getCookies().iterator().next();
+            assertFalse("" + cookie.getExpires(), cookie.getExpires().after(date));
+
+            webClient.getPage("http://localhost:" + PORT + SetCookieExpires1000Servlet.URL);
+            assertEquals(1, webClient.getCookieManager().getCookies().size());
+            cookie = webClient.getCookieManager().getCookies().iterator().next();
+            assertTrue("" + cookie.getExpires(), cookie.getExpires().after(date));
+        }
+    }
+
+    @Override
+    protected WebClient getWebClient() {
+        final WebClient webClient = super.getWebClient();
+        // set timeout to fail fast when the url not mapped
+        webClient.getOptions().setTimeout(1000);
+        return webClient;
     }
 
     /**

src/test/java/org/htmlunit/WebServerTestCase.java

@@ -493,7 +493,7 @@ protected SslConnectionFactory getSslConnectionFactory() {
      * Returns the WebClient instance for the current test with the current {@link BrowserVersion}.
      * @return a WebClient with the current {@link BrowserVersion}
      */
-    protected final WebClient getWebClient() {
+    protected WebClient getWebClient() {
         if (webClient_ == null) {
             webClient_ = new WebClient(getBrowserVersion());
             webClient_.setAlertHandler(alertHandler_);