You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2010/06/01 11:34:19 UTC

svn commit: r949974 - /directory/apacheds/trunk/xdbm-search/src/test/java/org/apache/directory/server/xdbm/GenericIndexTest.java

Author: elecharny
Date: Tue Jun  1 09:34:19 2010
New Revision: 949974

URL: http://svn.apache.org/viewvc?rev=949974&view=rev
Log:
Fixed another system dependent file creation : using '/' won't work on windows

Modified:
    directory/apacheds/trunk/xdbm-search/src/test/java/org/apache/directory/server/xdbm/GenericIndexTest.java

Modified: directory/apacheds/trunk/xdbm-search/src/test/java/org/apache/directory/server/xdbm/GenericIndexTest.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/xdbm-search/src/test/java/org/apache/directory/server/xdbm/GenericIndexTest.java?rev=949974&r1=949973&r2=949974&view=diff
==============================================================================
--- directory/apacheds/trunk/xdbm-search/src/test/java/org/apache/directory/server/xdbm/GenericIndexTest.java (original)
+++ directory/apacheds/trunk/xdbm-search/src/test/java/org/apache/directory/server/xdbm/GenericIndexTest.java Tue Jun  1 09:34:19 2010
@@ -109,9 +109,12 @@ public class GenericIndexTest
     @Test
     public void testSetGetWkDirPath()
     {
-        index.setWkDirPath( new File( "/zzz" ) );
+        String tmpDir = System.getProperty( "java.io.tmpdir" );
+        String zzzDir = tmpDir + File.separator + "zzz";
+
+        index.setWkDirPath( new File( zzzDir ) );
         assertNotNull( index.getWkDirPath() );
-        assertEquals( "/zzz", index.getWkDirPath().getPath() );
+        assertEquals( zzzDir, index.getWkDirPath().getPath() );
         index.setWkDirPath( null );
         assertNull( index.getWkDirPath() );
     }



Re: svn commit: r949974 - /directory/apacheds/trunk/xdbm-search/src/test/java/org/apache/directory/server/xdbm/GenericIndexTest.java

Posted by Stefan Seelmann <se...@apache.org>.
Author: elecharny
> Date: Tue Jun  1 09:34:19 2010
> New Revision: 949974
>
> URL: http://svn.apache.org/viewvc?rev=949974&view=rev
> Log:
> Fixed another system dependent file creation : using '/' won't work on windows
>   
Sorry, my fault. I didn't consider that this may cause problems on windows.

Kind Regards,
Stefan

> Modified:
>     directory/apacheds/trunk/xdbm-search/src/test/java/org/apache/directory/server/xdbm/GenericIndexTest.java
>
> Modified: directory/apacheds/trunk/xdbm-search/src/test/java/org/apache/directory/server/xdbm/GenericIndexTest.java
> URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/xdbm-search/src/test/java/org/apache/directory/server/xdbm/GenericIndexTest.java?rev=949974&r1=949973&r2=949974&view=diff
> ==============================================================================
> --- directory/apacheds/trunk/xdbm-search/src/test/java/org/apache/directory/server/xdbm/GenericIndexTest.java (original)
> +++ directory/apacheds/trunk/xdbm-search/src/test/java/org/apache/directory/server/xdbm/GenericIndexTest.java Tue Jun  1 09:34:19 2010
> @@ -109,9 +109,12 @@ public class GenericIndexTest
>      @Test
>      public void testSetGetWkDirPath()
>      {
> -        index.setWkDirPath( new File( "/zzz" ) );
> +        String tmpDir = System.getProperty( "java.io.tmpdir" );
> +        String zzzDir = tmpDir + File.separator + "zzz";
> +
> +        index.setWkDirPath( new File( zzzDir ) );
>          assertNotNull( index.getWkDirPath() );
> -        assertEquals( "/zzz", index.getWkDirPath().getPath() );
> +        assertEquals( zzzDir, index.getWkDirPath().getPath() );
>          index.setWkDirPath( null );
>          assertNull( index.getWkDirPath() );
>      }
>
>
>