slachiewicz opened a new pull request, #1263: URL: https://github.com/apache/maven-plugin-tools/pull/1263
Moves seven ITs into unit tests: the nested-mojo and comment-only-source extraction cases (`JavaAnnotationsMojoDescriptorExtractorTest`), and in maven-plugin-plugin the `skip` and packaging guards, the Maven-dependency scope warning (bad and good), and reproducible help-mojo sources. maven-plugin-plugin gets `maven-plugin-testing-harness` in test scope for that. Each new test fails when the production line it guards is broken. packaging-jar moves only its `packagingTypes` guard; goal and implementation stay covered by the java-basic-annotations ITs. The skip test sets the parameter in its POM, so binding `maven.plugin.skip` from the command line is no longer exercised. Part of apache/maven#13822. Verified: unit tests 37 → 39 (annotations) and 12 → 18 (maven-plugin-plugin); `mvn -B verify -P run-its -pl maven-plugin-tools-annotations,maven-plugin-plugin` → Passed: 25, Skipped: 1 (ITs 33 → 26) --- To make clear that you license your contribution under the [Apache License Version 2.0, January 2004](https://www.apache.org/licenses/LICENSE-2.0) check one of the following: - [ ] I hereby declare this contribution to be licensed under the [Apache License Version 2.0, January 2004](https://www.apache.org/licenses/LICENSE-2.0) - [x] I hereby declare that I have filed an [Apache Individual Contributor License Agreement](https://www.apache.org/licenses/icla.pdf). -- 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]
