You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tika.apache.org by tp...@apache.org on 2015/03/31 23:46:21 UTC

svn commit: r1670490 - /tika/trunk/tika-parsers/src/test/java/org/apache/tika/config/TikaParserConfigTest.java

Author: tpalsulich
Date: Tue Mar 31 21:46:21 2015
New Revision: 1670490

URL: http://svn.apache.org/r1670490
Log:
TIKA-1558. Better error message and fix typo.

Modified:
    tika/trunk/tika-parsers/src/test/java/org/apache/tika/config/TikaParserConfigTest.java

Modified: tika/trunk/tika-parsers/src/test/java/org/apache/tika/config/TikaParserConfigTest.java
URL: http://svn.apache.org/viewvc/tika/trunk/tika-parsers/src/test/java/org/apache/tika/config/TikaParserConfigTest.java?rev=1670490&r1=1670489&r2=1670490&view=diff
==============================================================================
--- tika/trunk/tika-parsers/src/test/java/org/apache/tika/config/TikaParserConfigTest.java (original)
+++ tika/trunk/tika-parsers/src/test/java/org/apache/tika/config/TikaParserConfigTest.java Tue Mar 31 21:46:21 2015
@@ -152,14 +152,14 @@ public class TikaParserConfigTest {
         }
         assertTrue("Default config should include an XMLParser.", hasXML);
 
-        // This custom TikaConfig should exclude all AbstractParsers.
+        // This custom TikaConfig should exclude XMLParser and all of its subclasses.
         config = getConfig("TIKA-1558-blacklistsub.xml");
         cp = (CompositeParser) config.getParser();
         parsers = cp.getAllComponentParsers();
 
         for (Parser p : parsers) {
             if (p instanceof XMLParser)
-                fail("Custom config should not include an XMLParser.");
+                fail("Custom config should not include an XMLParser (" + p.getClass() + ").");
         }
     }
 }