From 3e0e4e81b9fc63b18e3804715f579d7418449f56 Mon Sep 17 00:00:00 2001 From: Stephane Nicoll Date: Thu, 2 Mar 2017 14:31:31 +0100 Subject: [PATCH] Polish --- .../cache/CacheAutoConfigurationTests.java | 17 +++++------------ 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/cache/CacheAutoConfigurationTests.java b/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/cache/CacheAutoConfigurationTests.java index 14b8141a85..1432592157 100644 --- a/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/cache/CacheAutoConfigurationTests.java +++ b/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/cache/CacheAutoConfigurationTests.java @@ -441,7 +441,7 @@ public class CacheAutoConfigurationTests { cacheManager.getCache("defaultCache"); assertThat(cacheManager.getCacheNames()).containsOnly("defaultCache"); assertThat(this.context.getBean(HazelcastInstance.class)) - .isEqualTo(getHazelcastInstance(cacheManager)); + .isEqualTo(cacheManager.getHazelcastInstance()); } @Test @@ -458,7 +458,7 @@ public class CacheAutoConfigurationTests { .getBean(HazelcastInstance.class); HazelcastCacheManager cacheManager = validateCacheManager( HazelcastCacheManager.class); - HazelcastInstance actual = getHazelcastInstance(cacheManager); + HazelcastInstance actual = cacheManager.getHazelcastInstance(); assertThat(actual).isSameAs(hazelcastInstance); assertThat(actual.getConfig().getConfigurationUrl()) .isEqualTo(new ClassPathResource( @@ -481,7 +481,7 @@ public class CacheAutoConfigurationTests { load(HazelcastCustomHazelcastInstance.class, "spring.cache.type=hazelcast"); HazelcastCacheManager cacheManager = validateCacheManager( HazelcastCacheManager.class); - assertThat(getHazelcastInstance(cacheManager)) + assertThat(cacheManager.getHazelcastInstance()) .isEqualTo(this.context.getBean("customHazelcastInstance")); } @@ -499,7 +499,7 @@ public class CacheAutoConfigurationTests { HazelcastCacheManager.class); HazelcastInstance hazelcastInstance = this.context .getBean(HazelcastInstance.class); - assertThat(getHazelcastInstance(cacheManager)).isEqualTo(hazelcastInstance); + assertThat(cacheManager.getHazelcastInstance()).isEqualTo(hazelcastInstance); assertThat(hazelcastInstance.getConfig().getConfigurationFile()) .isEqualTo(new ClassPathResource(mainConfig).getFile()); } @@ -522,8 +522,7 @@ public class CacheAutoConfigurationTests { .getBean(HazelcastInstance.class); HazelcastCacheManager cacheManager = validateCacheManager( HazelcastCacheManager.class); - HazelcastInstance cacheHazelcastInstance = (HazelcastInstance) new DirectFieldAccessor( - cacheManager).getPropertyValue("hazelcastInstance"); + HazelcastInstance cacheHazelcastInstance = cacheManager.getHazelcastInstance(); assertThat(cacheHazelcastInstance).isNotEqualTo(hazelcastInstance); // Our custom assertThat(hazelcastInstance.getConfig().getConfigurationFile()) .isEqualTo(new ClassPathResource(mainConfig).getFile()); @@ -760,12 +759,6 @@ public class CacheAutoConfigurationTests { this.context = applicationContext; } - private static HazelcastInstance getHazelcastInstance( - HazelcastCacheManager cacheManager) { - return (HazelcastInstance) new DirectFieldAccessor(cacheManager) - .getPropertyValue("hazelcastInstance"); - } - @Configuration static class EmptyConfiguration {