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' )