From c826310fd1bb69dcc7df151a1992f39219565276 Mon Sep 17 00:00:00 2001 From: Andy Wilkinson Date: Wed, 9 May 2018 08:44:15 +0100 Subject: [PATCH] Polish --- ...onfigurationWithLog4j2AndLogbackTests.java | 5 ++- .../HazelcastCacheMeterBinderProvider.java | 5 +-- .../orm/jpa/HibernateJpaConfiguration.java | 9 ++--- .../HibernateJpaAutoConfigurationTests.java | 34 +++++++++++-------- 4 files changed, 27 insertions(+), 26 deletions(-) diff --git a/spring-boot-project/spring-boot-actuator-autoconfigure/src/test/java/org/springframework/boot/actuate/autoconfigure/metrics/MetricsAutoConfigurationWithLog4j2AndLogbackTests.java b/spring-boot-project/spring-boot-actuator-autoconfigure/src/test/java/org/springframework/boot/actuate/autoconfigure/metrics/MetricsAutoConfigurationWithLog4j2AndLogbackTests.java index f73061fc49c..4c385063af5 100644 --- a/spring-boot-project/spring-boot-actuator-autoconfigure/src/test/java/org/springframework/boot/actuate/autoconfigure/metrics/MetricsAutoConfigurationWithLog4j2AndLogbackTests.java +++ b/spring-boot-project/spring-boot-actuator-autoconfigure/src/test/java/org/springframework/boot/actuate/autoconfigure/metrics/MetricsAutoConfigurationWithLog4j2AndLogbackTests.java @@ -43,9 +43,8 @@ public class MetricsAutoConfigurationWithLog4j2AndLogbackTests { @Test public void doesNotConfigureLogbackMetrics() { - this.contextRunner.run((context) -> { - assertThat(context).doesNotHaveBean(LogbackMetrics.class); - }); + this.contextRunner.run( + (context) -> assertThat(context).doesNotHaveBean(LogbackMetrics.class)); } } diff --git a/spring-boot-project/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/metrics/cache/HazelcastCacheMeterBinderProvider.java b/spring-boot-project/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/metrics/cache/HazelcastCacheMeterBinderProvider.java index a3aa5560041..809478d2488 100644 --- a/spring-boot-project/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/metrics/cache/HazelcastCacheMeterBinderProvider.java +++ b/spring-boot-project/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/metrics/cache/HazelcastCacheMeterBinderProvider.java @@ -16,7 +16,6 @@ package org.springframework.boot.actuate.metrics.cache; -import com.hazelcast.core.IMap; import com.hazelcast.spring.cache.HazelcastCache; import io.micrometer.core.instrument.Tag; import io.micrometer.core.instrument.binder.MeterBinder; @@ -32,10 +31,8 @@ public class HazelcastCacheMeterBinderProvider implements CacheMeterBinderProvider { @Override - @SuppressWarnings("unchecked") public MeterBinder getMeterBinder(HazelcastCache cache, Iterable tags) { - return new HazelcastCacheMetrics((IMap) cache.getNativeCache(), - tags); + return new HazelcastCacheMetrics(cache.getNativeCache(), tags); } } diff --git a/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/orm/jpa/HibernateJpaConfiguration.java b/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/orm/jpa/HibernateJpaConfiguration.java index d234631853d..6273a398322 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/orm/jpa/HibernateJpaConfiguration.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/orm/jpa/HibernateJpaConfiguration.java @@ -112,7 +112,8 @@ class HibernateJpaConfiguration extends JpaBaseConfiguration { ImplicitNamingStrategy implicitNamingStrategy, List hibernatePropertiesCustomizers) { if (physicalNamingStrategy != null || implicitNamingStrategy != null) { - LinkedList customizers = new LinkedList<>(hibernatePropertiesCustomizers); + LinkedList customizers = new LinkedList<>( + hibernatePropertiesCustomizers); customizers.addFirst(new NamingStrategiesHibernatePropertiesCustomizer( physicalNamingStrategy, implicitNamingStrategy)); return customizers; @@ -129,9 +130,9 @@ class HibernateJpaConfiguration extends JpaBaseConfiguration { protected Map getVendorProperties() { Supplier defaultDdlMode = () -> this.defaultDdlAutoProvider .getDefaultDdlAuto(getDataSource()); - return new LinkedHashMap<>(getProperties() - .getHibernateProperties(new HibernateSettings().ddlAuto(defaultDdlMode) - .hibernatePropertiesCustomizers( + return new LinkedHashMap<>( + getProperties().getHibernateProperties(new HibernateSettings() + .ddlAuto(defaultDdlMode).hibernatePropertiesCustomizers( this.hibernatePropertiesCustomizers))); } diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/orm/jpa/HibernateJpaAutoConfigurationTests.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/orm/jpa/HibernateJpaAutoConfigurationTests.java index 97943718f4f..27034bfaafa 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/orm/jpa/HibernateJpaAutoConfigurationTests.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/orm/jpa/HibernateJpaAutoConfigurationTests.java @@ -278,10 +278,11 @@ public class HibernateJpaAutoConfigurationTests contextRunner() .withUserConfiguration(TestPhysicalNamingStrategyConfiguration.class) .run((context) -> { - Map hibernateProperties = context.getBean( - HibernateJpaConfiguration.class).getVendorProperties(); - assertThat(hibernateProperties).contains( - entry("hibernate.physical_naming_strategy", + Map hibernateProperties = context + .getBean(HibernateJpaConfiguration.class) + .getVendorProperties(); + assertThat(hibernateProperties) + .contains(entry("hibernate.physical_naming_strategy", context.getBean("testPhysicalNamingStrategy"))); assertThat(hibernateProperties) .doesNotContainKeys("hibernate.ejb.naming_strategy"); @@ -293,10 +294,11 @@ public class HibernateJpaAutoConfigurationTests contextRunner() .withUserConfiguration(TestImplicitNamingStrategyConfiguration.class) .run((context) -> { - Map hibernateProperties = context.getBean( - HibernateJpaConfiguration.class).getVendorProperties(); - assertThat(hibernateProperties).contains( - entry("hibernate.implicit_naming_strategy", + Map hibernateProperties = context + .getBean(HibernateJpaConfiguration.class) + .getVendorProperties(); + assertThat(hibernateProperties) + .contains(entry("hibernate.implicit_naming_strategy", context.getBean("testImplicitNamingStrategy"))); assertThat(hibernateProperties) .doesNotContainKeys("hibernate.ejb.naming_strategy"); @@ -312,8 +314,9 @@ public class HibernateJpaAutoConfigurationTests "spring.jpa.hibernate.naming.physical-strategy:com.example.Physical", "spring.jpa.hibernate.naming.implicit-strategy:com.example.Implicit") .run((context) -> { - Map hibernateProperties = context.getBean( - HibernateJpaConfiguration.class).getVendorProperties(); + Map hibernateProperties = context + .getBean(HibernateJpaConfiguration.class) + .getVendorProperties(); assertThat(hibernateProperties).contains( entry("hibernate.physical_naming_strategy", context.getBean("testPhysicalNamingStrategy")), @@ -335,10 +338,12 @@ public class HibernateJpaAutoConfigurationTests "spring.jpa.hibernate.naming.physical-strategy:com.example.Physical", "spring.jpa.hibernate.naming.implicit-strategy:com.example.Implicit") .run((context) -> { - Map hibernateProperties = context.getBean( - HibernateJpaConfiguration.class).getVendorProperties(); - TestHibernatePropertiesCustomizerConfiguration configuration = context.getBean( - TestHibernatePropertiesCustomizerConfiguration.class); + Map hibernateProperties = context + .getBean(HibernateJpaConfiguration.class) + .getVendorProperties(); + TestHibernatePropertiesCustomizerConfiguration configuration = context + .getBean( + TestHibernatePropertiesCustomizerConfiguration.class); assertThat(hibernateProperties).contains( entry("hibernate.physical_naming_strategy", configuration.physicalNamingStrategy), @@ -349,7 +354,6 @@ public class HibernateJpaAutoConfigurationTests }); } - @Configuration @TestAutoConfigurationPackage(City.class) static class TestInitializedJpaConfiguration {