diff --git a/archaius2-core/src/test/java/com/netflix/archaius/ConfigManagerTest.java b/archaius2-core/src/test/java/com/netflix/archaius/ConfigManagerTest.java index 941d3b3b..ae529b11 100644 --- a/archaius2-core/src/test/java/com/netflix/archaius/ConfigManagerTest.java +++ b/archaius2-core/src/test/java/com/netflix/archaius/ConfigManagerTest.java @@ -15,6 +15,10 @@ */ package com.netflix.archaius; +import static org.junit.jupiter.api.Assertions.assertEquals; + +import java.util.concurrent.atomic.AtomicReference; + import com.netflix.archaius.api.Property; import com.netflix.archaius.api.PropertyFactory; import com.netflix.archaius.config.DefaultCompositeConfig; @@ -39,19 +43,26 @@ public void testBasicReplacement() throws ConfigException { .put("region", "us-east") .put("c", 123) .build())); - - + + PropertyFactory factory = DefaultPropertyFactory.from(config); Property prop = factory.getProperty("abc").asString("defaultValue"); - + + // Use an AtomicReference to capture the property value change + AtomicReference capturedValue = new AtomicReference<>(""); prop.addListener(new DefaultPropertyListener() { @Override public void onChange(String next) { - System.out.println("Configuration changed : " + next); + capturedValue.set(next); } }); - + + // Set property which should trigger the listener dyn.setProperty("abc", "${c}"); + + // Assert that the capturedValue was updated correctly + assertEquals("123", capturedValue.get(), "Configuration change did not occur as expected"); + } }