Author: fanningpj
Date: Tue Mar 22 17:55:53 2022
New Revision: 1899138

URL: http://svn.apache.org/viewvc?rev=1899138&view=rev
Log:
[bug-65973] partial fix for removing hyperlinks that span multiple cells

Modified:
    
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xssf/usermodel/TestXSSFSheet.java

Modified: 
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xssf/usermodel/TestXSSFSheet.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xssf/usermodel/TestXSSFSheet.java?rev=1899138&r1=1899137&r2=1899138&view=diff
==============================================================================
--- 
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xssf/usermodel/TestXSSFSheet.java
 (original)
+++ 
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xssf/usermodel/TestXSSFSheet.java
 Tue Mar 22 17:55:53 2022
@@ -2288,7 +2288,7 @@ public final class TestXSSFSheet extends
     }
 
     @Test
-    void testRowShiftWithHyperlink() throws IOException {
+    void testRowShiftWithHyperlink1() throws IOException {
         try (XSSFWorkbook wb = createWorkbookForRowShiftWithHyperlink1(true)) {
             XSSFSheet sheet = wb.getSheetAt(0);
             List<XSSFHyperlink> hyperlinks = sheet.getHyperlinkList();
@@ -2297,6 +2297,25 @@ public final class TestXSSFSheet extends
             assertEquals(3, sheet.getLastRowNum());
 
             sheet.shiftRows(2, 3, -2);
+            assertEquals(1, sheet.getLastRowNum());
+            XSSFRow row0 = sheet.getRow(0);
+            XSSFRow row1 = sheet.getRow(1);
+            assertEquals("row2", row0.getCell(0).getStringCellValue());
+            assertEquals("row3", row1.getCell(0).getStringCellValue());
+            assertEquals(0, sheet.getHyperlinkList().size());
+        }
+    }
+
+    @Test
+    void testRowShiftWithHyperlink2() throws IOException {
+        try (XSSFWorkbook wb = createWorkbookForRowShiftWithHyperlink1(false)) 
{
+            XSSFSheet sheet = wb.getSheetAt(0);
+            List<XSSFHyperlink> hyperlinks = sheet.getHyperlinkList();
+            assertEquals(1, hyperlinks.size());
+            assertEquals("B1:B2", hyperlinks.get(0).getCellRef());
+            assertEquals(3, sheet.getLastRowNum());
+
+            sheet.shiftRows(2, 3, -2);
             assertEquals(1, sheet.getLastRowNum());
             XSSFRow row0 = sheet.getRow(0);
             XSSFRow row1 = sheet.getRow(1);



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to