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/22 08:43:06 UTC

svn commit: r1901113 - /poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xwpf/usermodel/TestXWPFBugs.java

Author: fanningpj
Date: Sun May 22 08:43:06 2022
New Revision: 1901113

URL: http://svn.apache.org/viewvc?rev=1901113&view=rev
Log:
[bug-66080] XWPFNumbering.addAbstractNum uses abstract numberings count as id

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=1901113&r1=1901112&r2=1901113&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 Sun May 22 08:43:06 2022
@@ -268,6 +268,26 @@ class TestXWPFBugs {
         }
     }
 
+    @Test
+    void test66080() throws IOException {
+        try (XWPFDocument doc = new XWPFDocument()) {
+            XWPFNumbering numbering = doc.createNumbering();
+
+            // Add abstract numbering with id 1
+            addNumberingWithAbstractId(numbering, 1);
+
+            // Add abstract numbering with auto-generated id
+            numbering.addAbstractNum(new XWPFAbstractNum());
+
+            // Check that all abstract numbering ids are unique
+            long uniqueIdCount = numbering
+                    .getAbstractNums().stream()
+                    .map(e -> e.getCTAbstractNum().getAbstractNumId().intValue())
+                    .distinct().count();
+            assertEquals(numbering.getAbstractNums().size(), uniqueIdCount);
+        }
+    }
+
     private static void addNumberingWithAbstractId(XWPFNumbering documentNumbering, int id){
         // create a numbering scheme
         CTAbstractNum cTAbstractNum = CTAbstractNum.Factory.newInstance();



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org