added javadoc and few other improvements

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

2 files changed, 4 insertions, 17 deletions

diff --git a/src/testcases/org/apache/poi/hssf/model/TestDrawingShapes.java b/src/testcases/org/apache/poi/hssf/model/TestDrawingShapes.java
index 6a1638335d..dc3711f507 100644
--- a/src/testcases/org/apache/poi/hssf/model/TestDrawingShapes.java
+++ b/src/testcases/org/apache/poi/hssf/model/TestDrawingShapes.java
@@ -412,19 +412,6 @@ public class TestDrawingShapes extends TestCase {
         assertEquals(agg.getShapeToObjMapping().size(), 2);

     }

 

-    public void testComboboxRecords(){

-        HSSFWorkbook wb = new HSSFWorkbook();

-        HSSFSheet sheet = wb.createSheet();

-        HSSFPatriarch patriarch = sheet.createDrawingPatriarch();

-

-        HSSFCombobox combobox = new HSSFCombobox(null, new HSSFClientAnchor());

-        HSSFTestHelper.setShapeId(combobox, 1024);

-        ComboboxShape comboboxShape = new ComboboxShape(combobox, 1024);

-

-        assertTrue(Arrays.equals(comboboxShape.getSpContainer().serialize(), HSSFTestHelper.getEscherContainer(combobox).serialize()));

-        assertTrue(Arrays.equals(comboboxShape.getObjRecord().serialize(), HSSFTestHelper.getObjRecord(combobox).serialize()));

-    }

-

     public void testRemoveShapes(){

         HSSFWorkbook wb = new HSSFWorkbook();

         HSSFSheet sheet = wb.createSheet();

diff --git a/src/testcases/org/apache/poi/hssf/model/TestHSSFAnchor.java b/src/testcases/org/apache/poi/hssf/model/TestHSSFAnchor.java
index 868848a0ed..bda65884cd 100644
--- a/src/testcases/org/apache/poi/hssf/model/TestHSSFAnchor.java
+++ b/src/testcases/org/apache/poi/hssf/model/TestHSSFAnchor.java
@@ -151,9 +151,9 @@ public class TestHSSFAnchor extends TestCase {
 

         rectangle.setAnchor(anchor);

 

-        assertNotNull(anchor.getEscherAnchor());

+        assertNotNull(HSSFTestHelper.getEscherAnchor(anchor));

         assertNotNull(HSSFTestHelper.getEscherContainer(rectangle));

-        assertTrue(anchor.getEscherAnchor().equals(HSSFTestHelper.getEscherContainer(rectangle).getChildById(EscherClientAnchorRecord.RECORD_ID)));

+        assertTrue(HSSFTestHelper.getEscherAnchor(anchor).equals(HSSFTestHelper.getEscherContainer(rectangle).getChildById(EscherClientAnchorRecord.RECORD_ID)));

     }

 

     public void testClientAnchorFromEscher(){

@@ -188,7 +188,7 @@ public class TestHSSFAnchor extends TestCase {
 

     public void testClientAnchorFromScratch(){

         HSSFClientAnchor anchor = new HSSFClientAnchor();

-        EscherClientAnchorRecord escher = (EscherClientAnchorRecord) anchor.getEscherAnchor();

+        EscherClientAnchorRecord escher = (EscherClientAnchorRecord) HSSFTestHelper.getEscherAnchor(anchor);

         anchor.setAnchor((short)11, 12, 13, 14, (short)15, 16, 17, 18);

 

         assertEquals(anchor.getCol1(), 11);

@@ -254,7 +254,7 @@ public class TestHSSFAnchor extends TestCase {
 

     public void testChildAnchorFromScratch(){

         HSSFChildAnchor anchor = new HSSFChildAnchor();

-        EscherChildAnchorRecord escher = (EscherChildAnchorRecord) anchor.getEscherAnchor();

+        EscherChildAnchorRecord escher = (EscherChildAnchorRecord) HSSFTestHelper.getEscherAnchor(anchor);

         anchor.setAnchor(11, 12, 13, 14);

 

         assertEquals(anchor.getDx1(), 11);