virajjasani commented on PR #6197: URL: https://github.com/apache/hadoop/pull/6197#issuecomment-2167335766
My bad, i misunderstood this earlier. The recommendation was to use `assertj-core` jar location to test test ClassUtil functions (rather than using log4j jar location), i was under the impression that the recommendation was to use assertj for all assertions in the test class. Latest revision: using assertj-core jar location to test ClassUtil functions, so that even if log4j implementation changes, the tests should not be impacted. -- 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]
