[
https://issues.apache.org/jira/browse/KAFKA-14132?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17604794#comment-17604794
]
Chris Egerton commented on KAFKA-14132:
---------------------------------------
[~christo_lolov] Any preference on how we should indicate which tests on this
list have been migrated? I've just merged
[https://github.com/apache/kafka/pull/12615|https://github.com/apache/kafka/pull/12615,]
from [~yash.mayya], which takes care of the {{{}ErrorHandlingTaskTest{}}},
{{{}WorkerTaskTest{}}}, {{{}ErrorReporterTest{}}},
{{{}RetryWithToleranceOperatorTest{}}}, and {{WorkerErrantRecordReporterTest}}
test suites.
> Remaining PowerMock to Mockito tests
> ------------------------------------
>
> Key: KAFKA-14132
> URL: https://issues.apache.org/jira/browse/KAFKA-14132
> Project: Kafka
> Issue Type: Sub-task
> Reporter: Christo Lolov
> Assignee: Christo Lolov
> Priority: Major
>
> {color:#de350b}Some of the tests below use EasyMock as well. For those
> migrate both PowerMock and EasyMock to Mockito.{color}
> Unless stated in brackets the tests are in the connect module.
> A list of tests which still require to be moved from PowerMock to Mockito as
> of 2nd of August 2022 which do not have a Jira issue and do not have pull
> requests I am aware of which are opened:
> # ErrorHandlingTaskTest (owner: Divij)
> # SourceTaskOffsetCommiterTest (owner: Divij)
> # WorkerMetricsGroupTest (owner: Divij)
> # WorkerSinkTaskTest (owner: Divij)
> # WorkerSinkTaskThreadedTest (owner: Divij)
> # WorkerTaskTest (owner: [~yash.mayya])
> # ErrorReporterTest (owner: [~yash.mayya])
> # RetryWithToleranceOperatorTest (owner: [~yash.mayya])
> # WorkerErrantRecordReporterTest (owner: [~yash.mayya])
> # ConnectorsResourceTest
> # StandaloneHerderTest
> # KafkaConfigBackingStoreTest
> # KafkaOffsetBackingStoreTest (owner: Christo)
> ([https://github.com/apache/kafka/pull/12418])
> # KafkaBasedLogTest
> # RetryUtilTest
> # RepartitionTopicTest (streams) (owner: Christo)
> # StateManagerUtilTest (streams) (owner: Christo)
> *The coverage report for the above tests after the change should be >= to
> what the coverage is now.*
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
