From c31c1a4616649d04943c49c47d3915748d9c3aae Mon Sep 17 00:00:00 2001 From: Joe Grandja Date: Wed, 14 Feb 2018 12:46:24 -0500 Subject: [PATCH] AbstractConfiguredSecurityBuilderTests -> remove use of reflection Issue gh-4939 --- .../web/AbstractConfiguredSecurityBuilderTests.java | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/config/src/test/java/org/springframework/security/config/annotation/web/AbstractConfiguredSecurityBuilderTests.java b/config/src/test/java/org/springframework/security/config/annotation/web/AbstractConfiguredSecurityBuilderTests.java index dadf5fb8d5..86a59feb7c 100644 --- a/config/src/test/java/org/springframework/security/config/annotation/web/AbstractConfiguredSecurityBuilderTests.java +++ b/config/src/test/java/org/springframework/security/config/annotation/web/AbstractConfiguredSecurityBuilderTests.java @@ -21,10 +21,8 @@ import org.springframework.security.config.annotation.AbstractConfiguredSecurity import org.springframework.security.config.annotation.ObjectPostProcessor; import org.springframework.security.config.annotation.SecurityConfigurer; import org.springframework.security.config.annotation.SecurityConfigurerAdapter; -import org.springframework.test.util.ReflectionTestUtils; import java.util.List; -import java.util.Map; import static org.assertj.core.api.Assertions.assertThat; import static org.mockito.Mockito.mock; @@ -57,7 +55,7 @@ public class AbstractConfiguredSecurityBuilderTests { public void applyWhenDuplicateConfigurerAddedThenDuplicateConfigurerRemoved() throws Exception { this.builder.apply(new TestSecurityConfigurer()); this.builder.apply(new TestSecurityConfigurer()); - assertThat((Map) ReflectionTestUtils.getField(this.builder, "configurers")).hasSize(1); + assertThat(this.builder.getConfigurers(TestSecurityConfigurer.class)).hasSize(1); } @Test(expected = IllegalStateException.class)