Netflix / archaius

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

Enhancing Clarity with assume API in Precondition Checks for `validatePropertyUpdates` and `validateApiWhenRemovingChild` #713

Closed Codegass closed 5 months ago

Codegass commented 6 months ago

Fix #712

Description

This PR introduces a minor enhancement to the test cases validatePropertyUpdates and validateApiWhenRemovingChild in the codebase. By replacing assert statements used for initial state validation with JUnit5's assumeTrue, we aim to improve the clarity and robustness of our tests. This approach helps in clearly distinguishing between precondition checks and the actual test assertions, thereby ensuring that tests proceed only when their preconditions are met.

Key Changes