Always record original default values

quarkusio · May 8, 2024 · 3877f16 · 3877f16
1 parent 4331897
commit 3877f16
Show file tree

Hide file tree

Showing 6 changed files with 36 additions and 5 deletions.
diff --git a/...yment/src/main/java/io/quarkus/deployment/configuration/BuildTimeConfigurationReader.java b/...yment/src/main/java/io/quarkus/deployment/configuration/BuildTimeConfigurationReader.java
@@ -1211,12 +1211,12 @@ private static void getDefaults(
                 ClassDefinition.ItemMember itemMember = (ClassDefinition.ItemMember) member;
                 String defaultValue = itemMember.getDefaultValue();
                 if (defaultValue != null) {
-                    // lookup config to make sure we catch relocates or fallbacks
+                    // lookup config to make sure we catch relocates or fallbacks and override the value
                     ConfigValue configValue = config.getConfigValue(propertyName.toString());
-                    if (configValue.getValue() != null) {
-                        defaultValues.put(configValue.getName(), configValue.getValue());
+                    if (configValue.getValue() != null && !configValue.getName().equals(propertyName.toString())) {
+                        defaultValues.put(propertyName.toString(), configValue.getValue());
                     } else {
-                        defaultValues.put(configValue.getName(), defaultValue);
+                        defaultValues.put(propertyName.toString(), defaultValue);
                     }
                 }
             }

diff --git a/core/deployment/src/main/java/io/quarkus/deployment/steps/ConfigGenerationBuildStep.java b/core/deployment/src/main/java/io/quarkus/deployment/steps/ConfigGenerationBuildStep.java
@@ -220,7 +220,10 @@ void generateBuilders(
             // Runtime values may contain active profiled names that override sames names in defaults
             // We need to keep the original name definition in case a different profile is used to run the app
             String activeName = ProfileConfigSourceInterceptor.activeName(entry.getKey(), profiles);
-            defaultValues.remove(activeName);
+            // But keep the default
+            if (!configItem.getReadResult().getRunTimeDefaultValues().containsKey(activeName)) {
+                defaultValues.remove(activeName);
+            }
             defaultValues.put(entry.getKey(), entry.getValue());
         }
         defaultValues.putAll(configItem.getReadResult().getRunTimeValues());

diff --git a/...ation-tests/test-extension/extension/deployment/src/main/resources/application.properties b/...ation-tests/test-extension/extension/deployment/src/main/resources/application.properties
@@ -155,6 +155,8 @@ quarkus.arc.unremovable-types[0]=foo
 ### recording
 bt.ok.to.record=from-app
 %test.bt.profile.record=properties
+%test.quarkus.mapping.rt.record-default=from-app
+%test.quarkus.rt.record-default=from-app
 
 ### mappings
 quarkus.mapping.bt.value=value

diff --git a/...t-extension/extension/deployment/src/test/java/io/quarkus/extest/RuntimeDefaultsTest.java b/...t-extension/extension/deployment/src/test/java/io/quarkus/extest/RuntimeDefaultsTest.java
@@ -53,6 +53,19 @@ void doNotRecordActiveUnprofiledPropertiesDefaults() {
         assertNull(defaultValues.get().getValue("bt.profile.record"));
     }
 
+    @Test
+    void recordDefaultFromRootEvenIfInActiveProfile() {
+        Optional<ConfigSource> defaultValues = config.getConfigSource("DefaultValuesConfigSource");
+        assertTrue(defaultValues.isPresent());
+
+        // Old Roots
+        assertEquals("from-app", defaultValues.get().getValue("%test.quarkus.rt.record-default"));
+        assertEquals("from-default", defaultValues.get().getValue("quarkus.rt.record-default"));
+        // Mappings
+        assertEquals("from-app", defaultValues.get().getValue("%test.quarkus.mapping.rt.record-default"));
+        assertEquals("from-default", defaultValues.get().getValue("quarkus.mapping.rt.record-default"));
+    }
+
     @Test
     void recordProfile() {
         Optional<ConfigSource> defaultValues = config.getConfigSource("DefaultValuesConfigSource");

diff --git a/.../extension/runtime/src/main/java/io/quarkus/extest/runtime/config/TestMappingRunTime.java b/.../extension/runtime/src/main/java/io/quarkus/extest/runtime/config/TestMappingRunTime.java
@@ -5,6 +5,7 @@
 import io.quarkus.runtime.annotations.ConfigPhase;
 import io.quarkus.runtime.annotations.ConfigRoot;
 import io.smallrye.config.ConfigMapping;
+import io.smallrye.config.WithDefault;
 
 @ConfigMapping(prefix = "quarkus.mapping.rt")
 @ConfigRoot(phase = ConfigPhase.RUN_TIME)
@@ -25,6 +26,12 @@ public interface TestMappingRunTime {
     /** Record values with named profile **/
     Optional<String> recordProfiled();
 
+    /**
+     * Record Default
+     */
+    @WithDefault("from-default")
+    String recordDefault();
+
     interface Group {
         /**
          * A Group value.

diff --git a/...n/extension/runtime/src/main/java/io/quarkus/extest/runtime/config/TestRunTimeConfig.java b/...n/extension/runtime/src/main/java/io/quarkus/extest/runtime/config/TestRunTimeConfig.java
@@ -113,6 +113,12 @@ public class TestRunTimeConfig {
     @ConfigItem
     public Optional<String> doNotRecord;
 
+    /**
+     * Record Default
+     */
+    @ConfigItem(defaultValue = "from-default")
+    public String recordDefault;
+
     @Override
     public String toString() {
         return "TestRunTimeConfig{" +