test (kubernetes-client-api) : Add test in AbstractHttpLoggingInterce…

…ptor to not consume response bytes (#5251)

Add a unit test in AbstractHttpLoggingInterceptor to verify response
bytes are not consumed after they've been processed by
HttpLoggingInterceptor.

I've verified that test fails when I revert #5250
in httpclient-okhttp and httpclient-vertx modules

Signed-off-by: Rohan Kumar <[email protected]>

kubernetes-client-api/src/test/java/io/fabric8/kubernetes/client/http/AbstractHttpLoggingInterceptorTest.java

@@ -43,6 +43,7 @@
 import java.util.Queue;
 import java.util.concurrent.TimeUnit;
 
+import static org.assertj.core.api.AssertionsForClassTypes.assertThat;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.ArgumentMatchers.anyInt;
 import static org.mockito.ArgumentMatchers.anyString;
@@ -181,6 +182,18 @@ public void httpResponseBodyLogged() throws Exception {
     inOrder.verify(logger).trace("-HTTP END-");
   }
 
+  @Test
+  @DisplayName("Interceptor doesn't consume response bytes")
+  public void responseBodyIsNotConsumed() throws Exception {
+    server.expect().withPath("/response-body")
+        .andReturn(200, "This is the response body")
+        .always();
+    HttpResponse<String> httpResponse = httpClient.sendAsync(httpClient.newHttpRequestBuilder()
+        .uri(server.url("/response-body"))
+        .build(), String.class).get(10, TimeUnit.SECONDS);
+    assertThat(httpResponse.bodyString()).isEqualTo("This is the response body");
+  }
+
   @Test
   @DisplayName("HTTP large response body is logged")
   public void httpLargeResponseBodyLogged() throws Exception {