Skip to content

Commit 7575d1a

Browse files
committed
remove maven local + remove legacy test
1 parent dacaf6b commit 7575d1a

File tree

1 file changed

+0
-26
lines changed

1 file changed

+0
-26
lines changed

java-client/src/test/java/co/elastic/clients/transport/rest5_client/low_level/RestClientBuilderTests.java

Lines changed: 0 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919

2020
package co.elastic.clients.transport.rest5_client.low_level;
2121

22-
import org.apache.hc.client5.http.config.RequestConfig;
2322
import org.apache.hc.client5.http.impl.async.CloseableHttpAsyncClient;
2423
import org.apache.hc.client5.http.impl.async.HttpAsyncClients;
2524
import org.apache.hc.core5.http.Header;
@@ -179,29 +178,4 @@ private static void assertSetPathPrefixThrows(final String pathPrefix) {
179178
assertThat(e.getMessage(), containsString(pathPrefix));
180179
}
181180
}
182-
183-
/**
184-
* This test verifies that we don't change the default value for the connection request timeout as that
185-
* causes problems.
186-
* See https://github.com/elastic/elasticsearch/issues/24069
187-
*/
188-
@Test
189-
public void testDefaultConnectionRequestTimeout() throws IOException {
190-
Rest5ClientBuilder builder = Rest5Client.builder(new HttpHost("localhost", 9200));
191-
192-
RequestConfig requestConfig = RequestConfig.custom().build();
193-
assertEquals(RequestConfig.DEFAULT.getConnectionRequestTimeout(),
194-
requestConfig.getConnectionRequestTimeout());
195-
// this way we get notified if the default ever changes
196-
// TODO IT CHANGED from -1 to 3 minutes, does it mean we always need to explicitly set it?
197-
//assertEquals(-1, requestConfig.getConnectionRequestTimeout());
198-
199-
CloseableHttpAsyncClient httpclient = HttpAsyncClients.custom()
200-
.setDefaultRequestConfig(requestConfig)
201-
.build();
202-
203-
try (Rest5Client restClient = builder.build()) {
204-
assertNotNull(restClient);
205-
}
206-
}
207181
}

0 commit comments

Comments
 (0)