Set request timeout only for the read timeout test

[laurit]

Hopefully resolves https://develocity.opentelemetry.io/s/pk7tjfyd672c6/tests/task/:instrumentation:async-http-client:async-http-client-2.0:javaagent:testStableSemconv/details/io.opentelemetry.javaagent.instrumentation.asynchttpclient.v2_0.AsyncHttpClientCompletableFutureTest/highConcurrencyWithCallback()?expanded-stacktrace=WyIwIl0&top-execution=1

[Copilot]

Pull request overview

This PR fixes a flaky highConcurrencyWithCallback() test in AsyncHttpClientCompletableFutureTest by setting the request timeout only for the read-timeout test, rather than applying it globally to all requests. Previously, the shared client configured a request timeout equal to the connection timeout (5s), which could cause spurious timeouts under high concurrency. Now, a second client instance with the read timeout is used exclusively for /read-timeout requests.

Changes:

• Introduce two separate AsyncHttpClient instances: one without a request timeout (default) and one with a read timeout, selected by a getClient(URI) method based on the request path.
• Update sendRequest and sendRequestWithCallback in both AsyncHttpClientTest and AsyncHttpClientCompletableFutureTest to route through getClient(uri) for client selection.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
instrumentation/async-http-client/async-http-client-2.0/javaagent/src/test/java/…/AsyncHttpClientTest.java	Adds two client instances, a buildClient factory, a getClient(URI) method for path-based client selection, and updates configureTimeout to conditionally set the request timeout.
instrumentation/async-http-client/async-http-client-2.0/javaagent/src/test/java/…/AsyncHttpClientCompletableFutureTest.java	Updates sendRequestWithCallback to use getClient(uri) instead of the direct client field.