Netflix / archaius

Library for configuration management API
Apache License 2.0
2.46k stars 485 forks source link

Split the PropertyTest.test into `testPropertyValuesUpdateAndEffect` and `testServiceInitializationWithDefaultProperties` #715

Closed Codegass closed 5 months ago

Codegass commented 6 months ago

Fix #714

Description

Refactored PropertyTest to split the combined test() method into two distinct tests: testServiceInitializationWithDefaultProperties and testPropertyValuesUpdateAndEffect. This change improves test clarity by isolating specific behaviors into separate tests.

Key Changes

rgallardo-netflix commented 5 months ago

Thank you for these improvements :-)