From 543e45fc879135cdeca9ef4d35b3f590d4404c25 Mon Sep 17 00:00:00 2001 From: Olga Maciaszek-Sharma Date: Wed, 9 Oct 2024 15:32:07 +0200 Subject: [PATCH] Fix test setup. --- .../cloud/netflix/eureka/EurekaClientTests.java | 17 +++++++++++++++++ .../eureka/RestTemplateEurekaClientTests.java | 9 +++++++++ 2 files changed, 26 insertions(+) diff --git a/spring-cloud-netflix-eureka-client-tls-tests/src/test/java/org/springframework/cloud/netflix/eureka/EurekaClientTests.java b/spring-cloud-netflix-eureka-client-tls-tests/src/test/java/org/springframework/cloud/netflix/eureka/EurekaClientTests.java index 0ba9ca90d..44af58dd2 100644 --- a/spring-cloud-netflix-eureka-client-tls-tests/src/test/java/org/springframework/cloud/netflix/eureka/EurekaClientTests.java +++ b/spring-cloud-netflix-eureka-client-tls-tests/src/test/java/org/springframework/cloud/netflix/eureka/EurekaClientTests.java @@ -22,8 +22,12 @@ import org.junit.jupiter.api.BeforeAll; import org.springframework.boot.SpringBootConfiguration; import org.springframework.boot.autoconfigure.EnableAutoConfiguration; +import org.springframework.cloud.netflix.eureka.config.DiscoveryClientOptionalArgsConfiguration; +import org.springframework.cloud.netflix.eureka.http.DefaultEurekaClientHttpRequestFactorySupplier; import org.springframework.cloud.netflix.eureka.http.RestTemplateDiscoveryClientOptionalArgs; +import org.springframework.cloud.netflix.eureka.http.RestTemplateTransportClientFactories; import org.springframework.cloud.netflix.eureka.server.EnableEurekaServer; +import org.springframework.context.annotation.Bean; import static org.assertj.core.api.Assertions.assertThat; @@ -53,6 +57,19 @@ public class EurekaClientTests extends BaseCertTests { @EnableAutoConfiguration public static class TestApp { + @Bean + public RestTemplateTransportClientFactories forceRestTemplateTransportClientFactories( + DiscoveryClientOptionalArgsConfiguration configuration, + RestTemplateDiscoveryClientOptionalArgs discoveryClientOptionalArgs) { + return configuration.restTemplateTransportClientFactories(discoveryClientOptionalArgs); + } + + @Bean + public RestTemplateDiscoveryClientOptionalArgs discoveryClientOptionalArgs() { + return new RestTemplateDiscoveryClientOptionalArgs( + new DefaultEurekaClientHttpRequestFactorySupplier(new RestTemplateTimeoutProperties()), + null); } + } @SpringBootConfiguration diff --git a/spring-cloud-netflix-eureka-client-tls-tests/src/test/java/org/springframework/cloud/netflix/eureka/RestTemplateEurekaClientTests.java b/spring-cloud-netflix-eureka-client-tls-tests/src/test/java/org/springframework/cloud/netflix/eureka/RestTemplateEurekaClientTests.java index 633a08131..a9686848e 100644 --- a/spring-cloud-netflix-eureka-client-tls-tests/src/test/java/org/springframework/cloud/netflix/eureka/RestTemplateEurekaClientTests.java +++ b/spring-cloud-netflix-eureka-client-tls-tests/src/test/java/org/springframework/cloud/netflix/eureka/RestTemplateEurekaClientTests.java @@ -33,6 +33,7 @@ import org.springframework.cloud.configuration.TlsProperties; import org.springframework.cloud.netflix.eureka.config.DiscoveryClientOptionalArgsConfiguration; import org.springframework.cloud.netflix.eureka.http.EurekaClientHttpRequestFactorySupplier; import org.springframework.cloud.netflix.eureka.http.RestTemplateDiscoveryClientOptionalArgs; +import org.springframework.cloud.netflix.eureka.http.RestTemplateTransportClientFactories; import org.springframework.cloud.netflix.eureka.server.EnableEurekaServer; import org.springframework.context.annotation.Bean; @@ -82,6 +83,14 @@ public class RestTemplateEurekaClientTests extends BaseCertTests { eurekaClientHttpRequestFactorySupplier, new RestTemplateBuilderObjectProvider()); } + // Want to force reusing exactly the same bean as on production without excluding + // jersey from the classpath + @Bean + public RestTemplateTransportClientFactories forceRestTemplateTransportClientFactories( + DiscoveryClientOptionalArgsConfiguration configuration, + RestTemplateDiscoveryClientOptionalArgs discoveryClientOptionalArgs) { + return configuration.restTemplateTransportClientFactories(discoveryClientOptionalArgs); + } } @SpringBootConfiguration