yuzhiyang1 opened a new pull request, #16300: URL: https://github.com/apache/dubbo/pull/16300
## What changed - Updated ThreadPoolStatusCheckerTest to assert pool status entries independently instead of relying on DataStore map iteration order. - Added cleanup in a finally block so executors and DataStore entries are reset even if an assertion fails. ## Why NonDex can shuffle the internal map iteration order, which makes the previous full-message equality assertion flaky even though the checker result is semantically correct. Fixes #16152 ## Validation - .\mvnw.cmd test -pl dubbo-rpc/dubbo-rpc-dubbo '-Dtest=org.apache.dubbo.rpc.protocol.dubbo.status.ThreadPoolStatusCheckerTest#test' '-Dcheckstyle.skip=true' '-Drat.skip=true' - .\mvnw.cmd edu.illinois:nondex-maven-plugin:2.2.1:nondex -pl dubbo-rpc/dubbo-rpc-dubbo '-Dtest=org.apache.dubbo.rpc.protocol.dubbo.status.ThreadPoolStatusCheckerTest#test' -DnondexRuns=10 -- 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]
