From 0ba7448bdd30b691f00fddffc07983e6a6e33508 Mon Sep 17 00:00:00 2001 From: salaboy Date: Tue, 30 Sep 2025 14:59:02 +0200 Subject: [PATCH] setting default props for dapr client Signed-off-by: salaboy --- .../client/DaprClientProperties.java | 8 ++++---- .../client/DaprClientPropertiesTest.java | 17 +++++++++++++++++ 2 files changed, 21 insertions(+), 4 deletions(-) diff --git a/dapr-spring/dapr-spring-boot-autoconfigure/src/main/java/io/dapr/spring/boot/autoconfigure/client/DaprClientProperties.java b/dapr-spring/dapr-spring-boot-autoconfigure/src/main/java/io/dapr/spring/boot/autoconfigure/client/DaprClientProperties.java index ac3971579e..6759057aa0 100644 --- a/dapr-spring/dapr-spring-boot-autoconfigure/src/main/java/io/dapr/spring/boot/autoconfigure/client/DaprClientProperties.java +++ b/dapr-spring/dapr-spring-boot-autoconfigure/src/main/java/io/dapr/spring/boot/autoconfigure/client/DaprClientProperties.java @@ -17,10 +17,10 @@ @ConfigurationProperties(prefix = "dapr.client") public class DaprClientProperties { - private String httpEndpoint; - private String grpcEndpoint; - private Integer httpPort; - private Integer grpcPort; + private String httpEndpoint = "http://localhost"; + private String grpcEndpoint = "localhost"; + private Integer httpPort = 3500; + private Integer grpcPort = 50001; private String apiToken; /** diff --git a/dapr-spring/dapr-spring-boot-autoconfigure/src/test/java/io/dapr/spring/boot/autoconfigure/client/DaprClientPropertiesTest.java b/dapr-spring/dapr-spring-boot-autoconfigure/src/test/java/io/dapr/spring/boot/autoconfigure/client/DaprClientPropertiesTest.java index 33ea73989f..9a401e7850 100644 --- a/dapr-spring/dapr-spring-boot-autoconfigure/src/test/java/io/dapr/spring/boot/autoconfigure/client/DaprClientPropertiesTest.java +++ b/dapr-spring/dapr-spring-boot-autoconfigure/src/test/java/io/dapr/spring/boot/autoconfigure/client/DaprClientPropertiesTest.java @@ -64,6 +64,21 @@ public void shouldSetDaprClientPropertiesCorrectly() { }); } + @Test + @DisplayName("Should create DaprClientProperties default values correctly") + public void shouldSetDaprClientDefaultPropertiesCorrectly() { + + DaprClientProperties properties = new DaprClientProperties(); + + SoftAssertions.assertSoftly(softAssertions -> { + softAssertions.assertThat(properties.getGrpcEndpoint()).isEqualTo("localhost"); + softAssertions.assertThat(properties.getHttpEndpoint()).isEqualTo("http://localhost"); + softAssertions.assertThat(properties.getHttpPort()).isEqualTo(3500); + softAssertions.assertThat(properties.getGrpcPort()).isEqualTo(50001); + softAssertions.assertThat(properties.getApiToken()).isNull(); + }); + } + @Test @DisplayName("Should map DaprClient properties correctly") public void shouldMapDaprClientProperties() { @@ -105,6 +120,8 @@ public void shouldMapDaprClientPropertiesCamelCase() { }); } + + @EnableConfigurationProperties(DaprClientProperties.class) static class EnableDaprClientProperties {