diff --git a/.github/workflows/maven.yml b/.github/workflows/maven.yml index efced2ade..f9c048e4a 100644 --- a/.github/workflows/maven.yml +++ b/.github/workflows/maven.yml @@ -5,9 +5,9 @@ name: Build on: push: - branches: [ main ] + branches: [ main, 4.1.x ] pull_request: - branches: [ main ] + branches: [ main, 4.1.x ] jobs: build: diff --git a/spring-cloud-netflix-eureka-client-tls-tests/src/test/java/org/springframework/cloud/netflix/eureka/BaseCertTests.java b/spring-cloud-netflix-eureka-client-tls-tests/src/test/java/org/springframework/cloud/netflix/eureka/BaseCertTests.java index 1355c2d7c..c5f98b271 100644 --- a/spring-cloud-netflix-eureka-client-tls-tests/src/test/java/org/springframework/cloud/netflix/eureka/BaseCertTests.java +++ b/spring-cloud-netflix-eureka-client-tls-tests/src/test/java/org/springframework/cloud/netflix/eureka/BaseCertTests.java @@ -158,14 +158,6 @@ abstract class BaseCertTests { Assertions.assertThrows(BeanCreationException.class, client::start); } - @Test - void nonExistKeyStoreCauseFailure() { - EurekaClientRunner client = createEurekaClient(); - enableTlsClient(client); - client.setKeyStore(new File("nonExistFile")); - Assertions.assertThrows(BeanCreationException.class, client::start); - } - @Test void wrongTrustStoreCannotWork() { try (EurekaClientRunner client = createEurekaClient()) { 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..732edac4d 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