Patch from Javen ONeal from bug #58252 - More CellReference unit testing coverage

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

diff --git a/src/testcases/org/apache/poi/hssf/util/TestCellReference.java b/src/testcases/org/apache/poi/hssf/util/TestCellReference.java
index 79fc876e93d7b43add270e93430e018ac53153c1..d05c250d70e7b3df37c870d61d05d209b43f2eec 100644 (file)
--- a/src/testcases/org/apache/poi/hssf/util/TestCellReference.java
+++ b/src/testcases/org/apache/poi/hssf/util/TestCellReference.java
@@ -17,14 +17,12 @@
 
 package org.apache.poi.hssf.util;
 
-
 import junit.framework.TestCase;
 
 import org.apache.poi.ss.util.CellReference;
 import org.apache.poi.ss.util.CellReference.NameType;
 import org.apache.poi.ss.SpreadsheetVersion;
 
-
 public final class TestCellReference extends TestCase {
     public void testColNumConversion() {
         assertEquals(0, CellReference.convertColStringToIndex("A"));

diff --git a/src/testcases/org/apache/poi/ss/util/TestCellReference.java b/src/testcases/org/apache/poi/ss/util/TestCellReference.java
index d8bfee77c427a18841394f84cd2951eecb0455d4..a106e76e74a0f0a163101bcee9664ded19625745 100644 (file)
--- a/src/testcases/org/apache/poi/ss/util/TestCellReference.java
+++ b/src/testcases/org/apache/poi/ss/util/TestCellReference.java
@@ -25,9 +25,54 @@ import junit.framework.TestCase;
 
 
 /**
- * Tests that the common CellReference works as we need it to
+ * Tests that the common CellReference works as we need it to.
+ * Note - some additional testing is also done in the HSSF class,
+ *  {@link org.apache.poi.hssf.util.TestCellReference}
  */
 public final class TestCellReference extends TestCase {
+       public void testConstructors() {
+               CellReference cellReference;
+               final String sheet = "Sheet1";
+               final String cellRef = "A1";
+               final int row = 0;
+               final int col = 0;
+               final boolean absRow = true;
+               final boolean absCol = false;
+               
+               cellReference = new CellReference(row, col);
+               assertEquals("A1", cellReference.formatAsString());
+               
+               cellReference = new CellReference(row, col, absRow, absCol);
+               assertEquals("A$1", cellReference.formatAsString());
+               
+               cellReference = new CellReference(row, (short)col);
+               assertEquals("A1", cellReference.formatAsString());
+               
+               cellReference = new CellReference(cellRef);
+               assertEquals("A1", cellReference.formatAsString());
+               
+               cellReference = new CellReference(sheet, row, col, absRow, absCol);
+               assertEquals("Sheet1!A$1", cellReference.formatAsString());
+       }
+       
+       public void testFormatAsString() {
+               CellReference cellReference;
+               
+               cellReference = new CellReference(null, 0, 0, false, false);
+               assertEquals("A1", cellReference.formatAsString());
+               
+               //absolute references
+               cellReference = new CellReference(null, 0, 0, true, false);
+               assertEquals("A$1", cellReference.formatAsString());
+               
+               //sheet name with no spaces
+               cellReference = new CellReference("Sheet1", 0, 0, true, false);
+               assertEquals("Sheet1!A$1", cellReference.formatAsString());
+               
+               //sheet name with spaces
+               cellReference = new CellReference("Sheet 1", 0, 0, true, false);
+               assertEquals("'Sheet 1'!A$1", cellReference.formatAsString());
+       }
        
        public void testGetCellRefParts() {
                CellReference cellReference;