svn commit: r398633 - /maven/surefire/trunk/surefire-booter/src/main/java/org/apache/maven/surefire/booter/SurefireBooter.java

[brett]

Author: brett
Date: Mon May  1 09:49:55 2006
New Revision: 398633

URL: http://svn.apache.org/viewcvs?rev=398633&view=rev
Log:
[MSUREFIRE-74] don't duplicate classpath jars

Modified:
    
maven/surefire/trunk/surefire-booter/src/main/java/org/apache/maven/surefire/booter/SurefireBooter.java

Modified: 
maven/surefire/trunk/surefire-booter/src/main/java/org/apache/maven/surefire/booter/SurefireBooter.java
URL: 
http://svn.apache.org/viewcvs/maven/surefire/trunk/surefire-booter/src/main/java/org/apache/maven/surefire/booter/SurefireBooter.java?rev=398633&r1=398632&r2=398633&view=diff
==============================================================================
--- 
maven/surefire/trunk/surefire-booter/src/main/java/org/apache/maven/surefire/booter/SurefireBooter.java
 (original)
+++ 
maven/surefire/trunk/surefire-booter/src/main/java/org/apache/maven/surefire/booter/SurefireBooter.java
 Mon May  1 09:49:55 2006
@@ -713,6 +713,12 @@
                 }
             }
 
+            // TODO: fix. See also, surefire plugin. Currently, testng is 
putting all of the test classes into the surefire
+            // classpath as the classloaders are not separataed. Here, we need 
to remove them again to avoid duplication
+            List urls = new ArrayList( surefireBooter.classPathUrls );
+            urls.removeAll( surefireBooter.surefireClassPathUrls );
+            surefireBooter.classPathUrls = urls;
+
             boolean childDelegation = Boolean.valueOf( p.getProperty( 
"childDelegation" ) ).booleanValue();
             String testSet = p.getProperty( "testSet" );
             boolean result;