diff options
author | Jon Marius Venstad <jonmv@users.noreply.github.com> | 2023-09-07 19:00:02 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-09-07 19:00:02 +0200 |
commit | 932b76c32e50659d4c4f3d4a7694d35c35722ec1 (patch) | |
tree | c8219bfaf39a547cf88307f4da3caa1921954408 /container-core | |
parent | df4f7d5e840ac02f21167e57497a3cc6f7bbb89c (diff) | |
parent | 4ce892bf9c569dd8bd125ca5328f959b544871cf (diff) |
Merge pull request #28439 from vespa-engine/jonmv/visit-removes-in-doc-v1
Jonmv/visit removes in doc v1
Diffstat (limited to 'container-core')
3 files changed, 9 insertions, 25 deletions
diff --git a/container-core/src/test/java/com/yahoo/container/jdisc/HttpRequestTestCase.java b/container-core/src/test/java/com/yahoo/container/jdisc/HttpRequestTestCase.java index ca3c24113ff..4a71b28d578 100644 --- a/container-core/src/test/java/com/yahoo/container/jdisc/HttpRequestTestCase.java +++ b/container-core/src/test/java/com/yahoo/container/jdisc/HttpRequestTestCase.java @@ -19,7 +19,7 @@ import com.yahoo.text.Utf8; /** * API control of HttpRequest. * - * @author <a href="mailto:steinar@yahoo-inc.com">Steinar Knutsen</a> + * @author Steinar Knutsen */ public class HttpRequestTestCase { private static final String X_RAY_YANKEE_ZULU = "x-ray yankee zulu"; diff --git a/container-core/src/test/java/com/yahoo/container/jdisc/HttpResponseTestCase.java b/container-core/src/test/java/com/yahoo/container/jdisc/HttpResponseTestCase.java index 963c98c2469..388f2e83b31 100644 --- a/container-core/src/test/java/com/yahoo/container/jdisc/HttpResponseTestCase.java +++ b/container-core/src/test/java/com/yahoo/container/jdisc/HttpResponseTestCase.java @@ -17,7 +17,7 @@ import com.yahoo.text.Utf8; /** * API test for HttpResponse. * - * @author <a href="mailto:steinar@yahoo-inc.com">Steinar Knutsen</a> + * @author Steinar Knutsen */ public class HttpResponseTestCase { diff --git a/container-core/src/test/java/com/yahoo/jdisc/http/HttpResponseTestCase.java b/container-core/src/test/java/com/yahoo/jdisc/http/HttpResponseTestCase.java index 09682e73630..2e9c43bc6c7 100644 --- a/container-core/src/test/java/com/yahoo/jdisc/http/HttpResponseTestCase.java +++ b/container-core/src/test/java/com/yahoo/jdisc/http/HttpResponseTestCase.java @@ -1,13 +1,9 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.jdisc.http; -import com.yahoo.jdisc.Container; -import com.yahoo.jdisc.Request; import com.yahoo.jdisc.Response; -import com.yahoo.jdisc.service.CurrentContainer; import org.junit.jupiter.api.Test; -import java.net.URI; import java.util.Arrays; import java.util.Collections; import java.util.List; @@ -15,7 +11,6 @@ import java.util.List; import static org.junit.jupiter.api.Assertions.*; import static org.mockito.ArgumentMatchers.any; import static org.mockito.Mockito.mock; -import static org.mockito.Mockito.when; /** * @author Simon Thoresen Hult @@ -24,7 +19,7 @@ public class HttpResponseTestCase { @Test void requireThatAccessorsWork() throws Exception { - final HttpResponse response = newResponse(6, "foo"); + final HttpResponse response = newResponse(6); assertEquals(6, response.getStatus()); assertEquals("foo", response.getMessage()); assertNull(response.getError()); @@ -83,7 +78,7 @@ public class HttpResponseTestCase { @Test void requireThatCookieHeaderCanBeEncoded() throws Exception { - final HttpResponse response = newResponse(69, "foo"); + final HttpResponse response = newResponse(69); final List<Cookie> cookies = Collections.singletonList(new Cookie("foo", "bar")); response.encodeSetCookieHeader(cookies); final List<String> headers = response.headers().get(HttpHeaders.Names.SET_COOKIE); @@ -93,7 +88,7 @@ public class HttpResponseTestCase { @Test void requireThatMultipleCookieHeadersCanBeEncoded() throws Exception { - final HttpResponse response = newResponse(69, "foo"); + final HttpResponse response = newResponse(69); final List<Cookie> cookies = Arrays.asList(new Cookie("foo", "bar"), new Cookie("baz", "cox")); response.encodeSetCookieHeader(cookies); final List<String> headers = response.headers().get(HttpHeaders.Names.SET_COOKIE); @@ -104,7 +99,7 @@ public class HttpResponseTestCase { @Test void requireThatCookieHeaderCanBeDecoded() throws Exception { - final HttpResponse response = newResponse(69, "foo"); + final HttpResponse response = newResponse(69); final List<Cookie> cookies = Collections.singletonList(new Cookie("foo", "bar")); response.encodeSetCookieHeader(cookies); assertEquals(cookies, response.decodeSetCookieHeader()); @@ -112,25 +107,14 @@ public class HttpResponseTestCase { @Test void requireThatMultipleCookieHeadersCanBeDecoded() throws Exception { - final HttpResponse response = newResponse(69, "foo"); + final HttpResponse response = newResponse(69); final List<Cookie> cookies = Arrays.asList(new Cookie("foo", "bar"), new Cookie("baz", "cox")); response.encodeSetCookieHeader(cookies); assertEquals(cookies, response.decodeSetCookieHeader()); } - private static HttpResponse newResponse(final int status, final String message) throws Exception { - final Request request = HttpRequest.newServerRequest( - mockContainer(), - new URI("http://localhost:1234/status.html"), - HttpRequest.Method.GET, - HttpRequest.Version.HTTP_1_1); - return HttpResponse.newInstance(status, message); + private static HttpResponse newResponse(final int status) { + return HttpResponse.newInstance(status, "foo"); } - private static CurrentContainer mockContainer() { - final CurrentContainer currentContainer = mock(CurrentContainer.class); - when(currentContainer.newReference(any(URI.class))).thenReturn(mock(Container.class)); - when(currentContainer.newReference(any(URI.class), any(Object.class))).thenReturn(mock(Container.class)); - return currentContainer; - } } |