You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2009/03/17 17:11:28 UTC

svn commit: r755295 - /commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs/test/NamingTests.java

Author: sebb
Date: Tue Mar 17 16:11:28 2009
New Revision: 755295

URL: http://svn.apache.org/viewvc?rev=755295&view=rev
Log:
VFS-242 NamingTests - assertSameName invokes relName.replace('\\', '/'); without using return value

Modified:
    commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs/test/NamingTests.java

Modified: commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs/test/NamingTests.java
URL: http://svn.apache.org/viewvc/commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs/test/NamingTests.java?rev=755295&r1=755294&r2=755295&view=diff
==============================================================================
--- commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs/test/NamingTests.java (original)
+++ commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs/test/NamingTests.java Tue Mar 17 16:11:28 2009
@@ -226,14 +226,16 @@
         FileName name = getManager().resolveName(baseName, relName, scope);
         assertEquals(expectedPath, name.getPath());
 
+        String temp;
+        
         // Replace the separators
-        relName.replace('\\', '/');
-        name = getManager().resolveName(baseName, relName, scope);
+        temp = relName.replace('\\', '/');
+        name = getManager().resolveName(baseName, temp, scope);
         assertEquals(expectedPath, name.getPath());
 
         // And again
-        relName.replace('/', '\\');
-        name = getManager().resolveName(baseName, relName, scope);
+        temp = relName.replace('/', '\\');
+        name = getManager().resolveName(baseName, temp, scope);
         assertEquals(expectedPath, name.getPath());
     }