Copilot commented on code in PR #3371:
URL: https://github.com/apache/maven-surefire/pull/3371#discussion_r3616800215
##########
maven-surefire-plugin/src/site/apt/examples/skipping-tests.apt.vm:
##########
@@ -88,9 +88,15 @@ mvn install -DskipTests
mvn install -DskipTests
+---+
+#{if}(${project.artifactId}=="maven-surefire-plugin")
Since <<<skipTests>>> is also supported by the ${thatPlugin} Plugin, this
will have the effect
- of not running any tests. If, instead, you want to skip only the integration
tests
- being run by the ${thisPlugin} Plugin, you would use the <<<skipITs>>>
property instead:
+ of not running any tests. If, instead, you want to skip only the integration
tests
+ being run by the ${thatPlugin} Plugin, you would use the <<<skipITs>>>
property instead:
Review Comment:
In the Surefire-plugin branch of this example, the text still claims that
`-DskipTests` (aka `skipTests`) will also skip the Failsafe plugin, resulting
in *no tests* running. After this PR’s behavioral change, `-DskipTests` should
only skip Surefire unit tests and must not skip integration tests run by the
Failsafe plugin, so this paragraph becomes misleading.
--
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]
