This is an automated email from the ASF dual-hosted git repository. ggregory pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-vfs.git
The following commit(s) were added to refs/heads/master by this push: new ba112d0 Use try-with-resources. ba112d0 is described below commit ba112d08abcdc0900af61f429336ad24e827ff21 Author: Gary Gregory <ggreg...@rocketsoftware.com> AuthorDate: Thu Jan 31 21:30:51 2019 -0500 Use try-with-resources. --- .../https/test/GetContentInfoFunctionalTest.java | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/https/test/GetContentInfoFunctionalTest.java b/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/https/test/GetContentInfoFunctionalTest.java index febb832..9dd10c7 100644 --- a/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/https/test/GetContentInfoFunctionalTest.java +++ b/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/https/test/GetContentInfoFunctionalTest.java @@ -39,8 +39,9 @@ public class GetContentInfoFunctionalTest { /** * Tests VFS-427 NPE on HttpFileObject.getContent().getContentInfo(). * - * @throws FileSystemException thrown when the getContentInfo API fails. - * @throws MalformedURLException thrown when the System environment contains an invalid URL for an HTTPS proxy. + * @throws FileSystemException thrown when the getContentInfo API fails. + * @throws MalformedURLException thrown when the System environment contains an + * invalid URL for an HTTPS proxy. */ @Test public void testGetContentInfo() throws FileSystemException, MalformedURLException { @@ -65,10 +66,11 @@ public class GetContentInfoFunctionalTest { } final FileSystemManager fsManager = VFS.getManager(); - final FileObject fo = fsManager.resolveFile("http://www.apache.org/licenses/LICENSE-2.0.txt", opts); - 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", opts); + final FileContent content = fo.getContent();) { + Assert.assertNotNull(content); + // Used to NPE before fix: + content.getContentInfo(); + } } }