Compare commits

...
Author SHA1 Message Date
Olga Maciaszek-Sharma 1c9d37e812 Use raw types in ResponseData. 2022-02-28 16:18:57 +01:00
3 changed files with 13 additions and 11 deletions
@@ -33,7 +33,6 @@ import org.springframework.cloud.client.loadbalancer.RequestData;
import org.springframework.cloud.client.loadbalancer.ResponseData;
import org.springframework.http.HttpHeaders;
import org.springframework.http.HttpMethod;
import org.springframework.http.HttpStatus;
/**
* @author Olga Maciaszek-Sharma
@@ -73,8 +72,7 @@ final class LoadBalancerUtils {
static ResponseData buildResponseData(Response response) {
HttpHeaders responseHeaders = new HttpHeaders();
response.headers().forEach((key, value) -> responseHeaders.put(key, new ArrayList<>(value)));
return new ResponseData(HttpStatus.resolve(response.status()), responseHeaders, null,
buildRequestData(response.request()));
return new ResponseData(response.status(), responseHeaders, null, buildRequestData(response.request()));
}
static RequestData buildRequestData(Request request) {
@@ -167,11 +167,13 @@ class FeignBlockingLoadBalancerClientTests {
.get("anotherLoadBalancerLifecycle")).getCompleteLog().values();
assertThat(lifecycleLogRequests).extracting(lbRequest -> lbRequest.getContext().getHint())
.contains(callbackTestHint);
assertThat(lifecycleLogStartedRequests).extracting(lbRequest -> lbRequest.getContext().getHint())
.contains(callbackTestHint);
assertThat(lifecycleLogStartedRequests).extracting(lbRequest -> lbRequest.getContext()
.getHint())
.contains(callbackTestHint);
assertThat(anotherLifecycleLogRequests)
.extracting(completionContext -> completionContext.getClientResponse().getHttpStatus())
.contains(HttpStatus.OK);
.extracting(completionContext -> completionContext.getClientResponse()
.getHttpStatus())
.contains(HttpStatus.OK.value());
}
private String read(Response response) throws IOException {
@@ -232,11 +232,13 @@ class RetryableFeignBlockingLoadBalancerClientTests {
.get("anotherLoadBalancerLifecycle")).getCompleteLog().values();
assertThat(lifecycleLogRequests).extracting(lbRequest -> lbRequest.getContext().getHint())
.contains(callbackTestHint);
assertThat(lifecycleLogStartedRequests).extracting(lbRequest -> lbRequest.getContext().getHint())
.contains(callbackTestHint);
assertThat(lifecycleLogStartedRequests).extracting(lbRequest -> lbRequest.getContext()
.getHint())
.contains(callbackTestHint);
assertThat(anotherLifecycleLogRequests)
.extracting(completionContext -> completionContext.getClientResponse().getHttpStatus())
.contains(HttpStatus.OK);
.extracting(completionContext -> completionContext.getClientResponse()
.getHttpStatus())
.contains(HttpStatus.OK.value());
}
private Request testRequest() {