diff --git a/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/autoconfigure/ShellProperties.java b/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/autoconfigure/ShellProperties.java index 91fe9665f4..d5153a71a8 100644 --- a/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/autoconfigure/ShellProperties.java +++ b/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/autoconfigure/ShellProperties.java @@ -1,5 +1,5 @@ /* - * Copyright 2012-2016 the original author or authors. + * Copyright 2012-2017 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -120,7 +120,7 @@ public class ShellProperties { } public void setDisabledCommands(String[] disabledCommands) { - Assert.notEmpty(disabledCommands); + Assert.notEmpty(disabledCommands, "disabledCommands must not be empty"); this.disabledCommands = disabledCommands; } @@ -129,7 +129,7 @@ public class ShellProperties { } public void setDisabledPlugins(String[] disabledPlugins) { - Assert.notEmpty(disabledPlugins); + Assert.notEmpty(disabledPlugins, "disabledPlugins must not be empty"); this.disabledPlugins = disabledPlugins; } diff --git a/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/metrics/statsd/StatsdMetricWriter.java b/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/metrics/statsd/StatsdMetricWriter.java index 292dd46b5b..a92dd2eae4 100644 --- a/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/metrics/statsd/StatsdMetricWriter.java +++ b/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/metrics/statsd/StatsdMetricWriter.java @@ -1,5 +1,5 @@ /* - * Copyright 2012-2016 the original author or authors. + * Copyright 2012-2017 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -72,7 +72,7 @@ public class StatsdMetricWriter implements MetricWriter, Closeable { * @param client StatsD client to write metrics with */ public StatsdMetricWriter(StatsDClient client) { - Assert.notNull(client); + Assert.notNull(client, "client must not be null"); this.client = client; } diff --git a/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/trace/WebRequestTraceFilterTests.java b/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/trace/WebRequestTraceFilterTests.java index f8d83d9f8b..56f6a8db97 100644 --- a/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/trace/WebRequestTraceFilterTests.java +++ b/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/trace/WebRequestTraceFilterTests.java @@ -244,7 +244,6 @@ public class WebRequestTraceFilterTests { } @Test - @SuppressWarnings("unchecked") public void filterAddsTimeTaken() throws Exception { MockHttpServletRequest request = spy(new MockHttpServletRequest("GET", "/foo")); MockHttpServletResponse response = new MockHttpServletResponse(); diff --git a/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/cache/HazelcastInstanceConfiguration.java b/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/cache/HazelcastInstanceConfiguration.java index 667f6e29b3..286544cc8f 100644 --- a/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/cache/HazelcastInstanceConfiguration.java +++ b/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/cache/HazelcastInstanceConfiguration.java @@ -56,6 +56,7 @@ abstract class HazelcastInstanceConfiguration { @Bean public HazelcastCacheManager cacheManager( HazelcastInstance existingHazelcastInstance) throws IOException { + @SuppressWarnings("deprecation") Resource config = this.cacheProperties.getHazelcast().getConfig(); Resource location = this.cacheProperties.resolveConfigLocation(config); if (location != null) { @@ -86,6 +87,7 @@ abstract class HazelcastInstanceConfiguration { @Bean public HazelcastInstance hazelcastInstance() throws IOException { + @SuppressWarnings("deprecation") Resource config = this.cacheProperties.getHazelcast().getConfig(); Resource location = this.cacheProperties.resolveConfigLocation(config); if (location != null) { diff --git a/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/condition/ConditionalOnMissingBeanTests.java b/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/condition/ConditionalOnMissingBeanTests.java index 89ea64bfdf..947e9e6f86 100644 --- a/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/condition/ConditionalOnMissingBeanTests.java +++ b/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/condition/ConditionalOnMissingBeanTests.java @@ -646,7 +646,7 @@ public class ConditionalOnMissingBeanTests { public static class ExampleFactoryBean implements FactoryBean { public ExampleFactoryBean(String value) { - Assert.state(!value.contains("$")); + Assert.state(!value.contains("$"), "value must not contain $"); } @Override @@ -669,7 +669,7 @@ public class ConditionalOnMissingBeanTests { public static class NonspecificFactoryBean implements FactoryBean { public NonspecificFactoryBean(String value) { - Assert.state(!value.contains("$")); + Assert.state(!value.contains("$"), "value must not contain $"); } @Override diff --git a/spring-boot/src/main/java/org/springframework/boot/bind/PropertiesConfigurationFactory.java b/spring-boot/src/main/java/org/springframework/boot/bind/PropertiesConfigurationFactory.java index 1aec0338d1..83445c9ad1 100644 --- a/spring-boot/src/main/java/org/springframework/boot/bind/PropertiesConfigurationFactory.java +++ b/spring-boot/src/main/java/org/springframework/boot/bind/PropertiesConfigurationFactory.java @@ -91,7 +91,7 @@ public class PropertiesConfigurationFactory * @see #PropertiesConfigurationFactory(Class) */ public PropertiesConfigurationFactory(T target) { - Assert.notNull(target); + Assert.notNull(target, "target must not be null"); this.target = target; } @@ -102,7 +102,7 @@ public class PropertiesConfigurationFactory */ @SuppressWarnings("unchecked") public PropertiesConfigurationFactory(Class type) { - Assert.notNull(type); + Assert.notNull(type, "type must not be null"); this.target = (T) BeanUtils.instantiate(type); } diff --git a/spring-boot/src/main/java/org/springframework/boot/bind/YamlConfigurationFactory.java b/spring-boot/src/main/java/org/springframework/boot/bind/YamlConfigurationFactory.java index 6c14fc2a70..4bc53749c8 100644 --- a/spring-boot/src/main/java/org/springframework/boot/bind/YamlConfigurationFactory.java +++ b/spring-boot/src/main/java/org/springframework/boot/bind/YamlConfigurationFactory.java @@ -76,7 +76,7 @@ public class YamlConfigurationFactory * @param type the root type */ public YamlConfigurationFactory(Class type) { - Assert.notNull(type); + Assert.notNull(type, "type must not be null"); this.type = type; } diff --git a/spring-boot/src/main/java/org/springframework/boot/context/embedded/tomcat/TomcatEmbeddedServletContainerFactory.java b/spring-boot/src/main/java/org/springframework/boot/context/embedded/tomcat/TomcatEmbeddedServletContainerFactory.java index 0d59b1de6e..4e012a862d 100644 --- a/spring-boot/src/main/java/org/springframework/boot/context/embedded/tomcat/TomcatEmbeddedServletContainerFactory.java +++ b/spring-boot/src/main/java/org/springframework/boot/context/embedded/tomcat/TomcatEmbeddedServletContainerFactory.java @@ -354,8 +354,7 @@ public class TomcatEmbeddedServletContainerFactory Compression compression = getCompression(); protocol.setCompression("on"); protocol.setCompressionMinSize(compression.getMinResponseSize()); - protocol.setCompressableMimeType( - StringUtils.arrayToCommaDelimitedString(compression.getMimeTypes())); + configureCompressibleMimeTypes(protocol, compression); if (getCompression().getExcludedUserAgents() != null) { protocol.setNoCompressionUserAgents( StringUtils.arrayToCommaDelimitedString( @@ -364,6 +363,13 @@ public class TomcatEmbeddedServletContainerFactory } } + @SuppressWarnings("deprecation") + private void configureCompressibleMimeTypes(AbstractHttp11Protocol protocol, + Compression compression) { + protocol.setCompressableMimeType( + StringUtils.arrayToCommaDelimitedString(compression.getMimeTypes())); + } + /** * Configure Tomcat's {@link AbstractHttp11JsseProtocol} for SSL. * @param protocol the protocol diff --git a/spring-boot/src/test/java/org/springframework/boot/context/embedded/CompressionTests.java b/spring-boot/src/test/java/org/springframework/boot/context/embedded/CompressionTests.java index ac5a35fc75..d608454e51 100644 --- a/spring-boot/src/test/java/org/springframework/boot/context/embedded/CompressionTests.java +++ b/spring-boot/src/test/java/org/springframework/boot/context/embedded/CompressionTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2012-2016 the original author or authors. + * Copyright 2012-2017 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -36,7 +36,7 @@ public class CompressionTests { private String[] getTomcatDefaultCompressableMimeTypes() { Http11NioProtocol protocol = new Http11NioProtocol(); - return protocol.getCompressableMimeTypes(); + return protocol.getCompressibleMimeTypes(); } }