From baf923c988ce3e124db2723b85376a77280fef8d Mon Sep 17 00:00:00 2001 From: Arjen Poutsma Date: Wed, 10 Jan 2024 14:09:40 +0100 Subject: [PATCH] Upgrade to jetty-reactive-httpclient:4.0.2 Closes gh-31931 --- framework-platform/framework-platform.gradle | 2 +- .../function/client/WebClientIntegrationTests.java | 14 +++++++------- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/framework-platform/framework-platform.gradle b/framework-platform/framework-platform.gradle index 7bd51175e650..742d54a674a3 100644 --- a/framework-platform/framework-platform.gradle +++ b/framework-platform/framework-platform.gradle @@ -115,7 +115,7 @@ dependencies { api("org.codehaus.jettison:jettison:1.5.4") api("org.crac:crac:1.4.0") api("org.dom4j:dom4j:2.1.4") - api("org.eclipse.jetty:jetty-reactive-httpclient:4.0.1") + api("org.eclipse.jetty:jetty-reactive-httpclient:4.0.2") api("org.eclipse.persistence:org.eclipse.persistence.jpa:3.0.4") api("org.eclipse:yasson:2.0.4") api("org.ehcache:ehcache:3.10.8") diff --git a/spring-webflux/src/test/java/org/springframework/web/reactive/function/client/WebClientIntegrationTests.java b/spring-webflux/src/test/java/org/springframework/web/reactive/function/client/WebClientIntegrationTests.java index c5e2c219b63d..85f9156ba7b2 100644 --- a/spring-webflux/src/test/java/org/springframework/web/reactive/function/client/WebClientIntegrationTests.java +++ b/spring-webflux/src/test/java/org/springframework/web/reactive/function/client/WebClientIntegrationTests.java @@ -481,7 +481,7 @@ void retrieveJsonNull(ClientHttpConnector connector) { .retrieve() .bodyToMono(Map.class); - StepVerifier.create(result).verifyComplete(); + StepVerifier.create(result).expectComplete().verify(Duration.ofSeconds(3)); } @ParameterizedWebClientTest // SPR-15946 @@ -842,7 +842,7 @@ void statusHandlerWithErrorBodyTransformation(ClientHttpConnector connector) { MyException error = (MyException) throwable; assertThat(error.getMessage()).isEqualTo("foofoo"); }) - .verify(); + .verify(Duration.ofSeconds(3)); } @ParameterizedWebClientTest @@ -884,7 +884,7 @@ void statusHandlerSuppressedErrorSignal(ClientHttpConnector connector) { StepVerifier.create(result) .expectNext("Internal Server error") - .verifyComplete(); + .expectComplete().verify(Duration.ofSeconds(3)); expectRequestCount(1); expectRequest(request -> { @@ -914,7 +914,7 @@ void statusHandlerSuppressedErrorSignalWithFlux(ClientHttpConnector connector) { StepVerifier.create(result) .expectNext("Internal Server error") - .verifyComplete(); + .expectComplete().verify(Duration.ofSeconds(3)); expectRequestCount(1); expectRequest(request -> { @@ -1072,7 +1072,7 @@ void exchangeForEmptyBodyAsVoidEntity(ClientHttpConnector connector) { StepVerifier.create(result) .assertNext(r -> assertThat(r.getStatusCode().is2xxSuccessful()).isTrue()) - .verifyComplete(); + .expectComplete().verify(Duration.ofSeconds(3)); } @ParameterizedWebClientTest @@ -1241,7 +1241,7 @@ void malformedResponseChunksOnBodilessEntity(ClientHttpConnector connector) { WebClientException ex = (WebClientException) throwable; assertThat(ex.getCause()).isInstanceOf(IOException.class); }) - .verify(); + .verify(Duration.ofSeconds(3)); } @ParameterizedWebClientTest @@ -1253,7 +1253,7 @@ void malformedResponseChunksOnEntityWithBody(ClientHttpConnector connector) { WebClientException ex = (WebClientException) throwable; assertThat(ex.getCause()).isInstanceOf(IOException.class); }) - .verify(); + .verify(Duration.ofSeconds(3)); } @ParameterizedWebClientTest