svn commit: r709796 - in /maven/core-integration-testing/trunk/core-it-suite/src/test: java/org/apache/maven/it/ resources/it0118-attachedartifactinreactor/

[be...@apache.org]

Author: bentmann
Date: Sat Nov  1 15:47:19 2008
New Revision: 709796

URL: http://svn.apache.org/viewvc?rev=709796&view=rev
Log:
o Deleted it0118
  (This test was meant to check MNG-2877 but it fails to detect the regression MNG-2921 that was caused by the patch for MNG-2877. The test that is now in place for MNG-2921 subsumes it0118.)

Removed:
    maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/MavenIT0118AttachedArtifactsInReactorTest.java
    maven/core-integration-testing/trunk/core-it-suite/src/test/resources/it0118-attachedartifactinreactor/
Modified:
    maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/IntegrationTestSuite.java
    maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng2921ActiveAttachedArtifactsTest.java

Modified: maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/IntegrationTestSuite.java
URL: http://svn.apache.org/viewvc/maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/IntegrationTestSuite.java?rev=709796&r1=709795&r2=709796&view=diff
==============================================================================
--- maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/IntegrationTestSuite.java (original)
+++ maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/IntegrationTestSuite.java Sat Nov  1 15:47:19 2008
@@ -173,7 +173,6 @@
         suite.addTestSuite( MavenIT0131SiteLifecycleTest.class );
         suite.addTestSuite( MavenIT0130CleanLifecycleTest.class );
         suite.addTestSuite( MavenIT0129ResourceProvidedToAPluginAsAPluginDependencyTest.class );
-        suite.addTestSuite( MavenIT0118AttachedArtifactsInReactorTest.class );
         suite.addTestSuite( MavenIT0115CustomArtifactHandlerAndCustomLifecycleTest.class );
         suite.addTestSuite( MavenIT0113ServerAuthzAvailableToWagonMgrInPluginTest.class );
         suite.addTestSuite( MavenIT0110PluginDependenciesComeFromPluginReposTest.class );

Modified: maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng2921ActiveAttachedArtifactsTest.java
URL: http://svn.apache.org/viewvc/maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng2921ActiveAttachedArtifactsTest.java?rev=709796&r1=709795&r2=709796&view=diff
==============================================================================
--- maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng2921ActiveAttachedArtifactsTest.java (original)
+++ maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng2921ActiveAttachedArtifactsTest.java Sat Nov  1 15:47:19 2008
@@ -26,7 +26,9 @@
 import java.util.Collection;
 
 /**
- * This is a test set for <a href="http://jira.codehaus.org/browse/MNG-2921">MNG-2921</a>.
+ * This is a test set for <a href="http://jira.codehaus.org/browse/MNG-2921">MNG-2921</a>. It naturally includes the
+ * test for the related issue <a href="http://jira.codehaus.org/browse/MNG-2877">MNG-2877</a> whose original test was
+ * too weak to prevent this issue.
  * 
  * @author Benjamin Bentmann
  * @version $Id$
@@ -45,7 +47,8 @@
      * consumption on other module's class paths. Note the subtle difference of this test compared to the closely
      * related issue MNG-2871: This test is about *attached* artifacts, i.e. dependencies that have already been
      * packaged. MNG-2871 on the other hand is about dependencies that haven't been packaged yet but merely exist
-     * as loose class files in a module's output directory.
+     * as loose class files in a module's output directory. In other words, this test is concerned with the situation
+     * during the lifecycle phase "package" while MNG-2871 is concerned with earlier phases like "test".
      */
     public void testitMNG2921()
         throws Exception