From: Dominik Stadler <centic@apache.org>
Date: Sun, 14 Jul 2024 11:26:38 +0000 (+0000)
Subject: List expected failures in stress.xls instead of fully excluding them
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=148ac23c0d2e17f25d943659b5c5d769f1df9fdb;p=poi.git

List expected failures in stress.xls instead of fully excluding them

We list expected failures in stress.xls to verify that
the error message is correct and no unexpected exception
is introduced

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1919212 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/poi-integration/src/test/java/org/apache/poi/stress/TestAllFiles.java b/poi-integration/src/test/java/org/apache/poi/stress/TestAllFiles.java
index 2f8ba9aafa..2767ca9f72 100644
--- a/poi-integration/src/test/java/org/apache/poi/stress/TestAllFiles.java
+++ b/poi-integration/src/test/java/org/apache/poi/stress/TestAllFiles.java
@@ -97,11 +97,8 @@ public class TestAllFiles {
         "poifs/60320-protected.xlsx",
         "poifs/protected_sha512.xlsx",
 
-        // corrupt file
-        "document/unicode-path.docx",
-        "spreadsheet/duplicate-filename.xlsx",
-        "spreadsheet/duplicate-filename-case-insensitive.xlsx",
-        "document/clusterfuzz-testcase-minimized-POIXWPFFuzzer-5166796835258368.docx",
+        // NOTE: Expected failures should usually be added in file "stress.xls" instead
+        // of being listed here in order to also verify the expected exception details!
     };
 
     // cheap workaround of skipping the few problematic files
diff --git a/test-data/spreadsheet/stress.xls b/test-data/spreadsheet/stress.xls
index f4d52ecf69..04046bd635 100644
Binary files a/test-data/spreadsheet/stress.xls and b/test-data/spreadsheet/stress.xls differ