You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by fa...@apache.org on 2022/05/21 22:38:44 UTC
svn commit: r1901105 - /poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xwpf/usermodel/TestXWPFBugs.java
Author: fanningpj
Date: Sat May 21 22:38:43 2022
New Revision: 1901105
URL: http://svn.apache.org/viewvc?rev=1901105&view=rev
Log:
[bug-66079] apply test provided by Vladislav Arakelov
Modified:
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xwpf/usermodel/TestXWPFBugs.java
Modified: poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xwpf/usermodel/TestXWPFBugs.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xwpf/usermodel/TestXWPFBugs.java?rev=1901105&r1=1901104&r2=1901105&view=diff
==============================================================================
--- poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xwpf/usermodel/TestXWPFBugs.java (original)
+++ poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xwpf/usermodel/TestXWPFBugs.java Sat May 21 22:38:43 2022
@@ -196,23 +196,26 @@ class TestXWPFBugs {
//attempt to remove item with numId 2
assertTrue(numbering.removeAbstractNum(BigInteger.valueOf(2)));
- for (int i = 0; i <= 10; i++) {
- XWPFAbstractNum abstractNum = numbering.getAbstractNum(BigInteger.valueOf(i));
+ XWPFDocument docReloaded = writeOutAndReadBack(doc);
+ XWPFNumbering numberingReloaded = docReloaded.getNumbering();
+
+ for (int id = 0; id <= 10; id++) {
+ XWPFAbstractNum abstractNum = numberingReloaded.getAbstractNum(BigInteger.valueOf(id));
// we removed id "2", so this one should be empty, all others not
- if (i == 2) {
- assertNull(abstractNum, "Failed for " + i);
+ if (id == 2) {
+ assertNull(abstractNum, "Failed for " + id);
} else {
- assertNotNull(abstractNum, "Failed for " + i);
- assertEquals(i, abstractNum.getAbstractNum().getAbstractNumId().longValue());
+ assertNotNull(abstractNum, "Failed for " + id);
+ assertEquals(id, abstractNum.getAbstractNum().getAbstractNumId().longValue());
}
}
// removing the same again fails
- assertFalse(numbering.removeAbstractNum(BigInteger.valueOf(2)));
+ assertFalse(numberingReloaded.removeAbstractNum(BigInteger.valueOf(2)));
// removing another one works
- assertTrue(numbering.removeAbstractNum(BigInteger.valueOf(4)));
+ assertTrue(numberingReloaded.removeAbstractNum(BigInteger.valueOf(4)));
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org