Fix #12448 Flaky testHoldContent

Fixed the HttpTester usage to avoid discarding send response content that arrived with first read
jetty · Oct 31, 2024 · 1e4eb61 · 1e4eb61
1 parent b9cdfd3
commit 1e4eb61
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/jetty-core/jetty-server/src/test/java/org/eclipse/jetty/server/HttpServerTestBase.java b/jetty-core/jetty-server/src/test/java/org/eclipse/jetty/server/HttpServerTestBase.java
@@ -1980,13 +1980,13 @@ public void onClosed(Connection connection)
             }
             else
             {
-                HttpTester.Response response = HttpTester.parseResponse(client.getInputStream());
+                HttpTester.Input input = HttpTester.from(client.getInputStream());
+                HttpTester.Response response = HttpTester.parseResponse(input);
                 assertNotNull(response);
                 assertThat(response.getStatus(), is(200));
-
                 if (pipeline)
                 {
-                    response = HttpTester.parseResponse(client.getInputStream());
+                    response = HttpTester.parseResponse(input);
                     assertNotNull(response);
                     assertThat(response.getStatus(), is(200));
                 }