You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@maven.apache.org by br...@apache.org on 2006/03/04 04:07:27 UTC

svn commit: r383007 - /maven/plugins/branches/maven-surefire-plugin-testng/src/main/java/org/apache/maven/test/SurefirePlugin.java

Author: brett
Date: Fri Mar  3 19:07:24 2006
New Revision: 383007

URL: http://svn.apache.org/viewcvs?rev=383007&view=rev
Log:
[MSUREFIRE-23] get TestNG working under normal mode, and under forkMode=once

Modified:
    maven/plugins/branches/maven-surefire-plugin-testng/src/main/java/org/apache/maven/test/SurefirePlugin.java

Modified: maven/plugins/branches/maven-surefire-plugin-testng/src/main/java/org/apache/maven/test/SurefirePlugin.java
URL: http://svn.apache.org/viewcvs/maven/plugins/branches/maven-surefire-plugin-testng/src/main/java/org/apache/maven/test/SurefirePlugin.java?rev=383007&r1=383006&r2=383007&view=diff
==============================================================================
--- maven/plugins/branches/maven-surefire-plugin-testng/src/main/java/org/apache/maven/test/SurefirePlugin.java (original)
+++ maven/plugins/branches/maven-surefire-plugin-testng/src/main/java/org/apache/maven/test/SurefirePlugin.java Fri Mar  3 19:07:24 2006
@@ -515,6 +515,11 @@
             getLog().debug( "  " + classpathElement );
 
             surefireBooter.addClassPathUrl( classpathElement );
+
+            // TODO: fix
+            // Some test suites (TestNG) do not utilise the separate classloader, so we need to make all the tests
+            // available in the main classloader as well (this is consistent with the old behaviour in Surefire 1.5)
+            surefireBooter.addSurefireClassPathUrl( classpathElement );
         }
 
         // ----------------------------------------------------------------------