Re: [PR] KAFKA-16223 Replace EasyMock/PowerMock with Mockito for KafkaConfigBackingStoreTest (1/3) [kafka]

2024-03-29 Thread via GitHub
gharris1727 merged PR #15520: URL: https://github.com/apache/kafka/pull/15520 -- 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:

Re: [PR] KAFKA-16223 Replace EasyMock/PowerMock with Mockito for KafkaConfigBackingStoreTest (1/3) [kafka]

2024-03-29 Thread via GitHub
gharris1727 commented on PR #15520: URL: https://github.com/apache/kafka/pull/15520#issuecomment-2027797163 Test failures appear unrelated, and the connect runtime tests pass locally. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] KAFKA-16223 Replace EasyMock/PowerMock with Mockito for KafkaConfigBackingStoreTest (1/3) [kafka]

2024-03-29 Thread via GitHub
gharris1727 commented on code in PR #15520: URL: https://github.com/apache/kafka/pull/15520#discussion_r1544876870 ## connect/runtime/src/main/java/org/apache/kafka/connect/storage/KafkaConfigBackingStore.java: ## @@ -329,6 +329,11 @@ public static String

[PR] KAFKA-16223 Replace EasyMock/PowerMock with Mockito for KafkaConfigBackingStoreTest (1/3) [kafka]

2024-03-11 Thread via GitHub
hgeraldino opened a new pull request, #15520: URL: https://github.com/apache/kafka/pull/15520 This is the last remaining Kafka Connect test that needs migration from PowerMock/EasyMock to Mockito. Following the same approach as the `WorkerSinkTaskTest` migration, which was migrated