You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ca...@apache.org on 2007/10/19 20:06:48 UTC
svn commit: r586564 -
/maven/components/trunk/maven-core/src/test/java/org/apache/maven/extension/DefaultExtensionManagerTest.java
Author: carlos
Date: Fri Oct 19 11:06:48 2007
New Revision: 586564
URL: http://svn.apache.org/viewvc?rev=586564&view=rev
Log:
Add missing super.tearDown, and ignore problem in windows where file is locked for deletion (needs investigation)
Modified:
maven/components/trunk/maven-core/src/test/java/org/apache/maven/extension/DefaultExtensionManagerTest.java
Modified: maven/components/trunk/maven-core/src/test/java/org/apache/maven/extension/DefaultExtensionManagerTest.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/test/java/org/apache/maven/extension/DefaultExtensionManagerTest.java?rev=586564&r1=586563&r2=586564&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/test/java/org/apache/maven/extension/DefaultExtensionManagerTest.java (original)
+++ maven/components/trunk/maven-core/src/test/java/org/apache/maven/extension/DefaultExtensionManagerTest.java Fri Oct 19 11:06:48 2007
@@ -75,18 +75,25 @@
public void tearDown()
throws Exception
{
+ super.tearDown();
+
for ( Iterator it = toDelete.iterator(); it.hasNext(); )
{
File f = (File) it.next();
if ( f.exists() )
{
- File f2 = File.createTempFile( "xx.", "" );
-
- f2.delete();
- f.renameTo( f2 );
-
- FileUtils.forceDelete( f2 );
+ try
+ {
+ FileUtils.forceDelete( f );
+ }
+ catch ( IOException e )
+ {
+ // TODO carlos: delete fails on windows, needs investigation
+ // the test-extension-1.jar is locked for deletion
+ // I traced it and seems to get locked at getRealm().findRealmResources(
+ // name ) in org.codehaus.plexus.classworlds.strategy.DefaultStrategy:146
+ }
}
}
}