commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From s...@apache.org
Subject svn commit: r755295 - /commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs/test/NamingTests.java
Date Tue, 17 Mar 2009 16:11:28 GMT
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());
     }
 



Mime
View raw message