summaryrefslogtreecommitdiffstats
path: root/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/FilterTestCase.java
diff options
context:
space:
mode:
authorHåkon Hallingstad <hakon@verizonmedia.com>2019-10-18 15:19:59 +0200
committerHåkon Hallingstad <hakon@verizonmedia.com>2019-10-18 15:19:59 +0200
commit579754fa5b5f4768a75fc3aed08f70ffb29db0e3 (patch)
tree825e5366b8253535bb9366722006ef49c6e83c53 /jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/FilterTestCase.java
parent19ab63311c9354db919a9a9ac38d34342e1ca204 (diff)
Use mockito-core 3.1.0
Diffstat (limited to 'jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/FilterTestCase.java')
-rw-r--r--jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/FilterTestCase.java13
1 files changed, 7 insertions, 6 deletions
diff --git a/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/FilterTestCase.java b/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/FilterTestCase.java
index 7b01d5ca061..0b94829f720 100644
--- a/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/FilterTestCase.java
+++ b/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/FilterTestCase.java
@@ -39,7 +39,8 @@ import java.util.concurrent.atomic.AtomicReference;
import static org.hamcrest.CoreMatchers.is;
import static org.hamcrest.MatcherAssert.assertThat;
-import static org.mockito.Matchers.any;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.ArgumentMatchers.isNull;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.never;
import static org.mockito.Mockito.times;
@@ -284,8 +285,8 @@ public class FilterTestCase {
final HttpRequest request = null;
final ResponseHandler responseHandler = null;
requestFilterChain.filter(request, responseHandler);
- verify(requestFilter1).filter(any(HttpRequest.class), any(ResponseHandler.class));
- verify(requestFilter2).filter(any(HttpRequest.class), any(ResponseHandler.class));
+ verify(requestFilter1).filter(isNull(), any(ResponseHandler.class));
+ verify(requestFilter2).filter(isNull(), any(ResponseHandler.class));
}
@Test
@@ -299,7 +300,7 @@ public class FilterTestCase {
// Check that the filter is called with the same request argument as the chain was,
// in a manner that allows the request object to be wrapped.
final ArgumentCaptor<HttpRequest> requestCaptor = ArgumentCaptor.forClass(HttpRequest.class);
- verify(requestFilter).filter(requestCaptor.capture(), any(ResponseHandler.class));
+ verify(requestFilter).filter(requestCaptor.capture(), isNull());
verify(request, never()).getUri();
requestCaptor.getValue().getUri();
verify(request, times(1)).getUri();
@@ -316,7 +317,7 @@ public class FilterTestCase {
// Check that the filter is called with the same response handler argument as the chain was,
// in a manner that allows the handler object to be wrapped.
final ArgumentCaptor<ResponseHandler> responseHandlerCaptor = ArgumentCaptor.forClass(ResponseHandler.class);
- verify(requestFilter).filter(any(HttpRequest.class), responseHandlerCaptor.capture());
+ verify(requestFilter).filter(isNull(), responseHandlerCaptor.capture());
verify(responseHandler, never()).handleResponse(any(Response.class));
responseHandlerCaptor.getValue().handleResponse(mock(Response.class));
verify(responseHandler, times(1)).handleResponse(any(Response.class));
@@ -373,7 +374,7 @@ public class FilterTestCase {
ResponseFilterChain.newInstance(responseFilter1, responseFilter2).filter(response, request);
final ArgumentCaptor<Response> responseCaptor = ArgumentCaptor.forClass(Response.class);
- verify(responseFilter2).filter(responseCaptor.capture(), any(Request.class));
+ verify(responseFilter2).filter(responseCaptor.capture(), isNull());
assertThat(responseCaptor.getValue().getStatus(), is(statusCode));
assertThat(responseCaptor.getValue().headers().getFirst("foo"), is("bar"));