anmolanmol1234 commented on PR #5977: URL: https://github.com/apache/hadoop/pull/5977#issuecomment-1695276551
> looking at the source of TestTimelineAuthFilterForV2 i can see mockito references, so was worried it was a regression -but my own pr #5981 has the same failure, so its not a regression. > > TestDockerContainerRuntime does look new though; it only surfaced in #5273 after you updated mockito everywhere. > > Afraid you _do_ get to work out what's changed. sorry. This is why i'm not a fan of Mockito; tests are too brittle to internal code changes; risk of not wanting to make a change to avoid failures, or ignoring failure when it's an actual regression. Have corrected the test, apparently the test wanted to verify that nomoreinteractions was made after the command has been initialized. -- 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]
