SONAR-22039 Support new timeout properties

2 files changed, 10 insertions, 1 deletions

diff --git a/sonar-ws/src/test/java/org/sonarqube/ws/client/HttpConnectorTest.java b/sonar-ws/src/test/java/org/sonarqube/ws/client/HttpConnectorTest.java
index bbc9c0e127d..981d0eb145d 100644
--- a/sonar-ws/src/test/java/org/sonarqube/ws/client/HttpConnectorTest.java
+++ b/sonar-ws/src/test/java/org/sonarqube/ws/client/HttpConnectorTest.java
@@ -107,7 +107,7 @@ public class HttpConnectorTest {
     server.enqueue(new MockResponse().setResponseCode(200).addHeader("Content-Encoding", "gzip")
       .setBody(gzip("potentially a body with 100 GB of data normally encoded in gzip")));
 
-    //by default we dont accept gzip
+    // by default we dont accept gzip
     underTest = HttpConnector.newBuilder().url(serverUrl).build();
     GetRequest request = new GetRequest("rest/api/1.0/repos");
 
@@ -319,10 +319,12 @@ public class HttpConnectorTest {
       .url(serverUrl)
       .readTimeoutMilliseconds(42)
       .connectTimeoutMilliseconds(74)
+      .responseTimeoutMilliseconds(53)
       .build();
 
     assertThat(underTest.okHttpClient().readTimeoutMillis()).isEqualTo(42);
     assertThat(underTest.okHttpClient().connectTimeoutMillis()).isEqualTo(74);
+    assertThat(underTest.okHttpClient().callTimeoutMillis()).isEqualTo(53);
   }
 
   @Test
diff --git a/sonar-ws/src/test/java/org/sonarqube/ws/client/OkHttpClientBuilderTest.java b/sonar-ws/src/test/java/org/sonarqube/ws/client/OkHttpClientBuilderTest.java
index 7881f554f92..4daae827e66 100644
--- a/sonar-ws/src/test/java/org/sonarqube/ws/client/OkHttpClientBuilderTest.java
+++ b/sonar-ws/src/test/java/org/sonarqube/ws/client/OkHttpClientBuilderTest.java
@@ -83,4 +83,11 @@ public class OkHttpClientBuilderTest {
       .isInstanceOf(IllegalArgumentException.class)
       .hasMessage("Read timeout must be positive. Got -10");
   }
+
+  @Test
+  public void build_throws_IAE_if_response_timeout_is_negative() {
+    assertThatThrownBy(() -> underTest.setResponseTimeoutMs(-10))
+      .isInstanceOf(IllegalArgumentException.class)
+      .hasMessage("Response timeout must be positive. Got -10");
+  }
 }