amirdeljouyi opened a new pull request, #8151: URL: https://github.com/apache/incubator-seata/pull/8151
# test: add regression tests for FileConfiguration - [x] I have read the [CONTRIBUTING.md](https://github.com/apache/incubator-seata/blob/2.x/CONTRIBUTING.md) guidelines. - [ ] I have registered the PR [changes](https://github.com/apache/incubator-seata/tree/2.x/changes). ### Ⅰ. Describe what this PR did I added three regression tests for the file-backed configuration API. The tests cover reads, default-value behavior for missing keys, and mutation operation outcomes. Impact on coverage: The tests cover the main file-backed read and mutation paths in [`FileConfiguration`](https://github.com/seata/seata/blob/2.x/config/seata-config-core/src/main/java/org/apache/seata/config/FileConfiguration.java#L109-L247), plus timeout handling in the config operation runnable. In the focused JaCoCo run, `FileConfiguration.java` branch coverage increases around 3%. ### Ⅱ. Does this pull request fix one issue? No linked issue; this is a test-only regression-coverage change. ### Ⅲ. Why don't you add test cases (unit test/integration test)? This PR consists entirely of unit tests. ### Ⅳ. Describe how to verify it Run `./mvnw -pl config/seata-config-core -am -Dtest=FileConfigurationTest -DfailIfNoTests=false test`. ### Ⅴ. Special notes for reviews No production code or API behavior is changed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
