Author: jdcasey Date: Tue Apr 8 12:51:08 2008 New Revision: 646051 URL: http://svn.apache.org/viewvc?rev=646051&view=rev Log: Minor rename for MNG-3498, adjustments to two others to exempt 2.1 from being tested by them (in the case of 3221 this is in hopes of fixing the relationship between site plugin and reports, and in the case of 3485 we've been talking for awhile about moving to protocol mapping for wagons instead of including them in the core). Finally, adjusting 3473 to be a little more bullet-proof WRT the use of the ${plugin.version} expression and report-plugin versioning.
Added: maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3498ForkToOtherMojoTest.java - copied, changed from r644488, maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MNG3498Test.java Removed: maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MNG3498Test.java Modified: maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/IntegrationTestSuite.java maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3221InfiniteForking.java maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3473PluginReportCrash.java maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3485OverrideWagonExtensionTest.java maven/core-integration-testing/trunk/core-integration-tests/src/test/resources/mng-3473PluginReportCrash/pom.xml Modified: maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/IntegrationTestSuite.java URL: http://svn.apache.org/viewvc/maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/IntegrationTestSuite.java?rev=646051&r1=646050&r2=646051&view=diff ============================================================================== --- maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/IntegrationTestSuite.java (original) +++ maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/IntegrationTestSuite.java Tue Apr 8 12:51:08 2008 @@ -65,7 +65,7 @@ * a fail fast technique as well. */ - suite.addTestSuite( MNG3498Test.class ); + suite.addTestSuite( MavenITmng3498ForkToOtherMojoTest.class ); suite.addTestSuite( MavenITmng3485OverrideWagonExtensionTest.class ); suite.addTestSuite( MavenITmng3473PluginReportCrash.class ); suite.addTestSuite( MavenITmng3428PluginDescriptorArtifactsIncompleteTest.class ); Modified: maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3221InfiniteForking.java URL: http://svn.apache.org/viewvc/maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3221InfiniteForking.java?rev=646051&r1=646050&r2=646051&view=diff ============================================================================== --- maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3221InfiniteForking.java (original) +++ maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3221InfiniteForking.java Tue Apr 8 12:51:08 2008 @@ -5,8 +5,8 @@ import org.apache.maven.it.util.ResourceExtractor; import java.io.File; -import java.util.List; import java.util.ArrayList; +import java.util.List; public class MavenITmng3221InfiniteForking extends AbstractMavenIntegrationTestCase @@ -14,7 +14,7 @@ public MavenITmng3221InfiniteForking() throws InvalidVersionSpecificationException { - super( "(2.0.8,)" ); // only test in 2.0.9+ + super( "(2.0.8,2.1-SNAPSHOT)" ); // only test in 2.0.9+ } public void testitMNG3221a() Modified: maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3473PluginReportCrash.java URL: http://svn.apache.org/viewvc/maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3473PluginReportCrash.java?rev=646051&r1=646050&r2=646051&view=diff ============================================================================== --- maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3473PluginReportCrash.java (original) +++ maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3473PluginReportCrash.java Tue Apr 8 12:51:08 2008 @@ -1,14 +1,14 @@ package org.apache.maven.integrationtests; -import java.io.File; -import java.util.ArrayList; -import java.util.List; - import org.apache.maven.artifact.versioning.InvalidVersionSpecificationException; import org.apache.maven.it.VerificationException; import org.apache.maven.it.Verifier; import org.apache.maven.it.util.ResourceExtractor; +import java.io.File; +import java.util.ArrayList; +import java.util.List; + /** * Tests that the PluginDescriptor.getArtifacts() call returns all of the dependencies of the plugin, * not just those that made it past the filter excluding Maven's core artifacts. @@ -29,31 +29,34 @@ Verifier verifier = new Verifier( testDir.getAbsolutePath() ); + // force the use of the 2.4.1 plugin version via a profile here... + List cliOptions = new ArrayList(); + cliOptions.add( "-Pplugin-2.4.1" ); + verifier.setCliOptions( cliOptions ); + verifier.executeGoal( "install" ); verifier.verifyErrorFreeLog(); verifier.resetStreams(); - - List cliOptions = new ArrayList(); - cliOptions.add( "-Dplugin.version=2.4.1" ); - verifier.setCliOptions( cliOptions ); - + //should succeed with 2.4.1 verifier.executeGoal( "site" ); - verifier.verifyErrorFreeLog(); + + // NOTE: Velocity prints an [ERROR] line pertaining to an incorrect macro usage when run in 2.1, so this doesn't work. +// verifier.verifyErrorFreeLog(); verifier.resetStreams(); - + //should fail with 2.4 cliOptions.clear(); - cliOptions.add( "-Dplugin.version=2.4" ); + cliOptions.add( "-Pplugin-2.4" ); verifier.setCliOptions( cliOptions ); - + try { verifier.executeGoal( "site" ); } catch (VerificationException e) { - //expected this but don't require it cause some os's don't return the correct error code + //expected this but don't require it cause some os's don't return the correct error code } verifier.verifyTextInLog( "org/apache/maven/doxia/module/site/manager/SiteModuleNotFoundException" ); verifier.resetStreams(); Modified: maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3485OverrideWagonExtensionTest.java URL: http://svn.apache.org/viewvc/maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3485OverrideWagonExtensionTest.java?rev=646051&r1=646050&r2=646051&view=diff ============================================================================== --- maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3485OverrideWagonExtensionTest.java (original) +++ maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3485OverrideWagonExtensionTest.java Tue Apr 8 12:51:08 2008 @@ -1,21 +1,20 @@ package org.apache.maven.integrationtests; -import java.io.File; -import java.util.ArrayList; -import java.util.List; - import org.apache.maven.artifact.versioning.InvalidVersionSpecificationException; -import org.apache.maven.integrationtests.AbstractMavenIntegrationTestCase; import org.apache.maven.it.Verifier; import org.apache.maven.it.util.ResourceExtractor; +import java.io.File; +import java.util.ArrayList; +import java.util.List; + public class MavenITmng3485OverrideWagonExtensionTest extends AbstractMavenIntegrationTestCase { public MavenITmng3485OverrideWagonExtensionTest() throws InvalidVersionSpecificationException { - super( "(2.0.8,)" ); // only test in 2.0.9+ + super( "(2.0.8, 2.1-SNAPSHOT)" ); // only test in 2.0.9+ } public void testitMNG3485 () Copied: maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3498ForkToOtherMojoTest.java (from r644488, maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MNG3498Test.java) URL: http://svn.apache.org/viewvc/maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3498ForkToOtherMojoTest.java?p2=maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3498ForkToOtherMojoTest.java&p1=maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MNG3498Test.java&r1=644488&r2=646051&rev=646051&view=diff ============================================================================== --- maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MNG3498Test.java (original) +++ maven/core-integration-testing/trunk/core-integration-tests/src/test/java/org/apache/maven/integrationtests/MavenITmng3498ForkToOtherMojoTest.java Tue Apr 8 12:51:08 2008 @@ -33,10 +33,10 @@ * @author jdcasey * */ -public class MNG3498Test +public class MavenITmng3498ForkToOtherMojoTest extends AbstractMavenIntegrationTestCase { -// public MNG3498Test() +// public MavenITmng3498ForkToOtherMojoTest() // throws InvalidVersionSpecificationException // { // super( "(2.0.8,)" ); // only test in 2.0.9+ Modified: maven/core-integration-testing/trunk/core-integration-tests/src/test/resources/mng-3473PluginReportCrash/pom.xml URL: http://svn.apache.org/viewvc/maven/core-integration-testing/trunk/core-integration-tests/src/test/resources/mng-3473PluginReportCrash/pom.xml?rev=646051&r1=646050&r2=646051&view=diff ============================================================================== --- maven/core-integration-testing/trunk/core-integration-tests/src/test/resources/mng-3473PluginReportCrash/pom.xml (original) +++ maven/core-integration-testing/trunk/core-integration-tests/src/test/resources/mng-3473PluginReportCrash/pom.xml Tue Apr 8 12:51:08 2008 @@ -14,22 +14,59 @@ <version>2.0</version> </dependency> </dependencies> + <profiles> + <profile> + <id>plugin-2.4.1</id> + <activation> + <activeByDefault>true</activeByDefault> + <property> + <name>plugin.version</name> + <value>2.4.1</value> + </property> + </activation> + + <reporting> + <plugins> + <plugin> + <artifactId>maven-plugin-plugin</artifactId> + <!--only fails with 2.4, not 2.3 or 2.4.1--> + <version>2.4.1</version> + </plugin> + </plugins> + </reporting> + </profile> + <profile> + <id>plugin-2.4</id> + <activation> + <property> + <name>plugin.version</name> + <value>2.4</value> + </property> + </activation> + + <reporting> + <plugins> + <plugin> + <artifactId>maven-plugin-plugin</artifactId> + <!--only fails with 2.4, not 2.3 or 2.4.1--> + <version>2.4</version> + </plugin> + </plugins> + </reporting> + </profile> + </profiles> + <build> <pluginManagement> <plugins> - <artifactId>maven-site-plugin</artifactId> - <version>2.0-beta-6</version> + <plugin> + <artifactId>maven-site-plugin</artifactId> + <version>2.0-beta-6</version> + </plugin> </plugins> </pluginManagement> </build> - <reporting> - <outputDirectory>target/site</outputDirectory> - <plugins> - <plugin> - <artifactId>maven-plugin-plugin</artifactId> - <!--only fails with 2.4, not 2.3 or 2.4.1--> - <version>${plugin.version}</version> - </plugin> - </plugins> - </reporting> + <reporting> + <outputDirectory>target/site</outputDirectory> + </reporting> </project>