From 3cde79c505c0c936d1d4365b79073600682d1d0d Mon Sep 17 00:00:00 2001 From: Jesse Wilson Date: Mon, 1 Apr 2024 21:35:55 -0400 Subject: [PATCH] Don't test MockServer's HTTP proxy of an HTTP/2 call (#8325) This test is disabled, and it refers to a closed bug. --- .../java/okhttp3/containers/BasicProxyTest.kt | 22 ------------------- 1 file changed, 22 deletions(-) diff --git a/container-tests/src/test/java/okhttp3/containers/BasicProxyTest.kt b/container-tests/src/test/java/okhttp3/containers/BasicProxyTest.kt index 01f2b5e78690..f1dfe9e1af9c 100644 --- a/container-tests/src/test/java/okhttp3/containers/BasicProxyTest.kt +++ b/container-tests/src/test/java/okhttp3/containers/BasicProxyTest.kt @@ -15,7 +15,6 @@ import okhttp3.containers.BasicMockServerTest.Companion.MOCKSERVER_IMAGE import okhttp3.containers.BasicMockServerTest.Companion.trustMockServer import okio.buffer import okio.source -import org.junit.jupiter.api.Disabled import org.junit.jupiter.api.Test import org.mockserver.client.MockServerClient import org.mockserver.configuration.Configuration @@ -107,27 +106,6 @@ class BasicProxyTest { } } - @Test - @Disabled("https://github.com/square/okhttp/issues/8233") - fun testOkHttpSecureProxiedHttp2() { - testRequest { - val client = - OkHttpClient.Builder() - .trustMockServer() - .proxy(Proxy(Proxy.Type.HTTP, it.remoteAddress())) - .protocols(listOf(Protocol.HTTP_2, Protocol.HTTP_1_1)) - .build() - - val response = - client.newCall( - Request((mockServer.secureEndpoint + "/person?name=peter").toHttpUrl()), - ).execute() - - assertThat(response.body.string()).contains("Peter the person") - assertThat(response.protocol).isEqualTo(Protocol.HTTP_2) - } - } - @Test fun testUrlConnectionDirect() { testRequest {