Merge pull request #168 from rostilos/1.5.5-rc

test: Enhance VCS client error handling tests with improved timeout a…

Author: rostilos

java-ecosystem/libs/vcs-client/src/it/java/org/rostilos/codecrow/vcsclient/VcsClientErrorHandlingIT.java

@@ -46,8 +46,9 @@ void shouldTimeoutOnSlowResponse() {
 
         String url = mockServer.url("/slow-endpoint").toString();
         assertThatThrownBy(() -> {
-            try (okhttp3.Response ignored = httpClient.newCall(
+            try (okhttp3.Response response = httpClient.newCall(
                     new okhttp3.Request.Builder().url(url).build()).execute()) {
+                response.body().string(); // Force reading body to trigger read timeout
             }
         }).isInstanceOf(SocketTimeoutException.class);
     }
@@ -107,7 +108,7 @@ void shouldHandle502() throws Exception {
     void shouldHandleConnectionClose() throws Exception {
         mockServer.enqueue(new MockResponse()
                 .setResponseCode(200)
-                .setBody("{")
+                .setBody("x".repeat(1024 * 1024))
                 .setSocketPolicy(okhttp3.mockwebserver.SocketPolicy.DISCONNECT_DURING_RESPONSE_BODY));
 
         String url = mockServer.url("/repos/owner/repo").toString();

java-ecosystem/target/failsafe-reports/failsafe-summary.xml

@@ -0,0 +1,8 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<failsafe-summary xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="https://maven.apache.org/surefire/maven-surefire-plugin/xsd/failsafe-summary.xsd" result="254" timeout="false">
+    <completed>0</completed>
+    <errors>0</errors>
+    <failures>0</failures>
+    <skipped>0</skipped>
+    <failureMessage xsi:nil="true" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"/>
+</failsafe-summary>