Fix constructor-bound properties rebinding

Adjust `ConfigurationPropertiesBeanContext` to respect constructor binding during property initialization and rebinding. Instead of recalculating a bind constructor per nested type, it now consistently uses the context-level constructor and re-initializes bean properties when constructor binding is active, avoiding incorrect per-property rebinding behavior. The related test class was also updated to consistently reference instance fields via `this` for clarity.

Author: mercyblitz

microsphere-spring-boot-core/src/main/java/io/microsphere/spring/boot/context/properties/bind/ConfigurationPropertiesBeanContext.java

@@ -210,7 +210,7 @@ private void initBeanProperties() {
     private void initBeanProperties(ResolvableType beanType, ConfigurationPropertyName prefixName, String nestedPath) {
         Class<?> beanClass = beanType.getRawClass();
         if (isCandidateClass(beanClass)) {
-            Constructor<?> bindConstructor = getBindConstructor(beanType);
+            Constructor<?> bindConstructor = this.bindConstructor;
             if (bindConstructor == null) {
                 PropertyDescriptor[] descriptors = getPropertyDescriptors(beanClass);
                 initBeanProperties(beanType, descriptors, prefixName, nestedPath);
@@ -227,7 +227,11 @@ private void initBeanProperties(ResolvableType beanType, ConfigurationPropertyNa
      * @see Binder#bind(String, Bindable)
      */
     void bindPropertyValues() {
-        this.beanProperties.values().forEach(this::bindPropertyValue);
+        if (this.bindConstructor == null) {
+            this.beanProperties.values().forEach(this::bindPropertyValue);
+        } else {
+            initBeanProperties();
+        }
     }
 
     boolean bindPropertyValue(ConfigurationPropertiesBeanProperty beanProperty) {
@@ -519,8 +523,7 @@ static Object clone(Object value) {
         return value;
     }
 
-    static Map<String, ConfigurationPropertiesBeanContext> buildConfigurationPropertiesBeanContexts
-            (ConfigurableApplicationContext context) {
+    static Map<String, ConfigurationPropertiesBeanContext> buildConfigurationPropertiesBeanContexts(ConfigurableApplicationContext context) {
         ConfigurableListableBeanFactory beanFactory = context.getBeanFactory();
         String[] beanDefinitionNames = beanFactory.getBeanDefinitionNames();
         Map<String, ConfigurationPropertiesBeanContext> beanContexts = newHashMap();

microsphere-spring-boot-core/src/test/java/io/microsphere/spring/boot/context/properties/bind/EventPublishingConfigurationPropertiesBeanPropertyChangedListenerTest.java

@@ -153,16 +153,16 @@ void testWebProperties(int index) {
         }
 
         // assert the configured values
-        assertEquals(US, webProperties.getLocale());
-        assertEquals(FIXED, webProperties.getLocaleResolver());
+        assertEquals(US, this.webProperties.getLocale());
+        assertEquals(FIXED, this.webProperties.getLocaleResolver());
 
-        WebProperties.Resources resources = webProperties.getResources();
+        WebProperties.Resources resources = this.webProperties.getResources();
         String[] staticLocations = resources.getStaticLocations();
         assertArrayEquals(ofArray("/static/", "/public/", "/resources/"), staticLocations);
 
         this.eventHolder.reset();
 
-        setProperty("spring.web.locale", "zh_CN", webProperties);
+        setProperty("spring.web.locale", "zh_CN", this.webProperties);
 
         ConfigurationPropertiesBeanPropertyChangedEvent event = this.eventHolder.getValue();
         assertSame(this.webProperties, event.getSource());
@@ -180,14 +180,14 @@ void testTestConfigurationProperties(int index) {
         }
 
         Map<String, String> properties = ofMap("key-1", "value-1", "key-2", "value-2", "key-3", "value-3");
-        assertEquals("test-name", testConfigurationProperties.getName());
-        assertEquals(properties, testConfigurationProperties.getProperties());
-        assertArrayEquals(ofArray("a", "b", "c"), testConfigurationProperties.getAliases());
-        assertEquals(ofList(7070, 8080, 9090), testConfigurationProperties.getPorts());
+        assertEquals("test-name", this.testConfigurationProperties.getName());
+        assertEquals(properties, this.testConfigurationProperties.getProperties());
+        assertArrayEquals(ofArray("a", "b", "c"), this.testConfigurationProperties.getAliases());
+        assertEquals(ofList(7070, 8080, 9090), this.testConfigurationProperties.getPorts());
 
         this.eventHolder.reset();
 
-        setProperty("test.properties.key-1", "value-x", testConfigurationProperties);
+        setProperty("test.properties.key-1", "value-x", this.testConfigurationProperties);
 
         ConfigurationPropertiesBeanPropertyChangedEvent event = this.eventHolder.getValue();
         assertSame(this.testConfigurationProperties, event.getSource());