adutra commented on code in PR #3614:
URL: https://github.com/apache/polaris/pull/3614#discussion_r2769660722
##########
runtime/service/src/test/java/org/apache/polaris/service/tracing/RequestIdHeaderTest.java:
##########
@@ -57,14 +58,12 @@ public Map<String, String> getConfigOverrides() {
private static final String CLIENT_ID = "client1";
private static final String CLIENT_SECRET = "secret1";
- private static final URI baseUri =
- URI.create(
- "http://localhost:";
- + Objects.requireNonNull(
- Integer.getInteger("quarkus.http.test-port"),
- "System property not set correctly:
quarkus.http.test-port"));
+ @SuppressWarnings("CdiInjectionPointsInspection")
+ @Inject
+ HttpServer httpServer;
private Response request(Map<String, String> headers) {
+ var baseUri = URI.create("http://localhost:"; + httpServer.getPort());
Review Comment:
Can't you inject `@TestHTTPResource URL testUrl`?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
