diff --git a/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/embedded/tomcat/TomcatReactiveWebServerFactoryTests.java b/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/embedded/tomcat/TomcatReactiveWebServerFactoryTests.java index 48e3b2fe10..3333ae1f18 100644 --- a/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/embedded/tomcat/TomcatReactiveWebServerFactoryTests.java +++ b/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/embedded/tomcat/TomcatReactiveWebServerFactoryTests.java @@ -24,16 +24,14 @@ import org.apache.catalina.LifecycleListener; import org.apache.catalina.connector.Connector; import org.apache.catalina.core.AprLifecycleListener; import org.junit.Test; +import org.mockito.ArgumentCaptor; import org.mockito.InOrder; -import org.mockito.invocation.InvocationOnMock; -import org.mockito.stubbing.Answer; import org.springframework.boot.web.reactive.server.AbstractReactiveWebServerFactoryTests; import org.springframework.http.server.reactive.HttpHandler; import static org.assertj.core.api.Assertions.assertThat; import static org.mockito.ArgumentMatchers.any; -import static org.mockito.BDDMockito.willAnswer; import static org.mockito.Mockito.inOrder; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.verify; @@ -69,22 +67,14 @@ public class TomcatReactiveWebServerFactoryTests } @Test - public void contextIsAddedToHostBeforeCustomizersAreCalled() throws Exception { + public void contextIsAddedToHostBeforeCustomizersAreCalled() { TomcatReactiveWebServerFactory factory = getFactory(); TomcatContextCustomizer customizer = mock(TomcatContextCustomizer.class); - willAnswer(new Answer() { - - @Override - public Void answer(InvocationOnMock invocation) throws Throwable { - assertThat(((Context) invocation.getArguments()[0]).getParent()) - .isNotNull(); - return null; - } - - }).given(customizer).customize(any(Context.class)); factory.addContextCustomizers(customizer); this.webServer = factory.getWebServer(mock(HttpHandler.class)); - verify(customizer).customize(any(Context.class)); + ArgumentCaptor contextCaptor = ArgumentCaptor.forClass(Context.class); + verify(customizer).customize(contextCaptor.capture()); + assertThat(contextCaptor.getValue().getParent()).isNotNull(); } @Test diff --git a/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/embedded/tomcat/TomcatServletWebServerFactoryTests.java b/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/embedded/tomcat/TomcatServletWebServerFactoryTests.java index a533f8cc27..049e38c4c3 100644 --- a/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/embedded/tomcat/TomcatServletWebServerFactoryTests.java +++ b/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/embedded/tomcat/TomcatServletWebServerFactoryTests.java @@ -49,9 +49,8 @@ import org.apache.tomcat.JarScanFilter; import org.junit.After; import org.junit.Rule; import org.junit.Test; +import org.mockito.ArgumentCaptor; import org.mockito.InOrder; -import org.mockito.invocation.InvocationOnMock; -import org.mockito.stubbing.Answer; import org.springframework.boot.testsupport.rule.OutputCapture; import org.springframework.boot.web.server.WebServerException; @@ -63,7 +62,6 @@ import static org.assertj.core.api.Assertions.assertThat; import static org.junit.Assert.fail; import static org.mockito.ArgumentMatchers.any; import static org.mockito.BDDMockito.given; -import static org.mockito.BDDMockito.willAnswer; import static org.mockito.Mockito.inOrder; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.verify; @@ -147,22 +145,14 @@ public class TomcatServletWebServerFactoryTests } @Test - public void contextIsAddedToHostBeforeCustomizersAreCalled() throws Exception { + public void contextIsAddedToHostBeforeCustomizersAreCalled() { TomcatServletWebServerFactory factory = getFactory(); TomcatContextCustomizer customizer = mock(TomcatContextCustomizer.class); - willAnswer(new Answer() { - - @Override - public Void answer(InvocationOnMock invocation) throws Throwable { - assertThat(((Context) invocation.getArguments()[0]).getParent()) - .isNotNull(); - return null; - } - - }).given(customizer).customize(any(Context.class)); factory.addContextCustomizers(customizer); this.webServer = factory.getWebServer(); - verify(customizer).customize(any(Context.class)); + ArgumentCaptor contextCaptor = ArgumentCaptor.forClass(Context.class); + verify(customizer).customize(contextCaptor.capture()); + assertThat(contextCaptor.getValue().getParent()).isNotNull(); } @Test