From cc6caaa0c482105b522e87e15f20dfd05afdd890 Mon Sep 17 00:00:00 2001 From: dongyinuo Date: Thu, 24 Nov 2022 21:30:57 +0800 Subject: [PATCH] fix review --- .../config/properties/PolarisMetadataRouterPropertiesTest.java | 3 ++- .../config/properties/PolarisNearByRouterPropertiesTest.java | 3 ++- .../properties/PolarisRuleBasedRouterPropertiesTest.java | 3 ++- .../router/resttemplate/PolarisLoadBalancerRequestTests.java | 1 - 4 files changed, 6 insertions(+), 4 deletions(-) diff --git a/spring-cloud-starter-tencent-polaris-router/src/test/java/com/tencent/cloud/polaris/router/config/properties/PolarisMetadataRouterPropertiesTest.java b/spring-cloud-starter-tencent-polaris-router/src/test/java/com/tencent/cloud/polaris/router/config/properties/PolarisMetadataRouterPropertiesTest.java index 22bfa4504..d5f2fbec9 100644 --- a/spring-cloud-starter-tencent-polaris-router/src/test/java/com/tencent/cloud/polaris/router/config/properties/PolarisMetadataRouterPropertiesTest.java +++ b/spring-cloud-starter-tencent-polaris-router/src/test/java/com/tencent/cloud/polaris/router/config/properties/PolarisMetadataRouterPropertiesTest.java @@ -48,6 +48,7 @@ public class PolarisMetadataRouterPropertiesTest { @Test public void testToString() { - System.out.println(properties); + assertThat(properties.toString()) + .isEqualTo("PolarisMetadataRouterProperties{enabled=true}"); } } diff --git a/spring-cloud-starter-tencent-polaris-router/src/test/java/com/tencent/cloud/polaris/router/config/properties/PolarisNearByRouterPropertiesTest.java b/spring-cloud-starter-tencent-polaris-router/src/test/java/com/tencent/cloud/polaris/router/config/properties/PolarisNearByRouterPropertiesTest.java index 3bf0b1e3c..35c98adff 100644 --- a/spring-cloud-starter-tencent-polaris-router/src/test/java/com/tencent/cloud/polaris/router/config/properties/PolarisNearByRouterPropertiesTest.java +++ b/spring-cloud-starter-tencent-polaris-router/src/test/java/com/tencent/cloud/polaris/router/config/properties/PolarisNearByRouterPropertiesTest.java @@ -50,6 +50,7 @@ public class PolarisNearByRouterPropertiesTest { @Test public void testToString() { - System.out.println(properties); + assertThat(properties.toString()) + .isEqualTo("PolarisNearByRouterProperties{enabled=true}"); } } diff --git a/spring-cloud-starter-tencent-polaris-router/src/test/java/com/tencent/cloud/polaris/router/config/properties/PolarisRuleBasedRouterPropertiesTest.java b/spring-cloud-starter-tencent-polaris-router/src/test/java/com/tencent/cloud/polaris/router/config/properties/PolarisRuleBasedRouterPropertiesTest.java index ae081b1e3..6312803dd 100644 --- a/spring-cloud-starter-tencent-polaris-router/src/test/java/com/tencent/cloud/polaris/router/config/properties/PolarisRuleBasedRouterPropertiesTest.java +++ b/spring-cloud-starter-tencent-polaris-router/src/test/java/com/tencent/cloud/polaris/router/config/properties/PolarisRuleBasedRouterPropertiesTest.java @@ -48,7 +48,8 @@ public class PolarisRuleBasedRouterPropertiesTest { @Test public void testToString() { - System.out.println(properties); + assertThat(properties.toString()) + .isEqualTo("PolarisNearByRouterProperties{enabled=true}"); } } diff --git a/spring-cloud-starter-tencent-polaris-router/src/test/java/com/tencent/cloud/polaris/router/resttemplate/PolarisLoadBalancerRequestTests.java b/spring-cloud-starter-tencent-polaris-router/src/test/java/com/tencent/cloud/polaris/router/resttemplate/PolarisLoadBalancerRequestTests.java index 32bdac014..1e36d6f12 100644 --- a/spring-cloud-starter-tencent-polaris-router/src/test/java/com/tencent/cloud/polaris/router/resttemplate/PolarisLoadBalancerRequestTests.java +++ b/spring-cloud-starter-tencent-polaris-router/src/test/java/com/tencent/cloud/polaris/router/resttemplate/PolarisLoadBalancerRequestTests.java @@ -52,7 +52,6 @@ public class PolarisLoadBalancerRequestTests { @Override public Object apply(ServiceInstance instance) throws Exception { - System.out.println("Mock LoadBalancerRequest"); return instance; } }