Author: gboue
Date: Sun Jun 11 16:52:30 2017
New Revision: 1798383
URL: http://svn.apache.org/viewvc?rev=1798383&view=rev
Log:
[MCHECKSTYLE-338] Add support for 'omitIgnoredModules'
Make IT not rely on the 'check' goal invoking the 'checkstyle' goal first.
Modified:
maven/plugins/trunk/maven-checkstyle-plugin/src/it/MCHECKSTYLE-338/pom.xml
maven/plugins/trunk/maven-checkstyle-plugin/src/it/MCHECKSTYLE-338/verify.groovy
Modified:
maven/plugins/trunk/maven-checkstyle-plugin/src/it/MCHECKSTYLE-338/pom.xml
URL:
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-checkstyle-plugin/src/it/MCHECKSTYLE-338/pom.xml?rev=1798383&r1=1798382&r2=1798383&view=diff
==============================================================================
--- maven/plugins/trunk/maven-checkstyle-plugin/src/it/MCHECKSTYLE-338/pom.xml
(original)
+++ maven/plugins/trunk/maven-checkstyle-plugin/src/it/MCHECKSTYLE-338/pom.xml
Sun Jun 11 16:52:30 2017
@@ -46,7 +46,9 @@
</dependencies>
<executions>
<execution>
+ <phase>verify</phase>
<goals>
+ <goal>checkstyle</goal>
<goal>check</goal>
</goals>
</execution>
Modified:
maven/plugins/trunk/maven-checkstyle-plugin/src/it/MCHECKSTYLE-338/verify.groovy
URL:
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-checkstyle-plugin/src/it/MCHECKSTYLE-338/verify.groovy?rev=1798383&r1=1798382&r2=1798383&view=diff
==============================================================================
---
maven/plugins/trunk/maven-checkstyle-plugin/src/it/MCHECKSTYLE-338/verify.groovy
(original)
+++
maven/plugins/trunk/maven-checkstyle-plugin/src/it/MCHECKSTYLE-338/verify.groovy
Sun Jun 11 16:52:30 2017
@@ -24,7 +24,7 @@ buildLog = new File( basedir, 'build.log
assert buildLog.text.contains( 'EmptyLogging: EmptyLoggingCheck on file
TestFalse.java' )
assert !buildLog.text.contains( 'EmptyLogging: EmptyLoggingCheck on file
TestTrue.java' )
-// verify that the "checkstyle" goal launched by "check" does the same
+// verify that the "checkstyle" goal does the same
site = new File( basedir,
'omitignoredmodules-false/target/site/checkstyle.html' )
assert site.text.contains( 'EmptyLoggingCheck on file TestFalse.java' )
site = new File( basedir,
'omitignoredmodules-true/target/site/checkstyle.html' )
