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>


Reply via email to