You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tika.apache.org by ta...@apache.org on 2021/04/13 16:31:12 UTC

[tika] branch main updated: TIKA-3351 -- prevent multiple "parsed by" entries...might need to revert this approach...

This is an automated email from the ASF dual-hosted git repository.

tallison pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tika.git


The following commit(s) were added to refs/heads/main by this push:
     new df38ce5  TIKA-3351 -- prevent multiple "parsed by" entries...might need to revert this approach...
df38ce5 is described below

commit df38ce577f1bc4b6b70ff1d17c717851ca955a0f
Author: tballison <ta...@apache.org>
AuthorDate: Tue Apr 13 12:31:02 2021 -0400

    TIKA-3351 -- prevent multiple "parsed by" entries...might need to revert this approach...
---
 .../apache/tika/parser/multiple/MultipleParserTest.java   | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

diff --git a/tika-core/src/test/java/org/apache/tika/parser/multiple/MultipleParserTest.java b/tika-core/src/test/java/org/apache/tika/parser/multiple/MultipleParserTest.java
index ffe5442..df78443 100644
--- a/tika-core/src/test/java/org/apache/tika/parser/multiple/MultipleParserTest.java
+++ b/tika-core/src/test/java/org/apache/tika/parser/multiple/MultipleParserTest.java
@@ -196,11 +196,10 @@ public class MultipleParserTest {
         assertEquals("Test1", metadata.get("TBoth"));
 
         usedParsers = metadata.getValues(TikaCoreProperties.TIKA_PARSED_BY);
-        assertEquals(4, usedParsers.length);
+        assertEquals(3, usedParsers.length);
         assertEquals(ErrorParser.class.getName(), usedParsers[0]);
         assertEquals(DummyParser.class.getName(), usedParsers[1]);
-        assertEquals(DummyParser.class.getName(), usedParsers[2]);
-        assertEquals(EmptyParser.class.getName(), usedParsers[3]);
+        assertEquals(EmptyParser.class.getName(), usedParsers[2]);
 
 
         // Last Wins
@@ -217,11 +216,10 @@ public class MultipleParserTest {
         assertEquals("Test2", metadata.get("TBoth"));
 
         usedParsers = metadata.getValues(TikaCoreProperties.TIKA_PARSED_BY);
-        assertEquals(4, usedParsers.length);
+        assertEquals(3, usedParsers.length);
         assertEquals(ErrorParser.class.getName(), usedParsers[0]);
         assertEquals(DummyParser.class.getName(), usedParsers[1]);
-        assertEquals(DummyParser.class.getName(), usedParsers[2]);
-        assertEquals(EmptyParser.class.getName(), usedParsers[3]);
+        assertEquals(EmptyParser.class.getName(), usedParsers[2]);
 
 
         // Merge
@@ -240,11 +238,10 @@ public class MultipleParserTest {
         assertEquals("Test2", metadata.getValues("TBoth")[1]);
 
         usedParsers = metadata.getValues(TikaCoreProperties.TIKA_PARSED_BY);
-        assertEquals(4, usedParsers.length);
+        assertEquals(3, usedParsers.length);
         assertEquals(ErrorParser.class.getName(), usedParsers[0]);
         assertEquals(DummyParser.class.getName(), usedParsers[1]);
-        assertEquals(DummyParser.class.getName(), usedParsers[2]);
-        assertEquals(EmptyParser.class.getName(), usedParsers[3]);
+        assertEquals(EmptyParser.class.getName(), usedParsers[2]);
 
 
         // Check the error details always come through, no matter the policy