You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@commons.apache.org by GitBox <gi...@apache.org> on 2021/09/09 06:25:14 UTC

[GitHub] [commons-vfs] wx930910 commented on pull request #213: refactor CountingListener to improve test design

wx930910 commented on pull request #213:
URL: https://github.com/apache/commons-vfs/pull/213#issuecomment-915799887


   > This PR looks quite complex compared to the exting code. Why make maintenance more of a burden going forward? What am I missing here?
   
   @garydgregory Thanks for replying, actually we can get rid of all overridden methods as well as the attributes for tracking the overridden method exectuion by using Mockito. We can directly verify the method execution time by using `Mockito.verify()`, do you think the logic and maintenance will be better if we do it in that way?


-- 
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: issues-unsubscribe@commons.apache.org

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