From: Dominik Stadler <centic@apache.org>
Date: Wed, 6 Dec 2023 19:49:56 +0000 (+0000)
Subject: Add more output for a failure that we saw once in CI
X-Git-Tag: REL_5_3_0~158
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=25e2d5ca691153d906c9451ffd95da09c22609f0;p=poi.git

Add more output for a failure that we saw once in CI

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

diff --git a/poi-ooxml/src/test/java/org/apache/poi/poifs/crypt/dsig/TestSignatureInfo.java b/poi-ooxml/src/test/java/org/apache/poi/poifs/crypt/dsig/TestSignatureInfo.java
index 932195193e..3a7e4eb5ed 100644
--- a/poi-ooxml/src/test/java/org/apache/poi/poifs/crypt/dsig/TestSignatureInfo.java
+++ b/poi-ooxml/src/test/java/org/apache/poi/poifs/crypt/dsig/TestSignatureInfo.java
@@ -252,7 +252,11 @@ class TestSignatureInfo {
             si.setOpcPackage(pkg);
             si.setSignatureConfig(sic);
             boolean isValid = si.verifySignature();
-            assertTrue(isValid);
+
+            assertTrue(isValid,
+                    // add some details to find out why "verifySignature()" returns false sometimes
+                    "Verifying signature failed, hasNext: " + si.getSignatureParts().iterator().hasNext() + ": " +
+                            (si.getSignatureParts().iterator().hasNext() ? si.getSignatureParts().iterator() : ""));
         }
     }