Merge branch '3.0.x' into 3.1.x

Closes gh-36160
pull/36168/head
Phillip Webb 1 year ago
commit dcde1fac7e

@ -385,7 +385,7 @@ public class ConfigurationPropertiesReportEndpoint implements ApplicationContext
} }
private Object stringifyIfNecessary(Object value) { private Object stringifyIfNecessary(Object value) {
if (value == null || value.getClass().isPrimitive()) { if (value == null || ClassUtils.isPrimitiveOrWrapper(value.getClass()) || value instanceof String) {
return value; return value;
} }
if (CharSequence.class.isAssignableFrom(value.getClass())) { if (CharSequence.class.isAssignableFrom(value.getClass())) {

@ -46,6 +46,9 @@ import org.springframework.boot.test.context.runner.ContextConsumer;
import org.springframework.context.ConfigurableApplicationContext; import org.springframework.context.ConfigurableApplicationContext;
import org.springframework.context.annotation.Bean; import org.springframework.context.annotation.Bean;
import org.springframework.context.annotation.Configuration; import org.springframework.context.annotation.Configuration;
import org.springframework.core.env.ConfigurableEnvironment;
import org.springframework.core.env.MapPropertySource;
import org.springframework.core.env.PropertySource;
import org.springframework.mock.env.MockPropertySource; import org.springframework.mock.env.MockPropertySource;
import org.springframework.util.unit.DataSize; import org.springframework.util.unit.DataSize;
@ -150,6 +153,21 @@ class ConfigurationPropertiesReportEndpointTests {
.isEqualTo(Duration.ofSeconds(10).toString()))); .isEqualTo(Duration.ofSeconds(10).toString())));
} }
@Test // gh-36076
void descriptorWithWrapperProperty() {
this.contextRunner.withUserConfiguration(TestPropertiesConfiguration.class).withInitializer((context) -> {
ConfigurableEnvironment environment = context.getEnvironment();
Map<String, Object> map = Collections.singletonMap("test.wrapper", 10);
PropertySource<?> propertySource = new MapPropertySource("test", map);
environment.getPropertySources().addLast(propertySource);
})
.run(assertProperties("test", (properties) -> assertThat(properties.get("wrapper")).isEqualTo(10),
(inputs) -> {
Map<String, Object> wrapper = (Map<String, Object>) inputs.get("wrapper");
assertThat(wrapper.get("value")).isEqualTo(10);
}));
}
@Test @Test
void descriptorWithNonCamelCaseProperty() { void descriptorWithNonCamelCaseProperty() {
this.contextRunner.withUserConfiguration(MixedCasePropertiesConfiguration.class) this.contextRunner.withUserConfiguration(MixedCasePropertiesConfiguration.class)
@ -452,6 +470,8 @@ class ConfigurationPropertiesReportEndpointTests {
private final String ignored = "dummy"; private final String ignored = "dummy";
private Integer wrapper;
public String getDbPassword() { public String getDbPassword() {
return this.dbPassword; return this.dbPassword;
} }
@ -488,6 +508,14 @@ class ConfigurationPropertiesReportEndpointTests {
return this.ignored; return this.ignored;
} }
public Integer getWrapper() {
return this.wrapper;
}
public void setWrapper(Integer wrapper) {
this.wrapper = wrapper;
}
} }
@Configuration(proxyBeanMethods = false) @Configuration(proxyBeanMethods = false)

Loading…
Cancel
Save