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 03e6d30d2..c195488ad 100644 --- 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 shapes = drawing.getShapes(); - str.append("drawing.getShapes().size() = " + shapes.size()); - Iterator 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 index 000000000..1f5599c42 --- /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 shapes = drawing.getShapes(); + str.append("drawing.getShapes().size() = " + shapes.size()); + Iterator 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()); + } +}