You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ba...@apache.org on 2011/10/17 06:55:03 UTC

svn commit: r1184998 - /maven/plugins/trunk/maven-eclipse-plugin/src/test/java/org/apache/maven/plugin/eclipse/it/MyEclipsePluginIT.java

Author: baerrach
Date: Mon Oct 17 04:55:02 2011
New Revision: 1184998

URL: http://svn.apache.org/viewvc?rev=1184998&view=rev
Log:
Fixed bug where reporting of build logs for MyEclipsePluginIT would incorrectly re-use the log file because the helper method used to invoke the tests matched the pattern used to generate the filename

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

Modified: maven/plugins/trunk/maven-eclipse-plugin/src/test/java/org/apache/maven/plugin/eclipse/it/MyEclipsePluginIT.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-eclipse-plugin/src/test/java/org/apache/maven/plugin/eclipse/it/MyEclipsePluginIT.java?rev=1184998&r1=1184997&r2=1184998&view=diff
==============================================================================
--- maven/plugins/trunk/maven-eclipse-plugin/src/test/java/org/apache/maven/plugin/eclipse/it/MyEclipsePluginIT.java (original)
+++ maven/plugins/trunk/maven-eclipse-plugin/src/test/java/org/apache/maven/plugin/eclipse/it/MyEclipsePluginIT.java Mon Oct 17 04:55:02 2011
@@ -18,6 +18,8 @@
  */
 package org.apache.maven.plugin.eclipse.it;
 
+import java.util.Properties;
+
 /**
  * Unit Tests for MyEclipse plugin
  * 
@@ -34,7 +36,7 @@ public class MyEclipsePluginIT
     public void testProject01()
         throws Exception
     {
-        testMyEclipseProject( "project-myeclipse-01" );
+        doMyEclipseProjectTest( "project-myeclipse-01" );
     }
 
     /**
@@ -45,7 +47,7 @@ public class MyEclipsePluginIT
     public void testProject02()
         throws Exception
     {
-        testMyEclipseProject( "project-myeclipse-02" );
+        doMyEclipseProjectTest( "project-myeclipse-02" );
     }
 
     /**
@@ -56,7 +58,7 @@ public class MyEclipsePluginIT
     public void testProject03()
         throws Exception
     {
-        testMyEclipseProject( "project-myeclipse-03" );
+        doMyEclipseProjectTest( "project-myeclipse-03" );
     }
 
     /**
@@ -67,7 +69,7 @@ public class MyEclipsePluginIT
     public void testProject04()
         throws Exception
     {
-        testMyEclipseProject( "project-myeclipse-04" );
+        doMyEclipseProjectTest( "project-myeclipse-04" );
     }
 
     /**
@@ -78,7 +80,7 @@ public class MyEclipsePluginIT
     public void testProject05()
         throws Exception
     {
-        testMyEclipseProject( "project-myeclipse-05" );
+        doMyEclipseProjectTest( "project-myeclipse-05" );
     }
 
     /**
@@ -89,7 +91,7 @@ public class MyEclipsePluginIT
     public void testMyEclipseProject06MECLIPSE427()
         throws Exception
     {
-        testMyEclipseProject( "project-myeclipse-06-MECLIPSE-427" );
+        doMyEclipseProjectTest( "project-myeclipse-06-MECLIPSE-427" );
     }
 
     /**
@@ -101,13 +103,13 @@ public class MyEclipsePluginIT
     public void testProject07MECLIPSE445()
         throws Exception
     {
-        testMyEclipseProject( "project-myeclipse-07-MECLIPSE-445" );
+        doMyEclipseProjectTest( "project-myeclipse-07-MECLIPSE-445" );
     }
 
-    public void testMyEclipseProject( String project )
+    private void doMyEclipseProjectTest( String project )
         throws Exception
     {
-        testProject( project, null, "myeclipse-clean", "myeclipse" );
+        testProject( project, new Properties(), "myeclipse-clean", "myeclipse" );
     }
 
 }