rmannibucau commented on a change in pull request #12: [MSHADE-307] defaults
for transformers and filters
URL: https://github.com/apache/maven-shade-plugin/pull/12#discussion_r243188080
##########
File path: src/test/java/org/apache/maven/plugins/shade/mojo/ShadeMojoTest.java
##########
@@ -60,6 +64,50 @@
public class ShadeMojoTest
extends PlexusTestCase
{
+ public void testDefaultConfiguration() throws Exception
Review comment:
Thought about it but I found it saner as a compromise cause this is really
more a whitebox test to check we mutate the user config as expected than an IT
which would be more complicated with fewer guarantee on what we do at that
level IMHO (here we really want to check the default config is setup and not
smoke test it, at least was the intention)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
