diff --git a/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/endpoint/mvc/EnvironmentMvcEndpoint.java b/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/endpoint/mvc/EnvironmentMvcEndpoint.java index 1f101b38ec..cac00aacd3 100644 --- a/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/endpoint/mvc/EnvironmentMvcEndpoint.java +++ b/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/endpoint/mvc/EnvironmentMvcEndpoint.java @@ -93,7 +93,8 @@ public class EnvironmentMvcEndpoint extends EndpointMvcAdapter @Override protected Object getOptionalValue(Environment source, String name) { - Object result = ((EnvironmentEndpoint) getDelegate()).getResolver().getProperty(name, Object.class); + Object result = ((EnvironmentEndpoint) getDelegate()).getResolver() + .getProperty(name, Object.class); if (result != null) { result = ((EnvironmentEndpoint) getDelegate()).sanitize(name, result); } diff --git a/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/endpoint/EnvironmentEndpointTests.java b/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/endpoint/EnvironmentEndpointTests.java index f94e149d0c..f2a7ffc364 100644 --- a/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/endpoint/EnvironmentEndpointTests.java +++ b/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/endpoint/EnvironmentEndpointTests.java @@ -112,8 +112,9 @@ public class EnvironmentEndpointTests extends AbstractEndpointTests env = report.invoke(); - Map testProperties = (Map) env - .get("test"); + Map testProperties = (Map) env.get("test"); assertThat(testProperties.get("my.foo")).isEqualTo("hello"); } @@ -223,14 +223,12 @@ public class EnvironmentEndpointTests extends AbstractEndpointTests env = report.invoke(); - Map testProperties = (Map) env - .get("test"); + Map testProperties = (Map) env.get("test"); assertThat(testProperties.get("my.foo")).isEqualTo("${bar.blah}"); } @@ -244,8 +242,7 @@ public class EnvironmentEndpointTests extends AbstractEndpointTests env = report.invoke(); - Map testProperties = (Map) env - .get("test"); + Map testProperties = (Map) env.get("test"); assertThat(testProperties.get("my.foo")).isEqualTo("http://******://hello"); } @@ -259,16 +256,17 @@ public class EnvironmentEndpointTests extends AbstractEndpointTests env = report.invoke(); - Map testProperties = (Map) env - .get("test"); - assertThat(testProperties.get("my.foo")).isEqualTo("http://${bar.password}://hello"); + Map testProperties = (Map) env.get("test"); + assertThat(testProperties.get("my.foo")) + .isEqualTo("http://${bar.password}://hello"); } - @SuppressWarnings("unchecked") @Test + @SuppressWarnings("unchecked") public void propertyWithTypeOtherThanStringShouldNotFail() throws Exception { this.context = new AnnotationConfigApplicationContext(); - MutablePropertySources propertySources = this.context.getEnvironment().getPropertySources(); + MutablePropertySources propertySources = this.context.getEnvironment() + .getPropertySources(); Map source = new HashMap(); source.put("foo", Collections.singletonMap("bar", "baz")); propertySources.addFirst(new MapPropertySource("test", source)); @@ -276,8 +274,7 @@ public class EnvironmentEndpointTests extends AbstractEndpointTests env = report.invoke(); - Map testProperties = (Map) env - .get("test"); + Map testProperties = (Map) env.get("test"); Map foo = (Map) testProperties.get("foo"); assertThat(foo.get("bar")).isEqualTo("baz"); } diff --git a/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/endpoint/mvc/EnvironmentMvcEndpointTests.java b/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/endpoint/mvc/EnvironmentMvcEndpointTests.java index df621110f0..4f5433db28 100644 --- a/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/endpoint/mvc/EnvironmentMvcEndpointTests.java +++ b/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/endpoint/mvc/EnvironmentMvcEndpointTests.java @@ -161,11 +161,10 @@ public class EnvironmentMvcEndpointTests { .andExpect(content().string(containsString("\"my.foo\":\"******\""))); } - @SuppressWarnings("unchecked") @Test public void propertyWithTypeOtherThanStringShouldNotFail() throws Exception { - ConfigurableEnvironment environment = (ConfigurableEnvironment) this.context.getEnvironment(); - MutablePropertySources propertySources = environment.getPropertySources(); + MutablePropertySources propertySources = ((ConfigurableEnvironment) this.context + .getEnvironment()).getPropertySources(); Map source = new HashMap(); source.put("foo", Collections.singletonMap("bar", "baz")); propertySources.addFirst(new MapPropertySource("test", source)); diff --git a/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/amqp/RabbitAutoConfigurationTests.java b/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/amqp/RabbitAutoConfigurationTests.java index 98058a3ceb..a7aecaef3e 100644 --- a/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/amqp/RabbitAutoConfigurationTests.java +++ b/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/amqp/RabbitAutoConfigurationTests.java @@ -296,7 +296,7 @@ public class RabbitAutoConfigurationTests { @Test public void testSimpleRabbitListenerContainerFactoryWithCustomSettings() { load(new Class[] { MessageConvertersConfiguration.class, - MessageRecoverersConfiguration.class }, + MessageRecoverersConfiguration.class }, "spring.rabbitmq.listener.simple.retry.enabled:true", "spring.rabbitmq.listener.simple.retry.maxAttempts:4", "spring.rabbitmq.listener.simple.retry.initialInterval:2000", @@ -346,29 +346,31 @@ public class RabbitAutoConfigurationTests { @Test public void testRabbitListenerContainerFactoryConfigurersAreAvailable() { - load(TestConfiguration.class, - "spring.rabbitmq.listener.simple.concurrency:5", + load(TestConfiguration.class, "spring.rabbitmq.listener.simple.concurrency:5", "spring.rabbitmq.listener.simple.maxConcurrency:10", "spring.rabbitmq.listener.simple.prefetch:40", "spring.rabbitmq.listener.direct.consumers-per-queue:5", "spring.rabbitmq.listener.direct.prefetch:40"); - assertThat(this.context.getBeansOfType( - SimpleRabbitListenerContainerFactoryConfigurer.class)).hasSize(1); - assertThat(this.context.getBeansOfType( - DirectRabbitListenerContainerFactoryConfigurer.class)).hasSize(1); + assertThat(this.context + .getBeansOfType(SimpleRabbitListenerContainerFactoryConfigurer.class)) + .hasSize(1); + assertThat(this.context + .getBeansOfType(DirectRabbitListenerContainerFactoryConfigurer.class)) + .hasSize(1); } @Test public void testSimpleRabbitListenerContainerFactoryConfigurerUsesConfig() { - load(TestConfiguration.class, - "spring.rabbitmq.listener.type:direct", // listener type is irrelevant + load(TestConfiguration.class, "spring.rabbitmq.listener.type:direct", // listener + // type is + // irrelevant "spring.rabbitmq.listener.simple.concurrency:5", "spring.rabbitmq.listener.simple.maxConcurrency:10", "spring.rabbitmq.listener.simple.prefetch:40"); SimpleRabbitListenerContainerFactoryConfigurer configurer = this.context .getBean(SimpleRabbitListenerContainerFactoryConfigurer.class); - SimpleRabbitListenerContainerFactory factory = - mock(SimpleRabbitListenerContainerFactory.class); + SimpleRabbitListenerContainerFactory factory = mock( + SimpleRabbitListenerContainerFactory.class); configurer.configure(factory, mock(ConnectionFactory.class)); verify(factory).setConcurrentConsumers(5); verify(factory).setMaxConcurrentConsumers(10); @@ -377,14 +379,15 @@ public class RabbitAutoConfigurationTests { @Test public void testDirectRabbitListenerContainerFactoryConfigurerUsesConfig() { - load(TestConfiguration.class, - "spring.rabbitmq.listener.type:simple", // listener type is irrelevant + load(TestConfiguration.class, "spring.rabbitmq.listener.type:simple", // listener + // type is + // irrelevant "spring.rabbitmq.listener.direct.consumers-per-queue:5", "spring.rabbitmq.listener.direct.prefetch:40"); DirectRabbitListenerContainerFactoryConfigurer configurer = this.context .getBean(DirectRabbitListenerContainerFactoryConfigurer.class); - DirectRabbitListenerContainerFactory factory = - mock(DirectRabbitListenerContainerFactory.class); + DirectRabbitListenerContainerFactory factory = mock( + DirectRabbitListenerContainerFactory.class); configurer.configure(factory, mock(ConnectionFactory.class)); verify(factory).setConsumersPerQueue(5); verify(factory).setPrefetchCount(40);