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

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

commit 7ec0dc07f360b431569e87f237b3cae3619d1568
Author: TALLISON <[email protected]>
AuthorDate: Tue May 14 08:36:58 2019 -0400

    TIKA-2871 -- don't include testChm_oom.chm in multithreaded tests.
---
 .../test/java/org/apache/tika/parser/chm/TestChmExtraction.java   | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git 
a/tika-parsers/src/test/java/org/apache/tika/parser/chm/TestChmExtraction.java 
b/tika-parsers/src/test/java/org/apache/tika/parser/chm/TestChmExtraction.java
index c49a006..eab657e 100644
--- 
a/tika-parsers/src/test/java/org/apache/tika/parser/chm/TestChmExtraction.java
+++ 
b/tika-parsers/src/test/java/org/apache/tika/parser/chm/TestChmExtraction.java
@@ -229,7 +229,13 @@ public class TestChmExtraction extends 
MultiThreadedTikaTest {
                             //this file is a beast, skip it
                             if (pathname.getName().equals("testChm2.chm")) {
                                 return false;
-                            } else {
+                                //this file throws an exception in the 
baseline and then
+                                //isn't included in the actual tests.
+                                //If we do want to include it we need to 
change the way
+                                //MultiThreadedTikaTest handles files that 
throw exceptions
+                            } else if 
(pathname.getName().equals("testChm_oom.chm")) {
+                                return false;
+                            } else{
                                 return true;
                             }
                         } else {

Reply via email to