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

[brett]

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 );
         }
 
         // 
----------------------------------------------------------------------