Tibor17 commented on PR #3232: URL: https://github.com/apache/maven-surefire/pull/3232#issuecomment-3703711451
@elharo Is it a broken functionality? No! Do you know what means "surefire"? It's purpose is to point to non-normal tests. This includes also the ignored tests. Are you committers accepting every single wish from a single user? I will tell you what will happen now and why the contributor did this. He has placed ignored tests on the same level of normally functioning tests which is against the idea of Surefire because Sirefire has to stop the build if a single test has failed or Surefire has to point to a problematic test quality. And we know for certain what the developers are like; so, we know that the developers don't want the manager to see ignored tests because a broken functionality is ignored from testing and everybody thinks that it's alright. But it's not due to the developer moved a broken functionality ahead, he wanted to get time and simply a temporary solution is a kind of solution which stays forever - IT jargon but it's real! -- 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]
