You are viewing a plain text version of this content. The canonical link for it is here.

Posted to jira@kafka.apache.org by "cadonna (via GitHub)" <gi...@apache.org> on 2023/04/08 09:00:34 UTC

[GitHub] [kafka] cadonna commented on pull request #12607: KAFKA-14133: Replace EasyMock with Mockito in streams tests

cadonna commented on PR #12607:
URL: https://github.com/apache/kafka/pull/12607#issuecomment-1500829973

   @clolov Since this PR is rather long and hard to review, I took a stab to subdivide it into smaller PRs by migrating single mocks. I opened a PR for the migration of the topology builder mock: https://github.com/apache/kafka/pull/13529.
   Let me know what you think about it. 
   If you think that is a good idea feel free to migrate the other mocks. I think we also need a bit to rethink the testing strategy in this test class. Sorry that this migration gets so complicated.


-- 
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: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org