You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2013/08/08 03:30:46 UTC

svn commit: r1511553 - in /commons/proper/vfs/trunk: core/src/test/java/org/apache/commons/vfs2/provider/ram/test/CustomRamProviderTest.java src/changes/changes.xml

Author: ggregory
Date: Thu Aug  8 01:30:45 2013
New Revision: 1511553

URL: http://svn.apache.org/r1511553
Log:
[VFS-482] Wrong assertion messages in RAM provider test case.

Modified:
    commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/provider/ram/test/CustomRamProviderTest.java
    commons/proper/vfs/trunk/src/changes/changes.xml

Modified: commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/provider/ram/test/CustomRamProviderTest.java
URL: http://svn.apache.org/viewvc/commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/provider/ram/test/CustomRamProviderTest.java?rev=1511553&r1=1511552&r2=1511553&view=diff
==============================================================================
--- commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/provider/ram/test/CustomRamProviderTest.java (original)
+++ commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/provider/ram/test/CustomRamProviderTest.java Thu Aug  8 01:30:45 2013
@@ -229,8 +229,8 @@ public class CustomRamProviderTest
         // Small FS
         final FileObject fo3 = manager.resolveFile("ram:/fo3", smallSized);
         final FileObject fo4 = manager.resolveFile("ram:/", smallSized);
-        assertTrue("Both files should exist in different fs instances.", fo3.getFileSystem() == fo4.getFileSystem());
-        assertTrue("These file shouldn't be in the same file system.", fo1.getFileSystem() != fo3.getFileSystem());
+        assertTrue("Both files should exist in the same FileSystem instance.", fo3.getFileSystem() == fo4.getFileSystem());
+        assertTrue("Both files should exist in different FileSystem instance.", fo1.getFileSystem() != fo3.getFileSystem());
 
         fo3.createFile();
         try
@@ -240,7 +240,7 @@ public class CustomRamProviderTest
             os.close();
         } catch (final FileSystemException e)
         {
-            fail("It shouldn't save such a small file");
+            fail("Test should be able to save such a small file");
         }
 
         try

Modified: commons/proper/vfs/trunk/src/changes/changes.xml
URL: http://svn.apache.org/viewvc/commons/proper/vfs/trunk/src/changes/changes.xml?rev=1511553&r1=1511552&r2=1511553&view=diff
==============================================================================
--- commons/proper/vfs/trunk/src/changes/changes.xml (original)
+++ commons/proper/vfs/trunk/src/changes/changes.xml Thu Aug  8 01:30:45 2013
@@ -26,6 +26,9 @@
 <!--       <action issue="VFS-443" dev="ggregory" type="update" due-to="nickallen"> -->
 <!--     	[Local] Need an easy way to convert from a FileObject to a File. -->
 <!--       </action> -->
+      <action issue="VFS-482" dev="ggregory" type="update">
+        Wrong assertion messages in RAM provider test case.
+      </action>
       <action issue="VFS-476" dev="ggregory" type="update">
         Update Apache Commons Logging to 1.1.3 from 1.1.2.
       </action>