[ 
https://issues.apache.org/jira/browse/KAFKA-9088?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16988210#comment-16988210
 ] 

Guozhang Wang commented on KAFKA-9088:
--------------------------------------

Yeah I had similar experiences with [~ableegoldman], while I feel that whenever 
I need to "fix" a test with EasyMock when making a minor change in the code 
like a small refactoring, it means that my test is not well written --- it 
should be testing the interfaces between classes / modules that are critical 
and treated like contracts between these classes. If not then this test is not 
useful and should be considered re-written. One example on top of my head are 
getters, like how many times you would call such functions, with minor changes 
the num.calls of getters may change, but should we really test it in our unit 
tests with EasyMock?

> Consolidate InternalMockProcessorContext and MockInternalProcessorContext
> -------------------------------------------------------------------------
>
>                 Key: KAFKA-9088
>                 URL: https://issues.apache.org/jira/browse/KAFKA-9088
>             Project: Kafka
>          Issue Type: Improvement
>          Components: streams, unit tests
>            Reporter: Bruno Cadonna
>            Assignee: Pierangelo Di Pilato
>            Priority: Minor
>              Labels: newbie
>
> Currently, we have two mocks for the {{InternalProcessorContext}}. The goal 
> of this ticket is to merge both into one mock or replace it with an 
> {{EasyMock}} mock. 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to