slfan1989 commented on PR #7350: URL: https://github.com/apache/hadoop/pull/7350#issuecomment-2637305850
@cnauroth The PR has the following points that need to be explained: #### TestAMWebApp.java In this unit test, EnvironmentVariables was introduced to simulate an SSL environment by modifying the System.Env environment variables during runtime. However, starting from JDK 9, direct modification of environment variables is prohibited. Therefore, we replaced it with the recommended approach using System.setProperty to set temporary variables. Additionally, we added logic in the WebApps to retrieve environment variables to ensure the test runs smoothly. #### TestStagingCleanup.java The Checkstyle issues in this class cannot be automatically fixed due to an extra space at the beginning of each line, which causes Checkstyle errors. #### New Dependency: mockito-junit-jupiter We introduced the mockito-junit-jupiter dependency to support the Mockito mocking functionality required in the TestTaskAttemptListenerImpl class. -- 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]
