Dzeri96 commented on PR #3597: URL: https://github.com/apache/celeborn/pull/3597#issuecomment-4245387067
I've spent so much time fighting Mockito on this one ([issue](https://github.com/mockito/mockito/issues/3811) opened), that I forgot what I wrote in this PR. Finally I managed to write a half-decent test that mocks the `MasterClient` and intercepts the communication between the two. In the future, we should inject this client in the Master/Worker to make testing easier, but for now, this should work. @SteNicholas Please be so kind to take another look. -- 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]
