slachiewicz opened a new pull request, #800:
URL: https://github.com/apache/maven-invoker-plugin/pull/800

   Moves eight ITs into unit tests: log directory and no-clone build logs, the 
JUnit report (success, failure, skipped, error), Maven extension and POM 
filtering (precedence and encoding), and skipping installation and invocation. 
`cloneProjects`, `writeJunitReport` and `setupBuildLogFile` become 
package-private so the tests can call them; nothing outside 
`AbstractInvokerMojo` uses them. Each new test fails when the production line 
it guards is broken.
   
   clone-clean and pom-filtering-reactor stay ITs: what they check lives in 
other private methods. The skip tests set the field directly, so binding 
`invoker.skip` is no longer exercised. Part of apache/maven#13822.
   
   Verified: unit tests 75 → 87; ITs 93 → 85; the kept ITs next to the moved 
behaviour (10, including clone-clean and pom-filtering-reactor) pass with `-P 
run-its`
   
   ---
   
   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]

Reply via email to