@WebMvcTest ignores base path configured via @WebAppConfiguration

spring-boot-project/spring-boot-test-autoconfigure/src/main/java/org/springframework/boot/test/autoconfigure/web/servlet/WebMvcTestContextBootstrapper.java

@@ -25,14 +25,17 @@
  * {@link TestContextBootstrapper} for {@link WebMvcTest @WebMvcTest} support.
  *
  * @author Phillip Webb
+ * @author Lorenzo Dee
  */
 class WebMvcTestContextBootstrapper extends SpringBootTestContextBootstrapper {
 
 	@Override
 	protected MergedContextConfiguration processMergedContextConfiguration(
 			MergedContextConfiguration mergedConfig) {
-		return new WebMergedContextConfiguration(
-				super.processMergedContextConfiguration(mergedConfig), "");
+		MergedContextConfiguration processedMergedConfiguration = super.processMergedContextConfiguration(
+				mergedConfig);
+		return new WebMergedContextConfiguration(processedMergedConfiguration,
+				getServletResourceBasePath(mergedConfig));
 	}
 
 }

spring-boot-project/spring-boot-test-autoconfigure/src/test/java/org/springframework/boot/test/autoconfigure/web/servlet/mockmvc/WebMvcTestServletContextResourceTests.java

@@ -0,0 +1,61 @@
+/*
+ * Copyright 2012-2019 the original author or authors.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.springframework.boot.test.autoconfigure.web.servlet.mockmvc;
+
+import java.net.MalformedURLException;
+import java.net.URL;
+
+import javax.servlet.ServletContext;
+
+import org.junit.Test;
+import org.junit.runner.RunWith;
+
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.boot.test.autoconfigure.web.servlet.WebMvcTest;
+import org.springframework.test.context.junit4.SpringRunner;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+/**
+ * Tests for {@link WebMvcTest} when loading resources via {@link ServletContext}.
+ *
+ * @author Lorenzo Dee
+ */
+@RunWith(SpringRunner.class)
+@WebMvcTest
+public class WebMvcTestServletContextResourceTests {
+
+	@Autowired
+	private ServletContext servletContext;
+
+	@Test
+	public void getResourceLocation() throws Exception {
+		testResource("/inwebapp", "src/main/webapp");
+		testResource("/inmetainfresources", "/META-INF/resources");
+		testResource("/inresources", "/resources");
+		testResource("/instatic", "/static");
+		testResource("/inpublic", "/public");
+	}
+
+	private void testResource(String path, String expectedLocation)
+			throws MalformedURLException {
+		URL resource = this.servletContext.getResource(path);
+		assertThat(resource).isNotNull();
+		assertThat(resource.getPath()).contains(expectedLocation);
+	}
+
+}

spring-boot-project/spring-boot-test-autoconfigure/src/test/java/org/springframework/boot/test/autoconfigure/web/servlet/mockmvc/WebMvcTestWithWebAppConfigurationTests.java

@@ -0,0 +1,64 @@
+/*
+ * Copyright 2012-2019 the original author or authors.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.springframework.boot.test.autoconfigure.web.servlet.mockmvc;
+
+import java.net.MalformedURLException;
+import java.net.URL;
+
+import javax.servlet.ServletContext;
+
+import org.junit.Test;
+import org.junit.runner.RunWith;
+
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.boot.test.autoconfigure.web.servlet.WebMvcTest;
+import org.springframework.test.context.junit4.SpringRunner;
+import org.springframework.test.context.web.WebAppConfiguration;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+/**
+ * Tests for {@link WebMvcTest} when loading resources via {@link ServletContext} with
+ * {@link WebAppConfiguration}.
+ *
+ * @author Lorenzo Dee
+ */
+@RunWith(SpringRunner.class)
+@WebMvcTest
+@WebAppConfiguration("src/test/webapp")
+public class WebMvcTestWithWebAppConfigurationTests {
+
+	@Autowired
+	private ServletContext servletContext;
+
+	@Test
+	public void getResourceLocation() throws Exception {
+		testResource("/inwebapp", "src/test/webapp");
+		testResource("/inmetainfresources", "/META-INF/resources");
+		testResource("/inresources", "/resources");
+		testResource("/instatic", "/static");
+		testResource("/inpublic", "/public");
+	}
+
+	private void testResource(String path, String expectedLocation)
+			throws MalformedURLException {
+		URL resource = this.servletContext.getResource(path);
+		assertThat(resource).isNotNull();
+		assertThat(resource.getPath()).contains(expectedLocation);
+	}
+
+}

spring-boot-project/spring-boot-test/src/main/java/org/springframework/boot/test/context/SpringBootTestContextBootstrapper.java

@@ -71,6 +71,7 @@
  * @author Andy Wilkinson
  * @author Brian Clozel
  * @author Madhura Bhave
+ * @author Lorenzo Dee
  * @since 1.4.0
  * @see SpringBootTest
  * @see TestConfiguration
@@ -163,13 +164,8 @@ protected MergedContextConfiguration processMergedContextConfiguration(
 			if (webApplicationType == WebApplicationType.SERVLET
 					&& (webEnvironment.isEmbedded()
 							|| webEnvironment == WebEnvironment.MOCK)) {
-				WebAppConfiguration webAppConfiguration = AnnotatedElementUtils
-						.findMergedAnnotation(mergedConfig.getTestClass(),
-								WebAppConfiguration.class);
-				String resourceBasePath = (webAppConfiguration != null)
-						? webAppConfiguration.value() : "src/main/webapp";
 				mergedConfig = new WebMergedContextConfiguration(mergedConfig,
-						resourceBasePath);
+						getServletResourceBasePath(mergedConfig));
 			}
 			else if (webApplicationType == WebApplicationType.REACTIVE
 					&& (webEnvironment.isEmbedded()
@@ -180,6 +176,15 @@ else if (webApplicationType == WebApplicationType.REACTIVE
 		return mergedConfig;
 	}
 
+	protected String getServletResourceBasePath(
+			MergedContextConfiguration configuration) {
+		WebAppConfiguration webAppConfiguration = AnnotatedElementUtils
+				.findMergedAnnotation(configuration.getTestClass(),
+						WebAppConfiguration.class);
+		return (webAppConfiguration != null) ? webAppConfiguration.value()
+				: "src/main/webapp";
+	}
+
 	private WebApplicationType getWebApplicationType(
 			MergedContextConfiguration configuration) {
 		ConfigurationPropertySource source = new MapConfigurationPropertySource(