olamy commented on pull request #478: URL: https://github.com/apache/maven-surefire/pull/478#issuecomment-1054711444
> Agree ... surefire code is enough complexity, we should reduce complexity not increase. > > Next question how to test such code with coverage all decision branch? there is an IT for this change. And IT are supposed to be recording jacoco as well. But strangly this doesn't look to go to it https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven-surefire/view/change-requests/job/PR-478/lastBuild/jacoco/org.apache.maven.plugin.surefire/SurefireHelper/ that's weird. I will review exactly how jacoco reports are merged or not. -- 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-unsubscr...@maven.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
