Hi,

This does change the semantic (thought it may not affect the test that much), 
as INDEX_PATH is a zip file - I wonder if you just needed 
file.getParentFile().mkdirs()? Or is there a problem with file.createNewFile() 
that it needs to actually create something?

Thanks,
Brett

On 29 Jan 2014, at 12:34 pm, [email protected] wrote:

> Author: skygo
> Date: Wed Jan 29 01:34:02 2014
> New Revision: 1562340
> 
> URL: http://svn.apache.org/r1562340
> Log:
> fix test for windoz os (hope not breaking semantic)
> 
> Modified:
>    
> archiva/branches/archiva-1.3.x/archiva-modules/archiva-base/archiva-consumers/archiva-core-consumers/src/test/java/org/apache/maven/archiva/consumers/core/repository/CleanupReleasedSnapshotsRepositoryPurgeTest.java
> 
> Modified: 
> archiva/branches/archiva-1.3.x/archiva-modules/archiva-base/archiva-consumers/archiva-core-consumers/src/test/java/org/apache/maven/archiva/consumers/core/repository/CleanupReleasedSnapshotsRepositoryPurgeTest.java
> URL: 
> http://svn.apache.org/viewvc/archiva/branches/archiva-1.3.x/archiva-modules/archiva-base/archiva-consumers/archiva-core-consumers/src/test/java/org/apache/maven/archiva/consumers/core/repository/CleanupReleasedSnapshotsRepositoryPurgeTest.java?rev=1562340&r1=1562339&r2=1562340&view=diff
> ==============================================================================
> --- 
> archiva/branches/archiva-1.3.x/archiva-modules/archiva-base/archiva-consumers/archiva-core-consumers/src/test/java/org/apache/maven/archiva/consumers/core/repository/CleanupReleasedSnapshotsRepositoryPurgeTest.java
>  (original)
> +++ 
> archiva/branches/archiva-1.3.x/archiva-modules/archiva-base/archiva-consumers/archiva-core-consumers/src/test/java/org/apache/maven/archiva/consumers/core/repository/CleanupReleasedSnapshotsRepositoryPurgeTest.java
>  Wed Jan 29 01:34:02 2014
> @@ -137,6 +137,9 @@ public class CleanupReleasedSnapshotsRep
>         listenerControl.replay();
> 
>         File file = new File( repoRoot, INDEX_PATH );
> +        // windows fix
> +        file.mkdirs();
> +        
>         file.createNewFile();
>         assertTrue( file.exists() );
> 
> 
> 

--
Brett Porter   @brettporter
http://brettporter.wordpress.com/
http://au.linkedin.com/in/brettporter

Reply via email to