svn commit: r441093 - /maven/plugins/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java

[ke...@apache.org]

Author: kenney
Date: Thu Sep  7 07:25:27 2006
New Revision: 441093

URL: http://svn.apache.org/viewvc?view=rev&rev=441093
Log:
PR: SUREFIRE-54

Related to commit revision 441092.

Removed adding test classpathelements to the surefire classpath
since the surefire classloader is a child of the test classloader
and already has access to all the test classes.

Modified:
    maven/plugins/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java

Modified: maven/plugins/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java?view=diff&rev=441093&r1=441092&r2=441093
==============================================================================
--- maven/plugins/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java (original)
+++ maven/plugins/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java Thu Sep  7 07:25:27 2006
@@ -585,11 +585,6 @@
             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 );
         }
 
         // ----------------------------------------------------------------------