Author: ggregory
Date: Thu Aug  8 17:53:05 2013
New Revision: 1511900

URL: http://svn.apache.org/r1511900
Log:
[VFS-483][RAM] Many suggestions to improve the RAM file provider. Use long for 
FS size limit. Apply test patch 
https://issues.apache.org/jira/secure/attachment/12596890/unittest-fsoption.patch.
 Thanks to Bernd Eckenfels.

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

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=1511900&r1=1511899&r2=1511900&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 17:53:05 2013
@@ -219,19 +219,33 @@ public class CustomRamProviderTest
     }
 
     @Test
-    public void testSmallFS() throws Exception
+    public void testFSOptions() throws Exception
     {
         // Default FS
         final FileObject fo1 = manager.resolveFile("ram:/");
         final FileObject fo2 = manager.resolveFile("ram:/");
         assertTrue("Both files should exist in the same fs instance.", 
fo1.getFileSystem() == fo2.getFileSystem());
 
+        FileSystemOptions fsOptions = 
fo1.getFileSystem().getFileSystemOptions();
+        long maxFilesystemSize = 
RamFileSystemConfigBuilder.getInstance().getLongMaxSize(fsOptions);
+        assertEquals("Filesystem option maxSize must be unlimited", 
Long.MAX_VALUE, maxFilesystemSize);
+
         // Small FS
         final FileObject fo3 = manager.resolveFile("ram:/fo3", smallSizedFso);
         final FileObject fo4 = manager.resolveFile("ram:/", smallSizedFso);
         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());
 
+        fsOptions = fo3.getFileSystem().getFileSystemOptions();
+        maxFilesystemSize = 
RamFileSystemConfigBuilder.getInstance().getLongMaxSize(fsOptions);
+        assertEquals("Filesystem option maxSize must be set", 10, 
maxFilesystemSize);
+    }
+
+    @Test
+    public void testSmallFS() throws Exception
+    {
+        // Small FS
+        final FileObject fo3 = manager.resolveFile("ram:/fo3", smallSizedFso);
         fo3.createFile();
         try
         {


Reply via email to