You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ah...@apache.org on 2008/01/30 22:56:28 UTC

svn commit: r616918 - /maven/plugins/trunk/maven-eclipse-plugin/src/test/java/org/apache/maven/plugin/eclipse/EclipsePluginTest.java

Author: aheritier
Date: Wed Jan 30 13:56:27 2008
New Revision: 616918

URL: http://svn.apache.org/viewvc?rev=616918&view=rev
Log:
* tests 07 & 33 are now deactivated due to MECLIPSE-367
* tests 16 & 17 are ok but the plugin has to be built with maven >= 2.0.8

Modified:
    maven/plugins/trunk/maven-eclipse-plugin/src/test/java/org/apache/maven/plugin/eclipse/EclipsePluginTest.java

Modified: maven/plugins/trunk/maven-eclipse-plugin/src/test/java/org/apache/maven/plugin/eclipse/EclipsePluginTest.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-eclipse-plugin/src/test/java/org/apache/maven/plugin/eclipse/EclipsePluginTest.java?rev=616918&r1=616917&r2=616918&view=diff
==============================================================================
--- maven/plugins/trunk/maven-eclipse-plugin/src/test/java/org/apache/maven/plugin/eclipse/EclipsePluginTest.java (original)
+++ maven/plugins/trunk/maven-eclipse-plugin/src/test/java/org/apache/maven/plugin/eclipse/EclipsePluginTest.java Wed Jan 30 13:56:27 2008
@@ -82,14 +82,12 @@
     }
 
     /**
-     * @TODO temporarily disabled, since it randomly fails due to a different order for dependencies in classpath and
-     *       wtpmodules. This is not a problem, since order could be ignored in this test, but we should rewrite the
-     *       file-comparing step which at the moment just does line by line comparison
      * @throws Exception
      */
     public void testProject07()
         throws Exception
     {
+        // Fails because of MECLIPSE-367
         // testProject( "project-07" );
     }
 
@@ -168,27 +166,29 @@
         testProject( "project-15", props, "clean", "eclipse" );
     }
 
-    // Commented out: failing due to MNG-2025
-    // /**
-    // * UTF8 encoding - MECLIPSE-56
-    // * @throws Exception any exception thrown during test
-    // */
-    // public void testProject16()
-    // throws Exception
-    // {
-    // testProject( "project-16" );
-    // }
-
-    // Commented out: failing on Continuum, due to MNG-2025 too?
-    // /**
-    // * ISO-8859-15 encoding - MECLIPSE-56
-    // * @throws Exception any exception thrown during test
-    // */
-    // public void testProject17()
-    // throws Exception
-    // {
-    // testProject( "project-17" );
-    // }
+    /**
+     * UTF8 encoding - MECLIPSE-56
+     * 
+     * @throws Exception any exception thrown during test
+     */
+    public void testProject16()
+        throws Exception
+    {
+        // failing with maven < 2.0.8 due to MNG-2025
+        testProject( "project-16" );
+    }
+
+    /**
+     * ISO-8859-15 encoding - MECLIPSE-56
+     * 
+     * @throws Exception any exception thrown during test
+     */
+    public void testProject17()
+        throws Exception
+    {
+        // failing with maven < 2.0.8 due to MNG-2025
+        testProject( "project-17" );
+    }
 
     /**
      * relative location of system dependencies - MECLIPSE-89
@@ -368,15 +368,12 @@
     /**
      * MECLIPSE-287 : dependencies with and without classifiers MECLIPSE-151 : test jar source attachments
      * 
-     * @TODO temporarily disabled, since it randomly fails due to a different order for dependencies in classpath and
-     *       wtpmodules. This is not a problem, since order could be ignored in this test, but we should rewrite the
-     *       file-comparing step which at the moment just does line by line comparison project 7 is affected by this as
-     *       well.
      * @throws Exception any exception thrown during test
      */
     public void testProject33()
         throws Exception
     {
+        // Fails because of MECLIPSE-367
         // testProject( "project-33" );
     }