Close FileObject resources in HTTP tests. Trying to fix build on Linux. Tests already all pass on Windows 10 and macOS.

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/vfs/trunk@1852381 13f79535-47bb-0310-9956-ffa450edef68

commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/http/test/GetContentInfoFunctionalTest.java

diff --git a/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/http/test/GetContentInfoFunctionalTest.java b/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/http/test/GetContentInfoFunctionalTest.java
index 4cfded3..91a0fea 100644
--- a/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/http/test/GetContentInfoFunctionalTest.java
+++ b/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/http/test/GetContentInfoFunctionalTest.java
@@ -39,10 +39,11 @@
     @Test
     public void testGetContentInfo() throws FileSystemException {
         final FileSystemManager fsManager = VFS.getManager();
-        final FileObject fo = fsManager.resolveFile("http://www.apache.org/licenses/LICENSE-2.0.txt");
-        final FileContent content = fo.getContent();
-        Assert.assertNotNull(content);
-        // Used to NPE before fix:
-        content.getContentInfo();
+        try (final FileObject fo = fsManager.resolveFile("http://www.apache.org/licenses/LICENSE-2.0.txt");
+                final FileContent content = fo.getContent();) {
+            Assert.assertNotNull(content);
+            // Used to NPE before fix:
+            content.getContentInfo();
+        }
     }
 }

commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/http/test/HttpFilesCacheTestCase.java

diff --git a/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/http/test/HttpFilesCacheTestCase.java b/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/http/test/HttpFilesCacheTestCase.java
index 2ca498f..a6024ef 100644
--- a/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/http/test/HttpFilesCacheTestCase.java
+++ b/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/http/test/HttpFilesCacheTestCase.java
@@ -28,7 +28,8 @@
 /**
  * Tests https://issues.apache.org/jira/browse/VFS-426
  *
- * @version $Id$
+ * @version $Id: HttpFilesCacheTestCase.java 1808381 2017-09-14 19:26:39Z
+ *          ggregory $
  * @since 2.1
  */
 public class HttpFilesCacheTestCase extends TestCase {
@@ -44,13 +45,16 @@
 
         final FileSystemManager fileSystemManager = VFS.getManager();
 
-        final FileObject noQueryFile = fileSystemManager.resolveFile(noQueryStringUrl);
-        Assert.assertEquals(noQueryStringUrl, noQueryFile.getURL().toExternalForm());
+        try (final FileObject noQueryFile = fileSystemManager.resolveFile(noQueryStringUrl)) {
+            Assert.assertEquals(noQueryStringUrl, noQueryFile.getURL().toExternalForm());
+        }
 
-        final FileObject queryFile = fileSystemManager.resolveFile(queryStringUrl);
-        Assert.assertEquals(queryStringUrl, queryFile.getURL().toExternalForm()); // failed for VFS-426
+        try (final FileObject queryFile = fileSystemManager.resolveFile(queryStringUrl)) {
+            Assert.assertEquals(queryStringUrl, queryFile.getURL().toExternalForm()); // failed for VFS-426
+        }
 
-        final FileObject queryFile2 = fileSystemManager.resolveFile(queryStringUrl2);
-        Assert.assertEquals(queryStringUrl2, queryFile2.getURL().toExternalForm()); // failed for VFS-426
+        try (final FileObject queryFile2 = fileSystemManager.resolveFile(queryStringUrl2)) {
+            Assert.assertEquals(queryStringUrl2, queryFile2.getURL().toExternalForm()); // failed for VFS-426
+        }
     }
 }

commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/http/test/HttpProviderTestCase.java

diff --git a/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/http/test/HttpProviderTestCase.java b/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/http/test/HttpProviderTestCase.java
index 6995195..73a5962 100644
--- a/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/http/test/HttpProviderTestCase.java
+++ b/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/http/test/HttpProviderTestCase.java
@@ -150,8 +150,7 @@
         VFS.getManager().getFilesCache().close();
         final FileSystemOptions opts = new FileSystemOptions();
         HttpFileSystemConfigBuilder.getInstance().setFollowRedirect(opts, followRedirect);
-        final FileObject file = VFS.getManager().resolveFile(uri, opts);
-        try {
+        try (final FileObject file = VFS.getManager().resolveFile(uri, opts)) {
             checkReadTestsFolder(file);
         } catch (final FileNotFolderException e) {
             // Expected: VFS HTTP does not support listing children yet.
@@ -176,9 +175,10 @@
 
     // Test no longer passing 2016/04/28
     public void ignoreTestHttp405() throws FileSystemException {
-        final FileObject f = VFS.getManager()
-                .resolveFile("http://www.w3schools.com/webservices/tempconvert.asmx?action=WSDL");
-        assert f.getContent().getSize() > 0;
+        try (final FileObject fileObject = VFS.getManager()
+                .resolveFile("http://www.w3schools.com/webservices/tempconvert.asmx?action=WSDL")) {
+            assert fileObject.getContent().getSize() > 0;
+        }
     }
 
     /** Ensure VFS-453 options are present. */