DonalEvans commented on pull request #6757:
URL: https://github.com/apache/geode/pull/6757#issuecomment-900595283


   > I agree @DonalEvans and thanks for your review comments. I am on it :).
   
   The changes in the most recent commit don't really address the issue I'm 
afraid. The test names should be made more descriptive to make it clear what 
behaviour is under test. Simply removing the "003", "004" etc. doesn't achieve 
this. A good guide for how to write descriptive test names and why doing so is 
helpful can be found here: 
https://osherove.com/blog/2005/4/3/naming-standards-for-unit-tests.html


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to