Skip to content

Commit

Permalink
Fix build (change param type to builder) (#7640)
Browse files Browse the repository at this point in the history
Looks like there were some merge conflicts around test code in the last
couple of PRs.
I think this will fix up the build.
  • Loading branch information
breedx-splk committed Jan 24, 2023
1 parent db6b764 commit bb092bf
Showing 1 changed file with 7 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -90,12 +90,12 @@ public void sendRequestWithCallback(
}

@Override
protected void configure(HttpClientTestOptions options) {
options.disableTestRedirects();
options.enableTestReadTimeout();
options.setUserAgent(USER_AGENT);
protected void configure(HttpClientTestOptions.Builder optionsBuilder) {
optionsBuilder.disableTestRedirects();
optionsBuilder.enableTestReadTimeout();
optionsBuilder.setUserAgent(USER_AGENT);

options.setExpectedClientSpanNameMapper(
optionsBuilder.setExpectedClientSpanNameMapper(
(uri, method) -> {
switch (uri.toString()) {
case "http:https://localhost:61/": // unopened port
Expand All @@ -107,7 +107,7 @@ protected void configure(HttpClientTestOptions options) {
}
});

options.setClientSpanErrorMapper(
optionsBuilder.setClientSpanErrorMapper(
(uri, exception) -> {
if (exception.getClass().getName().endsWith("ReactiveException")) {
// unopened port or non routable address
Expand All @@ -119,7 +119,7 @@ protected void configure(HttpClientTestOptions options) {
return exception;
});

options.setHttpAttributes(
optionsBuilder.setHttpAttributes(
uri -> {
// unopened port or non routable address
if ("http:https://localhost:61/".equals(uri.toString())
Expand Down

0 comments on commit bb092bf

Please sign in to comment.