Upgrade to HtmlUnit 2.19

This upgrade includes AutoCloseable support for HtmlUnit WebConnections as introduced in 2.19, while remaining compatible with 2.18.

Issue: SPR-13686

Author: jhoeller

build.gradle

@@ -45,7 +45,7 @@ configure(allprojects) { project ->
 	ext.hibval4Version       = "4.3.2.Final"
 	ext.hibval5Version       = "5.2.2.Final"
 	ext.hsqldbVersion        = "2.3.3"
-	ext.htmlunitVersion      = "2.18"
+	ext.htmlunitVersion      = "2.19"
 	ext.httpasyncVersion     = "4.1.1"
 	ext.httpclientVersion    = "4.5.1"
 	ext.jackson2Version      = "2.6.3"
@@ -66,7 +66,7 @@ configure(allprojects) { project ->
 	ext.reactorVersion       = "2.0.7.RELEASE"
 	ext.romeVersion          = "1.5.1"
 	ext.seleniumVersion      = "2.48.2"
-	ext.slf4jVersion         = "1.7.12"
+	ext.slf4jVersion         = "1.7.13"
 	ext.snakeyamlVersion     = "1.16"
 	ext.snifferVersion       = "1.14"
 	ext.testngVersion        = "6.9.9"

spring-test/src/main/java/org/springframework/test/web/servlet/htmlunit/DelegatingWebConnection.java

@@ -20,12 +20,12 @@
 import java.util.Arrays;
 import java.util.List;
 
-import org.springframework.util.Assert;
-
 import com.gargoylesoftware.htmlunit.WebConnection;
 import com.gargoylesoftware.htmlunit.WebRequest;
 import com.gargoylesoftware.htmlunit.WebResponse;
 
+import org.springframework.util.Assert;
+
 /**
  * Implementation of {@link WebConnection} that allows delegating to various
  * {@code WebConnection} implementations.
@@ -62,8 +62,8 @@ public final class DelegatingWebConnection implements WebConnection {
 
 
 	public DelegatingWebConnection(WebConnection defaultConnection, List<DelegateWebConnection> connections) {
-		Assert.notNull(defaultConnection, "defaultConnection must not be null");
-		Assert.notEmpty(connections, "connections must not be empty");
+		Assert.notNull(defaultConnection, "Default WebConnection must not be null");
+		Assert.notEmpty(connections, "Connections List must not be empty");
 		this.connections = connections;
 		this.defaultConnection = defaultConnection;
 	}
@@ -72,6 +72,7 @@ public DelegatingWebConnection(WebConnection defaultConnection, DelegateWebConne
 		this(defaultConnection, Arrays.asList(connections));
 	}
 
+
 	@Override
 	public WebResponse getResponse(WebRequest request) throws IOException {
 		for (DelegateWebConnection connection : this.connections) {
@@ -82,26 +83,29 @@ public WebResponse getResponse(WebRequest request) throws IOException {
 		return this.defaultConnection.getResponse(request);
 	}
 
+	@Override
+	public void close() {
+	}
+
 
 	public static final class DelegateWebConnection {
 
 		private final WebRequestMatcher matcher;
 
 		private final WebConnection delegate;
 
-
 		public DelegateWebConnection(WebRequestMatcher matcher, WebConnection delegate) {
 			this.matcher = matcher;
 			this.delegate = delegate;
 		}
 
 		private WebRequestMatcher getMatcher() {
-			return matcher;
+			return this.matcher;
 		}
 
 		private WebConnection getDelegate() {
-			return delegate;
+			return this.delegate;
 		}
 	}
 
-}
+}

spring-test/src/main/java/org/springframework/test/web/servlet/htmlunit/ForwardRequestPostProcessor.java

@@ -31,7 +31,7 @@ final class ForwardRequestPostProcessor implements RequestPostProcessor {
 
 
 	public ForwardRequestPostProcessor(String forwardUrl) {
-		Assert.hasText(forwardUrl, "forwardUrl must not be null or empty");
+		Assert.hasText(forwardUrl, "Forward URL must not be null or empty");
 		this.forwardUrl = forwardUrl;
 	}
 

spring-test/src/main/java/org/springframework/test/web/servlet/htmlunit/HtmlUnitRequestBuilder.java

@@ -34,6 +34,11 @@
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpSession;
 
+import com.gargoylesoftware.htmlunit.CookieManager;
+import com.gargoylesoftware.htmlunit.WebClient;
+import com.gargoylesoftware.htmlunit.WebRequest;
+import com.gargoylesoftware.htmlunit.util.NameValuePair;
+
 import org.springframework.beans.Mergeable;
 import org.springframework.http.MediaType;
 import org.springframework.mock.web.MockHttpServletRequest;
@@ -46,11 +51,6 @@
 import org.springframework.web.util.UriComponents;
 import org.springframework.web.util.UriComponentsBuilder;
 
-import com.gargoylesoftware.htmlunit.CookieManager;
-import com.gargoylesoftware.htmlunit.WebClient;
-import com.gargoylesoftware.htmlunit.WebRequest;
-import com.gargoylesoftware.htmlunit.util.NameValuePair;
-
 /**
  * Internal class used to transform a {@link WebRequest} into a
  * {@link MockHttpServletRequest} using Spring MVC Test's {@link RequestBuilder}.
@@ -91,9 +91,9 @@ final class HtmlUnitRequestBuilder implements RequestBuilder, Mergeable {
 	 * {@link MockHttpServletRequest}; never {@code null}
 	 */
 	public HtmlUnitRequestBuilder(Map<String, MockHttpSession> sessions, WebClient webClient, WebRequest webRequest) {
-		Assert.notNull(sessions, "sessions map must not be null");
-		Assert.notNull(webClient, "webClient must not be null");
-		Assert.notNull(webRequest, "webRequest must not be null");
+		Assert.notNull(sessions, "Sessions Map must not be null");
+		Assert.notNull(webClient, "WebClient must not be null");
+		Assert.notNull(webRequest, "WebRequest must not be null");
 
 		this.sessions = sessions;
 		this.webClient = webClient;

spring-test/src/main/java/org/springframework/test/web/servlet/htmlunit/MockMvcWebClientBuilder.java

@@ -16,13 +16,13 @@
 
 package org.springframework.test.web.servlet.htmlunit;
 
+import com.gargoylesoftware.htmlunit.WebClient;
+
 import org.springframework.test.web.servlet.MockMvc;
 import org.springframework.test.web.servlet.setup.MockMvcConfigurer;
 import org.springframework.util.Assert;
 import org.springframework.web.context.WebApplicationContext;
 
-import com.gargoylesoftware.htmlunit.WebClient;
-
 /**
  * {@code MockMvcWebClientBuilder} simplifies the creation of an HtmlUnit
  * {@link WebClient} that delegates to a {@link MockMvc} instance.
@@ -57,6 +57,7 @@ protected MockMvcWebClientBuilder(WebApplicationContext context, MockMvcConfigur
 		super(context, configurer);
 	}
 
+
 	/**
 	 * Create a new {@code MockMvcWebClientBuilder} based on the supplied
 	 * {@link MockMvc} instance.
@@ -104,7 +105,7 @@ public static MockMvcWebClientBuilder webAppContextSetup(WebApplicationContext c
 	 * @see #build()
 	 */
 	public MockMvcWebClientBuilder withDelegate(WebClient webClient) {
-		Assert.notNull(webClient, "webClient must not be null");
+		Assert.notNull(webClient, "WebClient must not be null");
 		webClient.setWebConnection(createConnection(webClient.getWebConnection()));
 		this.webClient = webClient;
 		return this;
@@ -125,4 +126,4 @@ public WebClient build() {
 		return (this.webClient != null ? this.webClient : withDelegate(new WebClient()).build());
 	}
 
-}
+}

spring-test/src/main/java/org/springframework/test/web/servlet/htmlunit/MockMvcWebConnection.java

@@ -20,18 +20,18 @@
 import java.util.HashMap;
 import java.util.Map;
 
+import com.gargoylesoftware.htmlunit.WebClient;
+import com.gargoylesoftware.htmlunit.WebConnection;
+import com.gargoylesoftware.htmlunit.WebRequest;
+import com.gargoylesoftware.htmlunit.WebResponse;
+
 import org.springframework.mock.web.MockHttpServletResponse;
 import org.springframework.mock.web.MockHttpSession;
 import org.springframework.test.web.servlet.MockMvc;
 import org.springframework.test.web.servlet.RequestBuilder;
 import org.springframework.test.web.servlet.ResultActions;
 import org.springframework.util.Assert;
 
-import com.gargoylesoftware.htmlunit.WebClient;
-import com.gargoylesoftware.htmlunit.WebConnection;
-import com.gargoylesoftware.htmlunit.WebRequest;
-import com.gargoylesoftware.htmlunit.WebResponse;
-
 /**
  * {@code MockMvcWebConnection} enables {@link MockMvc} to transform a
  * {@link WebRequest} into a {@link WebResponse}.
@@ -86,21 +86,27 @@ public MockMvcWebConnection(MockMvc mockMvc) {
 	 * @param contextPath the contextPath to use
 	 */
 	public MockMvcWebConnection(MockMvc mockMvc, String contextPath) {
-		Assert.notNull(mockMvc, "mockMvc must not be null");
+		Assert.notNull(mockMvc, "MockMvc must not be null");
 		validateContextPath(contextPath);
 
 		this.webClient = new WebClient();
 		this.mockMvc = mockMvc;
 		this.contextPath = contextPath;
 	}
 
+
+	public void setWebClient(WebClient webClient) {
+		Assert.notNull(webClient, "WebClient must not be null");
+		this.webClient = webClient;
+	}
+
+
 	public WebResponse getResponse(WebRequest webRequest) throws IOException {
 		long startTime = System.currentTimeMillis();
 		HtmlUnitRequestBuilder requestBuilder = new HtmlUnitRequestBuilder(this.sessions, this.webClient, webRequest);
 		requestBuilder.setContextPath(this.contextPath);
 
 		MockHttpServletResponse httpServletResponse = getResponse(requestBuilder);
-
 		String forwardedUrl = httpServletResponse.getForwardedUrl();
 		while (forwardedUrl != null) {
 			requestBuilder.setForwardPostProcessor(new ForwardRequestPostProcessor(forwardedUrl));
@@ -111,23 +117,23 @@ public WebResponse getResponse(WebRequest webRequest) throws IOException {
 		return new MockWebResponseBuilder(startTime, webRequest, httpServletResponse).build();
 	}
 
-	public void setWebClient(WebClient webClient) {
-		Assert.notNull(webClient, "webClient must not be null");
-		this.webClient = webClient;
-	}
-
 	private MockHttpServletResponse getResponse(RequestBuilder requestBuilder) throws IOException {
 		ResultActions resultActions;
 		try {
 			resultActions = this.mockMvc.perform(requestBuilder);
 		}
-		catch (Exception e) {
-			throw (IOException) new IOException(e.getMessage()).initCause(e);
+		catch (Exception ex) {
+			throw new IOException(ex);
 		}
 
 		return resultActions.andReturn().getResponse();
 	}
 
+	@Override
+	public void close() {
+	}
+
+
 	/**
 	 * Validate the supplied {@code contextPath}.
 	 * <p>If the value is not {@code null}, it must conform to
@@ -148,4 +154,4 @@ static void validateContextPath(String contextPath) {
 		}
 	}
 
-}
+}

spring-test/src/main/java/org/springframework/test/web/servlet/htmlunit/MockMvcWebConnectionBuilderSupport.java

@@ -19,14 +19,14 @@
 import java.util.ArrayList;
 import java.util.List;
 
+import com.gargoylesoftware.htmlunit.WebConnection;
+
 import org.springframework.test.web.servlet.MockMvc;
 import org.springframework.test.web.servlet.setup.MockMvcBuilders;
 import org.springframework.test.web.servlet.setup.MockMvcConfigurer;
 import org.springframework.util.Assert;
 import org.springframework.web.context.WebApplicationContext;
 
-import com.gargoylesoftware.htmlunit.WebConnection;
-
 /**
  * Support class that simplifies the creation of a {@link WebConnection} that
  * uses {@link MockMvc} and optionally delegates to a real {@link WebConnection}
@@ -55,7 +55,7 @@ public abstract class MockMvcWebConnectionBuilderSupport<T extends MockMvcWebCon
 	 * @param mockMvc the {@code MockMvc} instance to use; never {@code null}
 	 */
 	protected MockMvcWebConnectionBuilderSupport(MockMvc mockMvc) {
-		Assert.notNull(mockMvc, "mockMvc must not be null");
+		Assert.notNull(mockMvc, "MockMvc must not be null");
 		this.mockMvc = mockMvc;
 		this.mockMvcRequestMatchers.add(new HostRequestMatcher("localhost"));
 	}
@@ -80,6 +80,7 @@ protected MockMvcWebConnectionBuilderSupport(WebApplicationContext context, Mock
 		this(MockMvcBuilders.webAppContextSetup(context).apply(configurer).build());
 	}
 
+
 	/**
 	 * Set the context path to use.
 	 * <p>If the supplied value is {@code null} or empty, the first path
@@ -146,7 +147,7 @@ public T useMockMvcForHosts(String... hosts) {
 	 * @see #useMockMvcForHosts(String...)
 	 */
 	protected final WebConnection createConnection(WebConnection defaultConnection) {
-		Assert.notNull(defaultConnection, "defaultConnection must not be null");
+		Assert.notNull(defaultConnection, "Default WebConnection must not be null");
 		MockMvcWebConnection mockMvcWebConnection = new MockMvcWebConnection(this.mockMvc, this.contextPath);
 
 		if (this.alwaysUseMockMvc) {
@@ -162,4 +163,4 @@ protected final WebConnection createConnection(WebConnection defaultConnection)
 		return new DelegatingWebConnection(defaultConnection, delegates);
 	}
 
-}
+}

spring-test/src/main/java/org/springframework/test/web/servlet/htmlunit/MockWebResponseBuilder.java

@@ -21,16 +21,16 @@
 import java.util.Collection;
 import java.util.List;
 
-import org.springframework.http.HttpStatus;
-import org.springframework.mock.web.MockHttpServletResponse;
-import org.springframework.util.Assert;
-import org.springframework.util.StringUtils;
-
 import com.gargoylesoftware.htmlunit.WebRequest;
 import com.gargoylesoftware.htmlunit.WebResponse;
 import com.gargoylesoftware.htmlunit.WebResponseData;
 import com.gargoylesoftware.htmlunit.util.NameValuePair;
 
+import org.springframework.http.HttpStatus;
+import org.springframework.mock.web.MockHttpServletResponse;
+import org.springframework.util.Assert;
+import org.springframework.util.StringUtils;
+
 /**
  * @author Rob Winch
  * @author Sam Brannen
@@ -48,8 +48,8 @@ final class MockWebResponseBuilder {
 
 
 	public MockWebResponseBuilder(long startTime, WebRequest webRequest, MockHttpServletResponse response) {
-		Assert.notNull(webRequest, "webRequest must not be null");
-		Assert.notNull(response, "response must not be null");
+		Assert.notNull(webRequest, "WebRequest must not be null");
+		Assert.notNull(response, "HttpServletResponse must not be null");
 		this.startTime = startTime;
 		this.webRequest = webRequest;
 		this.response = response;
@@ -63,8 +63,8 @@ public WebResponse build() throws IOException {
 
 	private WebResponseData webResponseData() throws IOException {
 		List<NameValuePair> responseHeaders = responseHeaders();
-		int statusCode = (this.response.getRedirectedUrl() != null ? HttpStatus.MOVED_PERMANENTLY.value()
-				: this.response.getStatus());
+		int statusCode = (this.response.getRedirectedUrl() != null ?
+				HttpStatus.MOVED_PERMANENTLY.value() : this.response.getStatus());
 		String statusMessage = statusMessage(statusCode);
 		return new WebResponseData(this.response.getContentAsByteArray(), statusCode, statusMessage, responseHeaders);
 	}