From 6efa98951de028971e948526d9d7ac2800834a55 Mon Sep 17 00:00:00 2001 From: Johnny Lim Date: Mon, 23 Dec 2019 13:26:46 +0900 Subject: [PATCH] Polish LdapAutoConfigurationTests See gh-19437 --- .../boot/autoconfigure/ldap/LdapAutoConfigurationTests.java | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/ldap/LdapAutoConfigurationTests.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/ldap/LdapAutoConfigurationTests.java index 7168020527..71c0503202 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/ldap/LdapAutoConfigurationTests.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/ldap/LdapAutoConfigurationTests.java @@ -29,7 +29,6 @@ import org.springframework.ldap.core.support.LdapContextSource; import org.springframework.ldap.core.support.SimpleDirContextAuthenticationStrategy; import org.springframework.ldap.pool2.factory.PoolConfig; import org.springframework.ldap.pool2.factory.PooledContextSource; -import org.springframework.test.util.ReflectionTestUtils; import static org.assertj.core.api.Assertions.assertThat; import static org.mockito.Mockito.mock; @@ -122,7 +121,7 @@ class LdapAutoConfigurationTests { this.contextRunner.withUserConfiguration(CustomDirContextAuthenticationStrategy.class).run((context) -> { assertThat(context).hasSingleBean(DirContextAuthenticationStrategy.class); LdapContextSource contextSource = context.getBean(LdapContextSource.class); - assertThat(ReflectionTestUtils.getField(contextSource, "authenticationStrategy")) + assertThat(contextSource).extracting("authenticationStrategy") .isSameAs(context.getBean("customDirContextAuthenticationStrategy")); }); } @@ -134,7 +133,7 @@ class LdapAutoConfigurationTests { assertThat(context).hasBean("customDirContextAuthenticationStrategy") .hasBean("anotherCustomDirContextAuthenticationStrategy"); LdapContextSource contextSource = context.getBean(LdapContextSource.class); - assertThat(ReflectionTestUtils.getField(contextSource, "authenticationStrategy")) + assertThat(contextSource).extracting("authenticationStrategy") .isNotSameAs(context.getBean("customDirContextAuthenticationStrategy")) .isNotSameAs(context.getBean("anotherCustomDirContextAuthenticationStrategy")) .isInstanceOf(SimpleDirContextAuthenticationStrategy.class);