You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2026/10/18 09:42:07 UTC

svn commit: r1939231 - pdfbox/branches/3.0/pdfbox/src/test/java/org/apache/pdfbox/multipdf

Author: tilman
Date: Sun Oct 18 09:42:07 2026
New Revision: 1939231

Log:
PDFBOX-6283: improve coverage

Modified:
   pdfbox/branches/3.0/pdfbox/src/test/java/org/apache/pdfbox/multipdf/PDFMergerUtilityTest.java

Modified: pdfbox/branches/3.0/pdfbox/src/test/java/org/apache/pdfbox/multipdf/PDFMergerUtilityTest.java
==============================================================================
--- pdfbox/branches/3.0/pdfbox/src/test/java/org/apache/pdfbox/multipdf/PDFMergerUtilityTest.java	Sun Oct 18 09:28:44 2026	(r1939230)
+++ pdfbox/branches/3.0/pdfbox/src/test/java/org/apache/pdfbox/multipdf/PDFMergerUtilityTest.java	Sun Oct 18 09:42:07 2026	(r1939231)
@@ -1283,6 +1283,20 @@ class PDFMergerUtilityTest
             assertNotEquals(-1, pageTree.indexOf(page), "Page is not in the page tree");
         }
     }
+    
+    @Test
+    void testIllegalSplitting()
+    {
+        Splitter splitter = new Splitter();
+        IllegalArgumentException ex = assertThrows(IllegalArgumentException.class, () -> splitter.setSplitAtPage(0));
+        assertEquals("Number of pages is smaller than one", ex.getMessage());
+        ex = assertThrows(IllegalArgumentException.class, () -> splitter.setStartPage(0));
+        assertEquals("Start page is smaller than one", ex.getMessage());
+        ex = assertThrows(IllegalArgumentException.class, () -> splitter.setEndPage(0));
+        assertEquals("End page is smaller than one", ex.getMessage());
+        ex = assertThrows(IllegalArgumentException.class, () -> { splitter.setStartPage(2); splitter.setEndPage(1); });
+        assertEquals("End page is smaller than startPage", ex.getMessage());
+    }
 
     @Test
     void testSplitWithStructureTree() throws IOException