]> source.dussan.org Git - poi.git/commitdiff
Do not fail test which compares two documents if the timestamp in the ZIP header...
authorDominik Stadler <centic@apache.org>
Mon, 15 Feb 2021 15:23:49 +0000 (15:23 +0000)
committerDominik Stadler <centic@apache.org>
Mon, 15 Feb 2021 15:23:49 +0000 (15:23 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1886532 13f79535-47bb-0310-9956-ffa450edef68

src/testcases/org/apache/poi/ss/usermodel/BaseTestBugzillaIssues.java

index c79abea7dc43465428c248743b8cc1c635e6bea6..a82a2a86818e11e1d030a846d1994dc2dce01db7 100644 (file)
@@ -39,6 +39,7 @@ import java.util.List;
 import java.util.Map;
 
 import org.apache.poi.hssf.usermodel.HSSFWorkbook;
+import org.apache.poi.poifs.filesystem.FileMagic;
 import org.apache.poi.ss.ITestDataProvider;
 import org.apache.poi.ss.SpreadsheetVersion;
 import org.apache.poi.ss.formula.FormulaParseException;
@@ -1827,8 +1828,25 @@ public abstract class BaseTestBugzillaIssues {
                 out1.flush();
                 out2.flush();
 
-                assertArrayEquals(out1.toByteArray(), out2.toByteArray());
+                // to avoid flaky tests if the documents are written at slightly different timestamps
+                // we clear some bytes which contain timestamps
+                assertArrayEquals(
+                        removeTimestamp(out1.toByteArray()),
+                        removeTimestamp(out2.toByteArray()));
             }
         }
     }
-}
\ No newline at end of file
+
+    private byte[] removeTimestamp(byte[] bytes) {
+        if (FileMagic.valueOf(bytes) == FileMagic.OOXML) {
+            // This removes the timestamp in the header of the ZIP-Format
+            // see "Local file header" at https://pkware.cachefly.net/webdocs/casestudies/APPNOTE.TXT
+            bytes[10] = 0;
+            bytes[11] = 0;
+            bytes[12] = 0;
+            bytes[13] = 0;
+        }
+
+        return bytes;
+    }
+}