Repository: tika Updated Branches: refs/heads/master 75fa1386b -> aadccbf97
TIKA-1933 -- one more place where we weren't properly closing the ForkParser and were leaving behind a tmp ForkParser jar Project: http://git-wip-us.apache.org/repos/asf/tika/repo Commit: http://git-wip-us.apache.org/repos/asf/tika/commit/aadccbf9 Tree: http://git-wip-us.apache.org/repos/asf/tika/tree/aadccbf9 Diff: http://git-wip-us.apache.org/repos/asf/tika/diff/aadccbf9 Branch: refs/heads/master Commit: aadccbf9719531ee1ddfff19c5e85c433d6930cc Parents: 75fa138 Author: tballison <[email protected]> Authored: Mon Nov 7 21:16:22 2016 -0500 Committer: tballison <[email protected]> Committed: Mon Nov 7 21:16:22 2016 -0500 ---------------------------------------------------------------------- .../org/apache/tika/parser/fork/ForkParserIntegrationTest.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/tika/blob/aadccbf9/tika-parsers/src/test/java/org/apache/tika/parser/fork/ForkParserIntegrationTest.java ---------------------------------------------------------------------- diff --git a/tika-parsers/src/test/java/org/apache/tika/parser/fork/ForkParserIntegrationTest.java b/tika-parsers/src/test/java/org/apache/tika/parser/fork/ForkParserIntegrationTest.java index 54c1427..6a7739c 100644 --- a/tika-parsers/src/test/java/org/apache/tika/parser/fork/ForkParserIntegrationTest.java +++ b/tika-parsers/src/test/java/org/apache/tika/parser/fork/ForkParserIntegrationTest.java @@ -151,7 +151,7 @@ public class ForkParserIntegrationTest { @Test public void testParsingErrorInForkedParserShouldBeReported() throws Exception { BrokenParser brokenParser = new BrokenParser(); - Parser parser = new ForkParser(ForkParser.class.getClassLoader(), brokenParser); + ForkParser parser = new ForkParser(ForkParser.class.getClassLoader(), brokenParser); InputStream stream = getClass().getResourceAsStream("/test-documents/testTXT.txt"); // With a serializable error, we'll get that back @@ -162,6 +162,8 @@ public class ForkParserIntegrationTest { fail("Expected TikaException caused by Error"); } catch (TikaException e) { assertEquals(brokenParser.err, e.getCause()); + } finally { + parser.close(); } // With a non serializable one, we'll get something else
