]> source.dussan.org Git - poi.git/commitdiff
Fix close warnings
authorNick Burch <nick@apache.org>
Tue, 25 Oct 2016 10:24:48 +0000 (10:24 +0000)
committerNick Burch <nick@apache.org>
Tue, 25 Oct 2016 10:24:48 +0000 (10:24 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1766503 13f79535-47bb-0310-9956-ffa450edef68

src/scratchpad/testcases/org/apache/poi/hdgf/TestHDGFCore.java

index e87b8ed8b8b360a99f70b285a7e266a2abacf0cc..ad3888d19e9ab547315807f4ae90551b388f1190 100644 (file)
@@ -29,19 +29,26 @@ public final class TestHDGFCore extends TestCase {
     private static POIDataSamples _dgTests = POIDataSamples.getDiagramInstance();
 
     private POIFSFileSystem fs;
+    private HDGFDiagram hdgf;
+    private VisioTextExtractor textExtractor;
 
     @Override
     protected void setUp() throws Exception {
         fs = new POIFSFileSystem(_dgTests.openResourceAsStream("Test_Visio-Some_Random_Text.vsd"));
     }
-    
+    @Override
+    protected void tearDown() throws Exception {
+        if (textExtractor != null) textExtractor.close();
+        if (hdgf != null) hdgf.close();
+    }
+
 
     public void testCreate() throws Exception {
-        new HDGFDiagram(fs);
+        hdgf = new HDGFDiagram(fs);
     }
 
     public void testTrailer() throws Exception {
-        HDGFDiagram hdgf = new HDGFDiagram(fs);
+        hdgf = new HDGFDiagram(fs);
         assertNotNull(hdgf);
         assertNotNull(hdgf.getTrailerStream());
 
@@ -70,7 +77,7 @@ public final class TestHDGFCore extends TestCase {
     public void testNegativeChunkLength() throws Exception {
         fs = new POIFSFileSystem(_dgTests.openResourceAsStream("NegativeChunkLength.vsd"));
 
-        HDGFDiagram hdgf = new HDGFDiagram(fs);
+        hdgf = new HDGFDiagram(fs);
         assertNotNull(hdgf);
 
         // And another file
@@ -88,17 +95,17 @@ public final class TestHDGFCore extends TestCase {
     public void DISABLEDtestAIOOB() throws Exception {
         fs = new POIFSFileSystem(_dgTests.openResourceAsStream("44501.vsd"));
 
-        HDGFDiagram hdgf = new HDGFDiagram(fs);
+        hdgf = new HDGFDiagram(fs);
         assertNotNull(hdgf);
     }
 
     public void testV5() throws Exception {
         fs = new POIFSFileSystem(_dgTests.openResourceAsStream("v5_Connection_Types.vsd"));
 
-        HDGFDiagram hdgf = new HDGFDiagram(fs);
+        hdgf = new HDGFDiagram(fs);
         assertNotNull(hdgf);
 
-        VisioTextExtractor textExtractor = new VisioTextExtractor(hdgf);
+        textExtractor = new VisioTextExtractor(hdgf);
         String text = textExtractor.getText().replace("\u0000", "").trim();
 
         assertEquals("Static to Static\nDynamic to Static\nDynamic to Dynamic", text);
@@ -107,10 +114,10 @@ public final class TestHDGFCore extends TestCase {
     public void testV6NonUtf16LE() throws Exception {
         fs = new POIFSFileSystem(_dgTests.openResourceAsStream("v6-non-utf16le.vsd"));
 
-        HDGFDiagram hdgf = new HDGFDiagram(fs);
+        hdgf = new HDGFDiagram(fs);
         assertNotNull(hdgf);
 
-        VisioTextExtractor textExtractor = new VisioTextExtractor(hdgf);
+        textExtractor = new VisioTextExtractor(hdgf);
         String text = textExtractor.getText().replace("\u0000", "").trim();
 
         assertEquals("Table\n\n\nPropertySheet\n\n\n\nPropertySheetField", text);
@@ -119,10 +126,10 @@ public final class TestHDGFCore extends TestCase {
     public void testUtf16LE() throws Exception {
         fs = new POIFSFileSystem(_dgTests.openResourceAsStream("Test_Visio-Some_Random_Text.vsd"));
 
-        HDGFDiagram hdgf = new HDGFDiagram(fs);
+        hdgf = new HDGFDiagram(fs);
         assertNotNull(hdgf);
 
-        VisioTextExtractor textExtractor = new VisioTextExtractor(hdgf);
+        textExtractor = new VisioTextExtractor(hdgf);
         String text = textExtractor.getText().trim();
 
         assertEquals("text\nView\nTest View\nI am a test view\nSome random text, on a page", text);