From 24fa15bb7b37b60feca702b4620c86de3aeb8207 Mon Sep 17 00:00:00 2001 From: Spencer Gibb Date: Tue, 6 Feb 2018 15:45:10 -0500 Subject: [PATCH] polish --- pom.xml | 12 +-- .../consul/discovery/ConsulLifecycle.java | 13 ++-- .../TestConsulLifecycleConfiguration.java | 3 - .../ConsulAutoRegistration.java | 16 ++-- ...oServiceRegistrationAutoConfiguration.java | 5 +- .../ConsulLifecycleCustomizedTests.java | 3 - ...Test.java => TtlSchedulerRemoveTests.java} | 4 +- ...edulerTest.java => TtlSchedulerTests.java} | 4 +- ...trationManagementDisabledServiceTests.java | 74 ++++++++++--------- ...ServiceRegistrationWithLifecycleTests.java | 3 - 10 files changed, 65 insertions(+), 72 deletions(-) rename spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/{TtlSchedulerRemoveTest.java => TtlSchedulerRemoveTests.java} (95%) rename spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/{TtlSchedulerTest.java => TtlSchedulerTests.java} (95%) diff --git a/pom.xml b/pom.xml index a2a2f677..e4ec1272 100644 --- a/pom.xml +++ b/pom.xml @@ -14,18 +14,18 @@ org.springframework.cloud spring-cloud-build - 1.3.6.RELEASE + 1.3.6.BUILD-SNAPSHOT - 1.3.2.RELEASE - 1.3.2.BUILD-SNAPSHOT - 1.4.0.RELEASE - 1.4.0.RELEASE + 1.3.2.BUILD-SNAPSHOT + 1.3.2.BUILD-SNAPSHOT + 1.4.2.BUILD-SNAPSHOT + 1.4.3.BUILD-SNAPSHOT 1.0.3.RELEASE - Ditmars.RELEASE + Ditmars.BUILD-SNAPSHOT 2.3.1 4.5.2 4.4.5 diff --git a/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/discovery/ConsulLifecycle.java b/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/discovery/ConsulLifecycle.java index 1ee3dbb0..b855f15f 100644 --- a/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/discovery/ConsulLifecycle.java +++ b/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/discovery/ConsulLifecycle.java @@ -16,11 +16,11 @@ package org.springframework.cloud.consul.discovery; -import javax.servlet.ServletContext; - import java.util.Collections; import java.util.List; +import javax.servlet.ServletContext; + import org.springframework.beans.BeansException; import org.springframework.beans.factory.ObjectProvider; import org.springframework.boot.bind.RelaxedPropertyResolver; @@ -121,10 +121,11 @@ public class ConsulLifecycle extends AbstractDiscoveryLifecycle { } Assert.notNull(service.getPort(), "service.port has not been set"); List registrationCustomizers = Collections - .singletonList(new ConsulServletRegistrationCustomizer(servletContext)); - ConsulAutoRegistration registration = ConsulAutoRegistration.lifecycleRegistration(service.getPort(), - getServiceId(), this.autoServiceRegistrationProperties, this.properties, - getContext(), registrationCustomizers, this.ttlConfig); + . singletonList(new ConsulServletRegistrationCustomizer(servletContext)); + ConsulAutoRegistration registration = ConsulAutoRegistration + .lifecycleRegistration(service.getPort(), getServiceId(), + this.autoServiceRegistrationProperties, this.properties, + getContext(), registrationCustomizers, this.ttlConfig); if (registration.getService().getPort() == null) { // not set by properties registration.initializePort(service.getPort()); } diff --git a/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/discovery/TestConsulLifecycleConfiguration.java b/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/discovery/TestConsulLifecycleConfiguration.java index 5ecd840b..e9497414 100644 --- a/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/discovery/TestConsulLifecycleConfiguration.java +++ b/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/discovery/TestConsulLifecycleConfiguration.java @@ -41,9 +41,6 @@ public class TestConsulLifecycleConfiguration { @Autowired(required = false) private ObjectProvider servletContext; - @Autowired(required = false) - AutoServiceRegistrationProperties autoServiceRegistrationProperties; - @Bean public ConsulLifecycle consulLifecycle(ConsulClient consulClient, AutoServiceRegistrationProperties autoServiceRegistrationProperties, diff --git a/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/serviceregistry/ConsulAutoRegistration.java b/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/serviceregistry/ConsulAutoRegistration.java index d51d03db..14ce1d18 100644 --- a/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/serviceregistry/ConsulAutoRegistration.java +++ b/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/serviceregistry/ConsulAutoRegistration.java @@ -43,9 +43,9 @@ public class ConsulAutoRegistration extends ConsulRegistration { private final HeartbeatProperties heartbeatProperties; public ConsulAutoRegistration(NewService service, AutoServiceRegistrationProperties autoServiceRegistrationProperties, - ConsulDiscoveryProperties properties, ApplicationContext context, HeartbeatProperties heartbeatProperties) { + ConsulDiscoveryProperties properties, ApplicationContext context, HeartbeatProperties heartbeatProperties) { super(service, properties); - this.autoServiceRegistrationProperties = autoServiceRegistrationProperties; + this.autoServiceRegistrationProperties = autoServiceRegistrationProperties; this.context = context; this.heartbeatProperties = heartbeatProperties; } @@ -59,16 +59,16 @@ public class ConsulAutoRegistration extends ConsulRegistration { // can create a check setCheck(getService(), this.autoServiceRegistrationProperties, getProperties(), - this.context, this.heartbeatProperties); + this.context, this.heartbeatProperties); } public ConsulAutoRegistration managementRegistration() { return managementRegistration(this.autoServiceRegistrationProperties, getProperties(), - this.context, this.heartbeatProperties); + this.context, this.heartbeatProperties); } public static ConsulAutoRegistration registration(AutoServiceRegistrationProperties autoServiceRegistrationProperties, - ConsulDiscoveryProperties properties, ApplicationContext context, + ConsulDiscoveryProperties properties, ApplicationContext context, List registrationCustomizers, HeartbeatProperties heartbeatProperties) { RelaxedPropertyResolver propertyResolver = new RelaxedPropertyResolver(context.getEnvironment()); @@ -89,7 +89,7 @@ public class ConsulAutoRegistration extends ConsulRegistration { } ConsulAutoRegistration registration = new ConsulAutoRegistration(service, autoServiceRegistrationProperties, - properties, context, heartbeatProperties); + properties, context, heartbeatProperties); customize(registrationCustomizers, registration); return registration; } @@ -104,7 +104,7 @@ public class ConsulAutoRegistration extends ConsulRegistration { @Deprecated //TODO: do I need this here, or should I just copy what I need back into lifecycle? public static ConsulAutoRegistration lifecycleRegistration(Integer port, String instanceId, AutoServiceRegistrationProperties autoServiceRegistrationProperties, - ConsulDiscoveryProperties properties, ApplicationContext context, + ConsulDiscoveryProperties properties, ApplicationContext context, List registrationCustomizers, HeartbeatProperties heartbeatProperties) { RelaxedPropertyResolver propertyResolver = new RelaxedPropertyResolver(context.getEnvironment()); @@ -130,7 +130,7 @@ public class ConsulAutoRegistration extends ConsulRegistration { setCheck(service, autoServiceRegistrationProperties, properties, context, heartbeatProperties); ConsulAutoRegistration registration = new ConsulAutoRegistration(service, autoServiceRegistrationProperties, - properties, context, heartbeatProperties); + properties, context, heartbeatProperties); customize(registrationCustomizers, registration); return registration; } diff --git a/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/serviceregistry/ConsulAutoServiceRegistrationAutoConfiguration.java b/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/serviceregistry/ConsulAutoServiceRegistrationAutoConfiguration.java index 0f0eaf38..21dbc1f1 100644 --- a/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/serviceregistry/ConsulAutoServiceRegistrationAutoConfiguration.java +++ b/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/serviceregistry/ConsulAutoServiceRegistrationAutoConfiguration.java @@ -27,7 +27,6 @@ import org.springframework.boot.autoconfigure.condition.ConditionalOnBean; import org.springframework.boot.autoconfigure.condition.ConditionalOnClass; import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean; import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty; -import org.springframework.cloud.client.serviceregistry.AutoServiceRegistrationAutoConfiguration; import org.springframework.cloud.client.serviceregistry.AutoServiceRegistrationConfiguration; import org.springframework.cloud.client.serviceregistry.AutoServiceRegistrationProperties; import org.springframework.cloud.consul.ConditionalOnConsulEnabled; @@ -65,10 +64,10 @@ public class ConsulAutoServiceRegistrationAutoConfiguration { @Bean @ConditionalOnMissingBean public ConsulAutoRegistration consulRegistration(AutoServiceRegistrationProperties autoServiceRegistrationProperties, - ConsulDiscoveryProperties properties, ApplicationContext applicationContext, + ConsulDiscoveryProperties properties, ApplicationContext applicationContext, ObjectProvider> registrationCustomizers, HeartbeatProperties heartbeatProperties) { return ConsulAutoRegistration.registration(autoServiceRegistrationProperties, properties, - applicationContext, registrationCustomizers.getIfAvailable(), heartbeatProperties); + applicationContext, registrationCustomizers.getIfAvailable(), heartbeatProperties); } @Configuration diff --git a/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/ConsulLifecycleCustomizedTests.java b/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/ConsulLifecycleCustomizedTests.java index 2b53afb6..aa7c8067 100644 --- a/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/ConsulLifecycleCustomizedTests.java +++ b/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/ConsulLifecycleCustomizedTests.java @@ -24,7 +24,6 @@ import org.springframework.beans.factory.annotation.Autowired; import org.springframework.boot.autoconfigure.EnableAutoConfiguration; import org.springframework.boot.test.context.SpringBootTest; import org.springframework.cloud.client.ServiceInstance; -import org.springframework.cloud.client.discovery.EnableDiscoveryClient; import org.springframework.cloud.client.serviceregistry.AutoServiceRegistrationProperties; import org.springframework.context.annotation.Bean; import org.springframework.context.annotation.Configuration; @@ -55,8 +54,6 @@ public class ConsulLifecycleCustomizedTests { @Autowired private CustomConsulLifecycle lifecycle2; @Autowired - AutoServiceRegistrationProperties autoServiceRegistrationProperties; - @Autowired private ConsulDiscoveryProperties properties; @Test diff --git a/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/TtlSchedulerRemoveTest.java b/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/TtlSchedulerRemoveTests.java similarity index 95% rename from spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/TtlSchedulerRemoveTest.java rename to spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/TtlSchedulerRemoveTests.java index 5fb72fe3..3237f1d2 100644 --- a/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/TtlSchedulerRemoveTest.java +++ b/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/TtlSchedulerRemoveTests.java @@ -28,13 +28,13 @@ import static org.springframework.boot.test.context.SpringBootTest.WebEnvironmen * @author Stéphane Leroy */ @RunWith(SpringRunner.class) -@SpringBootTest(classes = TtlSchedulerRemoveTest.TtlSchedulerRemoveTestConfig.class, +@SpringBootTest(classes = TtlSchedulerRemoveTests.TtlSchedulerRemoveTestConfig.class, properties = { "spring.application.name=ttlSchedulerRemove", "spring.cloud.consul.discovery.instanceId=ttlSchedulerRemove-id", "spring.cloud.consul.discovery.heartbeat.enabled=true", "spring.cloud.consul.discovery.heartbeat.ttlValue=2" }, webEnvironment = RANDOM_PORT) -public class TtlSchedulerRemoveTest { +public class TtlSchedulerRemoveTests { @Autowired private ConsulClient consul; diff --git a/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/TtlSchedulerTest.java b/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/TtlSchedulerTests.java similarity index 95% rename from spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/TtlSchedulerTest.java rename to spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/TtlSchedulerTests.java index 31b8f2c4..edea9ac9 100644 --- a/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/TtlSchedulerTest.java +++ b/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/TtlSchedulerTests.java @@ -27,13 +27,13 @@ import static org.springframework.boot.test.context.SpringBootTest.WebEnvironmen * @author Stéphane Leroy */ @RunWith(SpringRunner.class) -@SpringBootTest(classes = TtlSchedulerTest.TtlSchedulerTestConfig.class, +@SpringBootTest(classes = TtlSchedulerTests.TtlSchedulerTestConfig.class, properties = { "spring.application.name=ttlScheduler", "spring.cloud.consul.discovery.instanceId=ttlScheduler-id", "spring.cloud.consul.discovery.heartbeat.enabled=true", "spring.cloud.consul.discovery.heartbeat.ttlValue=2", "management.port=0" }, webEnvironment = RANDOM_PORT) -public class TtlSchedulerTest { +public class TtlSchedulerTests { @Autowired private ConsulClient consul; diff --git a/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/serviceregistry/ConsulAutoServiceRegistrationManagementDisabledServiceTests.java b/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/serviceregistry/ConsulAutoServiceRegistrationManagementDisabledServiceTests.java index 954c78b8..029e5025 100644 --- a/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/serviceregistry/ConsulAutoServiceRegistrationManagementDisabledServiceTests.java +++ b/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/serviceregistry/ConsulAutoServiceRegistrationManagementDisabledServiceTests.java @@ -16,10 +16,8 @@ package org.springframework.cloud.consul.serviceregistry; -import com.ecwid.consul.v1.ConsulClient; -import com.ecwid.consul.v1.Response; -import com.ecwid.consul.v1.agent.model.Service; -import org.junit.Ignore; +import java.util.Map; + import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.beans.factory.annotation.Autowired; @@ -33,7 +31,9 @@ import org.springframework.context.annotation.Configuration; import org.springframework.test.context.junit4.SpringRunner; import org.springframework.util.StringUtils; -import java.util.Map; +import com.ecwid.consul.v1.ConsulClient; +import com.ecwid.consul.v1.Response; +import com.ecwid.consul.v1.agent.model.Service; import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertFalse; @@ -46,43 +46,45 @@ import static org.springframework.boot.test.context.SpringBootTest.WebEnvironmen * @author Dmitry Zhikharev (jihor) */ @RunWith(SpringRunner.class) -@SpringBootTest(classes = ConsulAutoServiceRegistrationManagementDisabledServiceTests.TestConfig.class, - properties = {"spring.application.name=myTestService-NM", - "spring.cloud.consul.discovery.instanceId=myTestService1-NM", - "spring.cloud.service-registry.auto-registration.register-management=false", - "spring.cloud.consul.discovery.managementPort=4453", - "management.port=0" }, - webEnvironment = RANDOM_PORT) +@SpringBootTest(classes = ConsulAutoServiceRegistrationManagementDisabledServiceTests.TestConfig.class, properties = { + "spring.application.name=myTestService-NM", + "spring.cloud.consul.discovery.instanceId=myTestService1-NM", + "spring.cloud.service-registry.auto-registration.register-management=false", + "spring.cloud.consul.discovery.managementPort=4453", + "management.port=0" }, webEnvironment = RANDOM_PORT) public class ConsulAutoServiceRegistrationManagementDisabledServiceTests { - @Autowired - private ConsulClient consul; + @Autowired + private ConsulClient consul; - @Autowired - private ConsulDiscoveryProperties discoveryProperties; + @Autowired + private ConsulDiscoveryProperties discoveryProperties; - @Test - public void contextLoads() { - Response> response = consul.getAgentServices(); - Map services = response.getValue(); + @Test + public void contextLoads() { + Response> response = consul.getAgentServices(); + Map services = response.getValue(); - Service mgmtService = services.get("myTestService-NM-0-management"); - assertNull("Management service was not null", mgmtService); + Service mgmtService = services.get("myTestService-NM-0-management"); + assertNull("Management service was not null", mgmtService); - Service service = services.get("myTestService1-NM"); - assertNotNull("Service was not null", service); - assertNotEquals("service port was 0", 0, service.getPort().intValue()); - assertEquals("service id was wrong", "myTestService1-NM", service.getId()); - assertEquals("service name was wrong", "myTestService-NM", service.getService()); - assertFalse("service address must not be empty", StringUtils.isEmpty(service.getAddress())); - assertEquals("service address must equals hostname from discovery properties", discoveryProperties.getHostname(), service.getAddress()); + Service service = services.get("myTestService1-NM"); + assertNotNull("Service was not null", service); + assertNotEquals("service port was 0", 0, service.getPort().intValue()); + assertEquals("service id was wrong", "myTestService1-NM", service.getId()); + assertEquals("service name was wrong", "myTestService-NM", service.getService()); + assertFalse("service address must not be empty", + StringUtils.isEmpty(service.getAddress())); + assertEquals("service address must equals hostname from discovery properties", + discoveryProperties.getHostname(), service.getAddress()); - } + } - @Configuration - @EnableAutoConfiguration - @ImportAutoConfiguration({AutoServiceRegistrationConfiguration.class, - ConsulAutoConfiguration.class, - ConsulAutoServiceRegistrationAutoConfiguration.class}) - public static class TestConfig {} + @Configuration + @EnableAutoConfiguration + @ImportAutoConfiguration({ AutoServiceRegistrationConfiguration.class, + ConsulAutoConfiguration.class, + ConsulAutoServiceRegistrationAutoConfiguration.class }) + public static class TestConfig { + } } diff --git a/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/serviceregistry/ConsulAutoServiceRegistrationWithLifecycleTests.java b/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/serviceregistry/ConsulAutoServiceRegistrationWithLifecycleTests.java index 815493ff..af74754f 100644 --- a/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/serviceregistry/ConsulAutoServiceRegistrationWithLifecycleTests.java +++ b/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/serviceregistry/ConsulAutoServiceRegistrationWithLifecycleTests.java @@ -55,9 +55,6 @@ public class ConsulAutoServiceRegistrationWithLifecycleTests { @Autowired(required = false) private ConsulServiceRegistry consulServiceRegistry; - @Autowired(required = false) - private AutoServiceRegistrationProperties autoServiceRegistrationProperties; - @Test public void contextLoads() { assertNull("consulRegistration was created by mistake", consulRegistration);