mdedetrich commented on PR #12728:
URL: https://github.com/apache/kafka/pull/12728#issuecomment-1844822620

   I have just pushed a merge commit which adds the 
`testCreateConnectorWithStoppedInitialState()` test. The translation of the 
test was quite straight forward, only thing to note is that the original test 
used `PowerMock.verifyAll();` as an escape hatch, I replaced it with 
`verify(loaderSwap).close();`, not sure if you want to add any more 
verifications (or even remove `verify(loaderSwap).close();` as being 
misleading).


-- 
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-unsubscr...@kafka.apache.org

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

Reply via email to