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

   Moves 17 ITs into unit tests: transformer behaviour (resource bundles, XML 
without DTD resolution, manifest entries, Sisu indexes, Apache NOTICE project 
name and year, Groovy static extensions), relocation (annotations, string 
constants, enums, absolute resource paths, includes and excludes, serialized 
lambdas, run from a shaded jar in an isolated class loader), the module-info 
warning, skip, and unmatched artifact-set patterns. Ten more ITs are removed 
because existing tests assert the same thing, with new tests where the existing 
ones fell short. Each new test fails when the production line it guards is 
broken.
   
   The serialized-lambda test now deserializes the lambda in the shaded class 
loader, and reloc-class-from-string-pool's fixture called the dotted array case 
twice and never the slashed one; it now runs both. The dependency-reduced-POM 
ITs stay. Part of apache/maven#13822.
   
   Verified: unit tests 130 → 152 (1 skipped), also under `-Duser.language=de 
-Duser.timezone=Asia/Tokyo`; `mvn -B -P run-its verify` → Passed: 67, Skipped: 
1 (ITs 95 → 68)
   
   ---
   
   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