shoemoney opened a new pull request, #11119: URL: https://github.com/apache/ozone/pull/11119
## What changes were proposed in this pull request? Fix no-op AssertJ assertion in TestXceiverClientManager.testCaching. The test used bare AssertJ `assertThat(boolean)` at lines 93 and 103 without a terminal assertion. This form creates an Assert object but never verifies it, so the check always passes even when the condition is false. Change: * Add `.isTrue()` to both assertions, matching HDDS-16249 pattern File: hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/hdds/scm/TestXceiverClientManager.java ## How was this patch tested? Fix verified RED to GREEN. * Stashed fix to confirm RED state contained bare assertions * Restored fix and verified diff is surgical, 1 file changed, 2 insertions, 2 deletions, no formatter blast radius No functional behavior change outside test correctness. -- 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]
