Update the big file test to use POIFSFileSystem.create(File), and tweak javadocs

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1782454 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/src/java/org/apache/poi/poifs/filesystem/POIFSFileSystem.java b/src/java/org/apache/poi/poifs/filesystem/POIFSFileSystem.java
index 84f345504d32c58fbf208b5c6ba27ec7b16246b3..c4ff362721d9e8f674353b643c4bc30aa7e59878 100644 (file)
--- a/src/java/org/apache/poi/poifs/filesystem/POIFSFileSystem.java
+++ b/src/java/org/apache/poi/poifs/filesystem/POIFSFileSystem.java
@@ -27,9 +27,10 @@ import org.apache.poi.util.CloseIgnoringInputStream;
 /**
  * Transition class for the move from {@link POIFSFileSystem} to 
  *  {@link OPOIFSFileSystem}, and from {@link NPOIFSFileSystem} to
- *  {@link POIFSFileSystem}. Currently, this is OPOIFS-powered
+ *  {@link POIFSFileSystem}. 
+ * <p>This has been updated to be powered by the NIO-based NPOIFS
+ *  {@link NPOIFSFileSystem}.
  */
-
 public class POIFSFileSystem
     extends NPOIFSFileSystem // TODO Temporary workaround during #56791
     implements POIFSViewable

diff --git a/src/testcases/org/apache/poi/poifs/filesystem/TestNPOIFSFileSystem.java b/src/testcases/org/apache/poi/poifs/filesystem/TestNPOIFSFileSystem.java
index a4a5846c0b65780393ca128451f654877fc016c9..5da455f1281bc1951b0ecad4445b52308834a3e5 100644 (file)
--- a/src/testcases/org/apache/poi/poifs/filesystem/TestNPOIFSFileSystem.java
+++ b/src/testcases/org/apache/poi/poifs/filesystem/TestNPOIFSFileSystem.java
@@ -1569,8 +1569,7 @@ public final class TestNPOIFSFileSystem {
     * Note that to run this test, you will require 2.5+gb of free
     *  space on your TMP/TEMP partition/disk
     * 
-    * TODO Fix this to work
-    * TODO Update this to use a "create as new file" constructor too
+    * TODO Fix this to work...
     */
    @Test
    @Ignore("Work in progress test for #60670")
@@ -1584,14 +1583,8 @@ public final class TestNPOIFSFileSystem {
        int s512mb = 512*1024*1024;
        DocumentEntry entry;
        
-       // TODO Provide a create method that takes a file
-       
        // Create a just-sub 2gb file
-       NPOIFSFileSystem fs = new NPOIFSFileSystem();
-       fs.writeFilesystem(new FileOutputStream(big));
-       fs.close();
-       
-       fs = new NPOIFSFileSystem(big, false);
+       NPOIFSFileSystem fs = POIFSFileSystem.create(big);
        for (int i=0; i<19; i++) {
            fs.createDocument(new DummyDataInputStream(s100mb), "Entry"+i);
        }
@@ -1627,11 +1620,7 @@ public final class TestNPOIFSFileSystem {
        
        
        // Create a >2gb file
-       fs = new NPOIFSFileSystem();
-       fs.writeFilesystem(new FileOutputStream(big));
-       fs.close();
-       
-       fs = new NPOIFSFileSystem(big, false);
+       fs = POIFSFileSystem.create(big);
        for (int i=0; i<4; i++) {
            fs.createDocument(new DummyDataInputStream(s512mb), "Entry"+i);
        }