From bc04c220c0c81f240149e2ee6c5af7fff6fb6f54 Mon Sep 17 00:00:00 2001 From: Julien HENRY Date: Fri, 12 Apr 2024 18:52:43 +0200 Subject: SONAR-22039 Support new timeout properties --- .../src/test/java/org/sonarqube/ws/client/HttpConnectorTest.java | 4 +++- .../test/java/org/sonarqube/ws/client/OkHttpClientBuilderTest.java | 7 +++++++ 2 files changed, 10 insertions(+), 1 deletion(-) (limited to 'sonar-ws/src/test/java/org/sonarqube/ws/client') 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"); + } } -- cgit v1.2.3