diff --git a/spring-core/src/main/java/org/springframework/util/ReflectionUtils.java b/spring-core/src/main/java/org/springframework/util/ReflectionUtils.java index ec29a41f6bb..61bfccf223c 100644 --- a/spring-core/src/main/java/org/springframework/util/ReflectionUtils.java +++ b/spring-core/src/main/java/org/springframework/util/ReflectionUtils.java @@ -710,8 +710,7 @@ public abstract class ReflectionUtils { // Keep backing up the inheritance hierarchy. Class targetClass = clazz; do { - Field[] fields = getDeclaredFields(targetClass); - for (Field field : fields) { + for (Field field : getDeclaredFields(targetClass)) { if (ff != null && !ff.matches(field)) { continue; } diff --git a/spring-core/src/test/java/org/springframework/util/PropertyPlaceholderHelperTests.java b/spring-core/src/test/java/org/springframework/util/PropertyPlaceholderHelperTests.java index 3b0e1650177..8f580df3c8c 100644 --- a/spring-core/src/test/java/org/springframework/util/PropertyPlaceholderHelperTests.java +++ b/spring-core/src/test/java/org/springframework/util/PropertyPlaceholderHelperTests.java @@ -17,13 +17,11 @@ package org.springframework.util; import java.util.Properties; -import java.util.stream.Stream; import org.junit.jupiter.api.Nested; import org.junit.jupiter.api.Test; import org.junit.jupiter.params.ParameterizedTest; -import org.junit.jupiter.params.provider.Arguments; -import org.junit.jupiter.params.provider.MethodSource; +import org.junit.jupiter.params.provider.CsvSource; import org.springframework.util.PropertyPlaceholderHelper.PlaceholderResolver; @@ -44,6 +42,7 @@ class PropertyPlaceholderHelperTests { private final PropertyPlaceholderHelper helper = new PropertyPlaceholderHelper("${", "}"); + @Test void withProperties() { String text = "foo=${foo}"; @@ -116,8 +115,8 @@ class PropertyPlaceholderHelperTests { props.setProperty("foo", "bar"); PropertyPlaceholderHelper helper = new PropertyPlaceholderHelper("${", "}", null, null, false); - assertThatExceptionOfType(PlaceholderResolutionException.class).isThrownBy(() -> - helper.replacePlaceholders(text, props)); + assertThatExceptionOfType(PlaceholderResolutionException.class) + .isThrownBy(() -> helper.replacePlaceholders(text, props)); } @Nested @@ -126,7 +125,14 @@ class PropertyPlaceholderHelperTests { private final PropertyPlaceholderHelper helper = new PropertyPlaceholderHelper("${", "}", ":", null, true); @ParameterizedTest(name = "{0} -> {1}") - @MethodSource("defaultValues") + @CsvSource(delimiterString = "->", textBlock = """ + ${invalid:test} -> test + ${invalid:${one}} -> 1 + ${invalid:${one}${two}} -> 12 + ${invalid:${one}:${two}} -> 1:2 + ${invalid:${also_invalid:test}} -> test + ${invalid:${also_invalid:${one}}} -> 1 + """) void defaultValueIsApplied(String text, String value) { Properties properties = new Properties(); properties.setProperty("one", "1"); @@ -142,24 +148,26 @@ class PropertyPlaceholderHelperTests { verify(resolver, never()).resolvePlaceholder("two"); } - static Stream defaultValues() { - return Stream.of( - Arguments.of("${invalid:test}", "test"), - Arguments.of("${invalid:${one}}", "1"), - Arguments.of("${invalid:${one}${two}}", "12"), - Arguments.of("${invalid:${one}:${two}}", "1:2"), - Arguments.of("${invalid:${also_invalid:test}}", "test"), - Arguments.of("${invalid:${also_invalid:${one}}}", "1") - ); + @ParameterizedTest(name = "{0} -> {1}") + @CsvSource(delimiterString = "->", textBlock = """ + ${prefix://my-service} -> example-service + ${p1} -> example-service + """) + void placeholdersWithExactMatchAreConsidered(String text, String expected) { + Properties properties = new Properties(); + properties.setProperty("prefix://my-service", "example-service"); + properties.setProperty("px", "prefix"); + properties.setProperty("p1", "${prefix://my-service}"); + assertThat(this.helper.replacePlaceholders(text, properties)).isEqualTo(expected); } - } - PlaceholderResolver mockPlaceholderResolver(String... pairs) { + + private static PlaceholderResolver mockPlaceholderResolver(String... pairs) { if (pairs.length % 2 == 1) { throw new IllegalArgumentException("size must be even, it is a set of key=value pairs"); } - PlaceholderResolver resolver = mock(PlaceholderResolver.class); + PlaceholderResolver resolver = mock(); for (int i = 0; i < pairs.length; i += 2) { String key = pairs[i]; String value = pairs[i + 1]; @@ -168,5 +176,4 @@ class PropertyPlaceholderHelperTests { return resolver; } - } diff --git a/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/AbstractMockitoBeanOverrideHandler.java b/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/AbstractMockitoBeanOverrideHandler.java index 90550736ffb..7e1b1b347c7 100644 --- a/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/AbstractMockitoBeanOverrideHandler.java +++ b/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/AbstractMockitoBeanOverrideHandler.java @@ -57,20 +57,20 @@ abstract class AbstractMockitoBeanOverrideHandler extends BeanOverrideHandler { @Override protected void trackOverrideInstance(Object mock, SingletonBeanRegistry trackingBeanRegistry) { - getMockitoBeans(trackingBeanRegistry).add(mock); + getMockBeans(trackingBeanRegistry).add(mock); } - private static MockitoBeans getMockitoBeans(SingletonBeanRegistry trackingBeanRegistry) { - String beanName = MockitoBeans.class.getName(); - MockitoBeans mockitoBeans = null; + private static MockBeans getMockBeans(SingletonBeanRegistry trackingBeanRegistry) { + String beanName = MockBeans.class.getName(); + MockBeans mockBeans = null; if (trackingBeanRegistry.containsSingleton(beanName)) { - mockitoBeans = (MockitoBeans) trackingBeanRegistry.getSingleton(beanName); + mockBeans = (MockBeans) trackingBeanRegistry.getSingleton(beanName); } - if (mockitoBeans == null) { - mockitoBeans = new MockitoBeans(); - trackingBeanRegistry.registerSingleton(beanName, mockitoBeans); + if (mockBeans == null) { + mockBeans = new MockBeans(); + trackingBeanRegistry.registerSingleton(beanName, mockBeans); } - return mockitoBeans; + return mockBeans; } @Override diff --git a/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoBeans.java b/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockBeans.java similarity index 98% rename from spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoBeans.java rename to spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockBeans.java index 1c636f629d2..d57c3fe411a 100644 --- a/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoBeans.java +++ b/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockBeans.java @@ -28,7 +28,7 @@ import org.mockito.Mockito; * @author Sam Brannen * @since 6.2 */ -class MockitoBeans { +class MockBeans { private final List beans = new ArrayList<>(); diff --git a/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoBeanOverrideHandler.java b/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoBeanOverrideHandler.java index 26e9d66c435..773ec7a8976 100644 --- a/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoBeanOverrideHandler.java +++ b/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoBeanOverrideHandler.java @@ -64,13 +64,13 @@ class MockitoBeanOverrideHandler extends AbstractMockitoBeanOverrideHandler { } private MockitoBeanOverrideHandler(Field field, ResolvableType typeToMock, @Nullable String beanName, - BeanOverrideStrategy strategy, MockReset reset, Class[] extraInterfaces, @Nullable Answers answers, + BeanOverrideStrategy strategy, MockReset reset, Class[] extraInterfaces, Answers answers, boolean serializable) { super(field, typeToMock, beanName, strategy, reset); Assert.notNull(typeToMock, "'typeToMock' must not be null"); this.extraInterfaces = asClassSet(extraInterfaces); - this.answers = (answers != null ? answers : Answers.RETURNS_DEFAULTS); + this.answers = answers; this.serializable = serializable; } diff --git a/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoBeanOverrideProcessor.java b/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoBeanOverrideProcessor.java index fd4e3d389b4..21e2bd42973 100644 --- a/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoBeanOverrideProcessor.java +++ b/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoBeanOverrideProcessor.java @@ -35,8 +35,8 @@ class MockitoBeanOverrideProcessor implements BeanOverrideProcessor { @Override public AbstractMockitoBeanOverrideHandler createHandler(Annotation overrideAnnotation, Class testClass, Field field) { - if (overrideAnnotation instanceof MockitoBean mockBean) { - return new MockitoBeanOverrideHandler(field, ResolvableType.forField(field, testClass), mockBean); + if (overrideAnnotation instanceof MockitoBean mockitoBean) { + return new MockitoBeanOverrideHandler(field, ResolvableType.forField(field, testClass), mockitoBean); } else if (overrideAnnotation instanceof MockitoSpyBean spyBean) { return new MockitoSpyBeanOverrideHandler(field, ResolvableType.forField(field, testClass), spyBean); diff --git a/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoResetTestExecutionListener.java b/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoResetTestExecutionListener.java index 094440ccc6f..4a986f7fd57 100644 --- a/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoResetTestExecutionListener.java +++ b/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoResetTestExecutionListener.java @@ -115,7 +115,7 @@ public class MockitoResetTestExecutionListener extends AbstractTestExecutionList } } try { - beanFactory.getBean(MockitoBeans.class).resetAll(reset); + beanFactory.getBean(MockBeans.class).resetAll(reset); } catch (NoSuchBeanDefinitionException ex) { // Continue diff --git a/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoSpyBeanOverrideHandler.java b/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoSpyBeanOverrideHandler.java index 6f2bd2fdb88..043581c0da9 100644 --- a/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoSpyBeanOverrideHandler.java +++ b/spring-test/src/main/java/org/springframework/test/context/bean/override/mockito/MockitoSpyBeanOverrideHandler.java @@ -48,15 +48,9 @@ class MockitoSpyBeanOverrideHandler extends AbstractMockitoBeanOverrideHandler { new SpringAopBypassingVerificationStartedListener(); - MockitoSpyBeanOverrideHandler(Field field, ResolvableType typeToSpy, MockitoSpyBean spyAnnotation) { - this(field, typeToSpy, (StringUtils.hasText(spyAnnotation.name()) ? spyAnnotation.name() : null), - spyAnnotation.reset()); - } - - MockitoSpyBeanOverrideHandler(Field field, ResolvableType typeToSpy, @Nullable String beanName, - MockReset reset) { - - super(field, typeToSpy, beanName, BeanOverrideStrategy.WRAP, reset); + MockitoSpyBeanOverrideHandler(Field field, ResolvableType typeToSpy, MockitoSpyBean spyBean) { + super(field, typeToSpy, (StringUtils.hasText(spyBean.name()) ? spyBean.name() : null), + BeanOverrideStrategy.WRAP, spyBean.reset()); Assert.notNull(typeToSpy, "typeToSpy must not be null"); } diff --git a/spring-test/src/test/java/org/springframework/test/context/bean/override/convention/TestBeanTests.java b/spring-test/src/test/java/org/springframework/test/context/bean/override/convention/TestBeanTests.java index 9685ed07179..8dfbbba78ee 100644 --- a/spring-test/src/test/java/org/springframework/test/context/bean/override/convention/TestBeanTests.java +++ b/spring-test/src/test/java/org/springframework/test/context/bean/override/convention/TestBeanTests.java @@ -113,7 +113,7 @@ public class TestBeanTests { } @Test - void contextCustomizerCannotBeCreatedWitCompetingOverrideMethods() { + void contextCustomizerCannotBeCreatedWithCompetingOverrideMethods() { GenericApplicationContext context = new GenericApplicationContext(); context.registerBean("bean", String.class, () -> "example"); assertThatIllegalStateException()