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());
}