DOJI45 commented on pull request #9247: URL: https://github.com/apache/kafka/pull/9247#issuecomment-693451000
> > @DOJI45 Thanks for the PR! Could you also add a unit test to complete the coverage? You can find some examples in StreamTaskTest or ProcessorStateManagerTest > > Seems this comment is not addressed yet? Hi @guozhangwang I wrote unit test cases in `ProcessorStateManagerTest` for the new method that I have added Not sure if I can write unit test case in `StreamTaskTest` class, because functionality wise i won't be able to validate anything, as deleting checkpoint file is tested in `ProcessorStateManagerTest`, and coverage is taken care by the existing unit test cases. Please suggest how to proceed? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
