bug 58325: enable TestXSSFShape unit tests. Thanks to Dominik Stadler for the initial...

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

diff --git a/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestUnfixedBugs.java b/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestUnfixedBugs.java
index 03e6d30d2da033c4b460376b6c598c2783c43c99..c195488adb7138d28d48e282555a183649340ae1 100644 (file)
--- a/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestUnfixedBugs.java
+++ b/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestUnfixedBugs.java
@@ -30,8 +30,6 @@ import java.io.OutputStream;
 import java.nio.charset.Charset;
 import java.util.Calendar;
 import java.util.Date;
-import java.util.Iterator;
-import java.util.List;
 
 import org.apache.poi.hssf.HSSFTestDataSamples;
 import org.apache.poi.ss.usermodel.Cell;
@@ -361,40 +359,4 @@ public final class TestUnfixedBugs {
        assertEquals("Did not have expected contents at rownum " + rowNum, 
                contents + ".0", cell.toString());
    }
-
-   @Test
-   public void test58325_one() {
-       check58325(XSSFTestDataSamples.openSampleWorkbook("58325_lt.xlsx"), 1);
-   }
-
-   @Test
-   public void test58325_three() {
-       check58325(XSSFTestDataSamples.openSampleWorkbook("58325_db.xlsx"), 3);
-   }
-
-   private void check58325(XSSFWorkbook wb, int expectedShapes) {
-       XSSFSheet sheet = wb.getSheet("MetasNM001");
-       assertNotNull(sheet);
-
-       StringBuilder str = new StringBuilder();
-       str.append("sheet " + sheet.getSheetName() + " - ");
-
-       XSSFDrawing drawing = sheet.getDrawingPatriarch();
-       //drawing = ((XSSFSheet)sheet).createDrawingPatriarch();
-
-       List<XSSFShape> shapes = drawing.getShapes();
-       str.append("drawing.getShapes().size() = " + shapes.size());
-       Iterator<XSSFShape> it = shapes.iterator();
-       while(it.hasNext()) {           
-           XSSFShape shape = it.next();
-           str.append(", " + shape);
-           str.append(", Col1:"+((XSSFClientAnchor)shape.getAnchor()).getCol1());
-           str.append(", Col2:"+((XSSFClientAnchor)shape.getAnchor()).getCol2());
-           str.append(", Row1:"+((XSSFClientAnchor)shape.getAnchor()).getRow1());
-           str.append(", Row2:"+((XSSFClientAnchor)shape.getAnchor()).getRow2());
-       }
-       
-       assertEquals("Having shapes: " + str, 
-               expectedShapes, shapes.size());
-   }
 }

diff --git a/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFShape.java b/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFShape.java
new file mode 100644 (file)
index 0000000..1f5599c
--- /dev/null
+++ b/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFShape.java
@@ -0,0 +1,69 @@
+/* ====================================================================
+   Licensed to the Apache Software Foundation (ASF) under one or more
+   contributor license agreements.  See the NOTICE file distributed with
+   this work for additional information regarding copyright ownership.
+   The ASF licenses this file to You under the Apache License, Version 2.0
+   (the "License"); you may not use this file except in compliance with
+   the License.  You may obtain a copy of the License at
+
+       http://www.apache.org/licenses/LICENSE-2.0
+
+   Unless required by applicable law or agreed to in writing, software
+   distributed under the License is distributed on an "AS IS" BASIS,
+   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+   See the License for the specific language governing permissions and
+   limitations under the License.
+==================================================================== */
+
+package org.apache.poi.xssf.usermodel;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
+
+import java.util.Iterator;
+import java.util.List;
+
+import org.apache.poi.xssf.XSSFTestDataSamples;
+import org.junit.Test;
+
+/**
+ * Tests for XSSFShape
+ */
+public final class TestXSSFShape {
+
+    @Test
+    public void test58325_one() {
+        check58325(XSSFTestDataSamples.openSampleWorkbook("58325_lt.xlsx"), 1);
+    }
+
+    @Test
+    public void test58325_three() {
+        check58325(XSSFTestDataSamples.openSampleWorkbook("58325_db.xlsx"), 3);
+    }
+
+    private void check58325(XSSFWorkbook wb, int expectedShapes) {
+        XSSFSheet sheet = wb.getSheet("MetasNM001");
+        assertNotNull(sheet);
+
+        StringBuilder str = new StringBuilder();
+        str.append("sheet " + sheet.getSheetName() + " - ");
+
+        XSSFDrawing drawing = sheet.getDrawingPatriarch();
+        //drawing = ((XSSFSheet)sheet).createDrawingPatriarch();
+
+        List<XSSFShape> shapes = drawing.getShapes();
+        str.append("drawing.getShapes().size() = " + shapes.size());
+        Iterator<XSSFShape> it = shapes.iterator();
+        while(it.hasNext()) {           
+            XSSFShape shape = it.next();
+            str.append(", " + shape);
+            str.append(", Col1:"+((XSSFClientAnchor)shape.getAnchor()).getCol1());
+            str.append(", Col2:"+((XSSFClientAnchor)shape.getAnchor()).getCol2());
+            str.append(", Row1:"+((XSSFClientAnchor)shape.getAnchor()).getRow1());
+            str.append(", Row2:"+((XSSFClientAnchor)shape.getAnchor()).getRow2());
+        }
+        
+        assertEquals("Having shapes: " + str, 
+                expectedShapes, shapes.size());
+    }
+}