You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by jd...@apache.org on 2008/04/08 21:51:11 UTC

svn commit: r646051 - in /maven/core-integration-testing/trunk/core-integration-tests/src/test: java/org/apache/maven/integrationtests/ resources/mng-3473PluginReportCrash/

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>