[jboss-cvs] JBossAS SVN: r69276 - in projects/vfs/trunk/src: test/java/org/jboss/test/virtual/test and 1 other directory.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Thu Jan 24 06:32:41 EST 2008


Author: alesj
Date: 2008-01-24 06:32:40 -0500 (Thu, 24 Jan 2008)
New Revision: 69276

Modified:
   projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/context/file/FileSystemContext.java
   projects/vfs/trunk/src/test/java/org/jboss/test/virtual/test/FileVFSUnitTestCase.java
Log:
Fix FileNotFound with returning null.
Adding test for this behavior.

Modified: projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/context/file/FileSystemContext.java
===================================================================
--- projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/context/file/FileSystemContext.java	2008-01-24 11:22:48 UTC (rev 69275)
+++ projects/vfs/trunk/src/main/java/org/jboss/virtual/plugins/context/file/FileSystemContext.java	2008-01-24 11:32:40 UTC (rev 69276)
@@ -23,7 +23,6 @@
 
 import java.io.File;
 import java.io.FileInputStream;
-import java.io.FileNotFoundException;
 import java.io.IOException;
 import java.net.URI;
 import java.net.URISyntaxException;
@@ -251,8 +250,6 @@
                break;
             }
          }
-         if( handler == null )
-            throw new FileNotFoundException("File does not exist: " + file.getCanonicalPath());
       }
       else if (file.exists())
       {

Modified: projects/vfs/trunk/src/test/java/org/jboss/test/virtual/test/FileVFSUnitTestCase.java
===================================================================
--- projects/vfs/trunk/src/test/java/org/jboss/test/virtual/test/FileVFSUnitTestCase.java	2008-01-24 11:22:48 UTC (rev 69275)
+++ projects/vfs/trunk/src/test/java/org/jboss/test/virtual/test/FileVFSUnitTestCase.java	2008-01-24 11:32:40 UTC (rev 69276)
@@ -509,6 +509,18 @@
       assertFalse("root isDirectory", root.isLeaf());
    }
 
+   public void testFileNotFoundInUnpackedJar()
+      throws Exception
+   {
+      URL rootURL = getResource("/vfs/test");
+      VFS vfs = VFS.getVFS(rootURL);
+
+      // Find the outer.jar
+      VirtualFile outerJar = vfs.findChild("unpacked-outer.jar");
+      assertNotNull("unpacked-outer.jar", outerJar);
+      assertNull(outerJar.getChild("WEB-INF"));
+   }
+
    /**
     * Test file resolution with nested jars
     * @throws Exception




More information about the jboss-cvs-commits mailing list