Author: centic
Date: Sat Jan 11 10:09:01 2025
New Revision: 1923063

URL: http://svn.apache.org/viewvc?rev=1923063&view=rev
Log:
Bug 58805: Adjust reproducer test-case some more

Do not write a local temp-file
It seems even parsing the resulting document fails

Modified:
    
poi/trunk/poi-scratchpad/src/test/java/org/apache/poi/hwpf/usermodel/TestBugs.java

Modified: 
poi/trunk/poi-scratchpad/src/test/java/org/apache/poi/hwpf/usermodel/TestBugs.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/poi-scratchpad/src/test/java/org/apache/poi/hwpf/usermodel/TestBugs.java?rev=1923063&r1=1923062&r2=1923063&view=diff
==============================================================================
--- 
poi/trunk/poi-scratchpad/src/test/java/org/apache/poi/hwpf/usermodel/TestBugs.java
 (original)
+++ 
poi/trunk/poi-scratchpad/src/test/java/org/apache/poi/hwpf/usermodel/TestBugs.java
 Sat Jan 11 10:09:01 2025
@@ -23,12 +23,12 @@ import static org.apache.poi.hwpf.HWPFTe
 import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
+import java.io.ByteArrayInputStream;
 import java.io.ByteArrayOutputStream;
 import java.io.File;
-import java.io.FileInputStream;
-import java.io.FileOutputStream;
 import java.io.IOException;
 import java.io.InputStream;
 import java.nio.charset.StandardCharsets;
@@ -37,6 +37,8 @@ import java.util.Collection;
 import java.util.List;
 
 import org.apache.commons.codec.digest.DigestUtils;
+import org.apache.poi.extractor.ExtractorFactory;
+import org.apache.poi.extractor.POITextExtractor;
 import org.apache.poi.hwpf.HWPFDocument;
 import org.apache.poi.hwpf.HWPFOldDocument;
 import org.apache.poi.hwpf.HWPFTestDataSamples;
@@ -798,9 +800,14 @@ class TestBugs {
                 run.replaceText("_TEST_", "This text is longer than the 
initial text. It goes on and on without interruption.");
             }
 
-            try (FileOutputStream fos = new FileOutputStream(new 
File("/tmp/test.doc"))) {
-                doc.write(fos);
-            }
+            ByteArrayOutputStream out = new ByteArrayOutputStream();
+            doc.write(out);
+            out.flush();
+
+            POITextExtractor extractor = ExtractorFactory.createExtractor(new 
ByteArrayInputStream(out.toByteArray()));
+            assertThrows(IllegalArgumentException.class,
+                    () -> /*String text =*/ extractor.getText());
+            // assertFalse(text.contains("_TEST_"), "Had: " + text);
         }
     }
 }



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to