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

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


The following commit(s) were added to refs/heads/TIKA-4410 by this push:
     new 76a75ce8a TIKA-4410 -- tweaks
76a75ce8a is described below

commit 76a75ce8a74586f8e621fd3b2396191e90c6a68e
Author: tallison <[email protected]>
AuthorDate: Fri May 30 08:39:32 2025 -0400

    TIKA-4410 -- tweaks
---
 .../tika/parser/microsoft/ooxml/XSSFExcelExtractorDecorator.java      | 1 -
 .../java/org/apache/tika/parser/microsoft/ooxml/OOXMLParserTest.java  | 4 +++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git 
a/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/parser/microsoft/ooxml/XSSFExcelExtractorDecorator.java
 
b/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/parser/microsoft/ooxml/XSSFExcelExtractorDecorator.java
index adc6aefbe..97c629b6a 100644
--- 
a/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/parser/microsoft/ooxml/XSSFExcelExtractorDecorator.java
+++ 
b/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/parser/microsoft/ooxml/XSSFExcelExtractorDecorator.java
@@ -243,7 +243,6 @@ public class XSSFExcelExtractorDecorator extends 
AbstractOOXMLExtractor {
                 XMLReaderUtils.parseSAX(is, new 
CommentPersonHandler(metadata), parseContext);
             }
         }
-
     }
 
     protected void addDrawingHyperLinks(PackagePart sheetPart) {
diff --git 
a/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/test/java/org/apache/tika/parser/microsoft/ooxml/OOXMLParserTest.java
 
b/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/test/java/org/apache/tika/parser/microsoft/ooxml/OOXMLParserTest.java
index b9e80869f..9559e73c2 100644
--- 
a/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/test/java/org/apache/tika/parser/microsoft/ooxml/OOXMLParserTest.java
+++ 
b/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/test/java/org/apache/tika/parser/microsoft/ooxml/OOXMLParserTest.java
@@ -1393,11 +1393,13 @@ public class OOXMLParserTest extends 
MultiThreadedTikaTest {
     public void testMacroinXlsm() throws Exception {
 
         //test default is "don't extract macros"
-        for (Metadata metadata : getRecursiveMetadata("testEXCEL_macro.xlsm")) 
{
+        List<Metadata> metadataList = 
getRecursiveMetadata("testEXCEL_macro.xlsm");
+        for (Metadata metadata : metadataList) {
             if (metadata.get(Metadata.CONTENT_TYPE).equals("text/x-vbasic")) {
                 fail("Shouldn't have extracted macros as default");
             }
         }
+        assertEquals("ThisWorkbook", 
metadataList.get(0).get(Office.WORKBOOK_CODENAME));
 
         //now test that they were extracted
         ParseContext context = new ParseContext();

Reply via email to