slawekjaranowski commented on pull request #434:
URL: https://github.com/apache/maven-surefire/pull/434#issuecomment-1008068699


   Page with internal concept should be refreshed but more important is to have 
tests which confirm that some feature is needed.
   
   There is about `1000 ITs` tests, take more than hour, and any one touch this 
... but it is needed ... I don't feel well with such approach
   
   If we have mixed responsibility and one class does many things it alway hard 
to test and understood and consequently is hard to maintenance. 
   
   I am an advocate to have things simply as possible with clear contract.
   
   I think that is good time in milestone version to do such refactor.


-- 
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