]> source.dussan.org Git - poi.git/commitdiff
added setDisplayZeros / isDisplayZeros and getMergedRegion(int) to common interface...
authorYegor Kozlov <yegor@apache.org>
Tue, 24 Feb 2009 08:38:16 +0000 (08:38 +0000)
committerYegor Kozlov <yegor@apache.org>
Tue, 24 Feb 2009 08:38:16 +0000 (08:38 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@747307 13f79535-47bb-0310-9956-ffa450edef68

13 files changed:
src/documentation/content/xdocs/changes.xml
src/documentation/content/xdocs/status.xml
src/java/org/apache/poi/hssf/usermodel/HSSFSheet.java
src/ooxml/interfaces-jdk15/org/apache/poi/ss/usermodel/Sheet.java
src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFSheet.java
src/ooxml/testcases/org/apache/poi/xssf/XSSFITestDataProvider.java [new file with mode: 0755]
src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestSheetShiftRows.java
src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFSheet.java
src/testcases/org/apache/poi/hssf/HSSFITestDataProvider.java [new file with mode: 0755]
src/testcases/org/apache/poi/hssf/usermodel/TestHSSFSheet.java
src/testcases/org/apache/poi/hssf/usermodel/TestSheetShiftRows.java
src/testcases/org/apache/poi/ss/ITestDataProvider.java [new file with mode: 0755]
src/testcases/org/apache/poi/ss/usermodel/BaseTestSheetShiftRows.java

index 9389059f55c9b1fc2076334e2563f077cbe70445..c461c0c5bb05263531b195969aeafd5f302d7b28 100644 (file)
     </devs>
 
                <!-- Don't forget to update status.xml too! -->
-        <release version="3.5-beta6" date="2008-??-??">
+        <release version="3.5-beta6" date="2009-??-??">
+           <action dev="POI-DEVELOPERS" type="add">46703 - added setDisplayZeros / isDisplayZeros to common interface org.apache.poi.ss.usermodel.Sheet</action>
+           <action dev="POI-DEVELOPERS" type="add">46708 - added getMergedRegion(int) to common interface org.apache.poi.ss.usermodel.Sheet</action>
            <action dev="POI-DEVELOPERS" type="fix">fixed Sheet.autoSizeColumn() to use cached formula values when processing formula cells </action>
            <action dev="POI-DEVELOPERS" type="fix">Fixed formula parser to handle names with backslashes</action>
            <action dev="POI-DEVELOPERS" type="add">46660 - added Workbook getHidden() and setHidden(boolean)</action>
            <action dev="POI-DEVELOPERS" type="fix">46693 - Fixed bugs serialization bugs in records: CHARTFORMAT, SHTPROPS, SXVD and SXVDEX</action>
            <action dev="POI-DEVELOPERS" type="fix">46627 - Fixed offset of added images if Pictures stream contains pictures with zero length</action>
         </release>
-        <release version="3.5-beta5" date="2008-02-19">
+        <release version="3.5-beta5" date="2009-02-19">
            <action dev="POI-DEVELOPERS" type="fix">46536 - When shifting rows, update formulas on that sheet to point to the new location of those rows</action>
            <action dev="POI-DEVELOPERS" type="fix">46663 - Fixed XSSFSheet.shiftRows to properly update references of the shifted cells</action>
            <action dev="POI-DEVELOPERS" type="fix">46535 - Remove reference from calculation chain when a formula is deleted</action>
index 989c45d384c7fc16a4d90c860a26cc007690356a..b85e04df0caf7b48f494e129da272496bfd22261 100644 (file)
 
        <!-- Don't forget to update changes.xml too! -->
     <changes>
-        <release version="3.5-beta6" date="2008-??-??">
+        <release version="3.5-beta6" date="2009-??-??">
+           <action dev="POI-DEVELOPERS" type="add">46703 - added setDisplayZeros / isDisplayZeros to common interface org.apache.poi.ss.usermodel.Sheet</action>
+           <action dev="POI-DEVELOPERS" type="add">46708 - added getMergedRegion(int) to common interface org.apache.poi.ss.usermodel.Sheet</action>
            <action dev="POI-DEVELOPERS" type="fix">fixed Sheet.autoSizeColumn() to use cached formula values when processing formula cells </action>
            <action dev="POI-DEVELOPERS" type="fix">Fixed formula parser to handle names with backslashes</action>
            <action dev="POI-DEVELOPERS" type="add">46660 - added Workbook getHidden() and setHidden(boolean)</action>
            <action dev="POI-DEVELOPERS" type="fix">46693 - Fixed bugs serialization bugs in records: CHARTFORMAT, SHTPROPS, SXVD and SXVDEX</action>
            <action dev="POI-DEVELOPERS" type="fix">46627 - Fixed offset of added images if Pictures stream contains pictures with zero length</action>
         </release>
-        <release version="3.5-beta5" date="2008-02-19">
+        <release version="3.5-beta5" date="2009-02-19">
            <action dev="POI-DEVELOPERS" type="fix">46536 - When shifting rows, update formulas on that sheet to point to the new location of those rows</action>
            <action dev="POI-DEVELOPERS" type="fix">46663 - Fixed XSSFSheet.shiftRows to properly update references of the shifted cells</action>
            <action dev="POI-DEVELOPERS" type="fix">46535 - Remove reference from calculation chain when a formula is deleted</action>
index 86336034df878278aab300e06875fabbb88f0e64..96a21810b28e4c4037f064e728a5bf564ea1bcfe 100644 (file)
@@ -841,6 +841,31 @@ public final class HSSFSheet implements org.apache.poi.ss.usermodel.Sheet {
                 .getDisplayGuts();
     }
 
+
+    /**
+     * Gets the flag indicating whether the window should show 0 (zero) in cells containing zero value.
+     * When false, cells with zero value appear blank instead of showing the number zero.
+     * <p>
+     * In Excel 2003 this option can be changed in the Options dialog on the View tab.
+     * </p>
+     * @return whether all zero values on the worksheet are displayed
+     */
+    public boolean isDisplayZeros(){
+        return _sheet.getWindowTwo().getDisplayZeros();
+    }
+
+    /**
+     * Set whether the window should show 0 (zero) in cells containing zero value.
+     * When false, cells with zero value appear blank instead of showing the number zero.
+     * <p>
+     * In Excel 2003 this option can be set in the Options dialog on the View tab.
+     * </p>
+     * @param value whether to display or hide all zero values on the worksheet
+     */
+    public void setDisplayZeros(boolean value){
+        _sheet.getWindowTwo().setDisplayZeros(value);    
+    }
+
     /**
      * fit to page option is on
      * @return fit or not
index ad3e50b09e7cbc9d7b5749d52fb8e4e86376ca92..0a508c382e81a6e378054ac423c6b860f8a19682 100644 (file)
@@ -238,6 +238,13 @@ public interface Sheet extends Iterable<Row> {
      */
     int getNumMergedRegions();
 
+    /**
+     * Returns the merged region at the specified index
+     *
+     * @return the merged region at the specified index
+     */
+    public CellRangeAddress getMergedRegion(int index);
+
     /**
      *  Returns an iterator of the physical rows
      *
@@ -260,6 +267,23 @@ public interface Sheet extends Iterable<Row> {
      */
     void setDisplayGuts(boolean value);
 
+    /**
+     * Set whether the window should show 0 (zero) in cells containing zero value.
+     * When false, cells with zero value appear blank instead of showing the number zero.
+     * 
+     * @param value whether to display or hide all zero values on the worksheet
+     */
+    void setDisplayZeros(boolean value);
+
+
+    /**
+     * Gets the flag indicating whether the window should show 0 (zero) in cells containing zero value.
+     * When false, cells with zero value appear blank instead of showing the number zero.
+     *
+     * @return whether all zero values on the worksheet are displayed
+     */
+    boolean isDisplayZeros();
+
     /**
      * Flag indicating whether the Fit to Page print option is enabled.
      *
index e44208b75e5461df2d47f607fd2dac24d706e7fe..bdc917d8395ea80a4f7adb8e1e1a290f33e279e2 100644 (file)
@@ -539,6 +539,28 @@ public class XSSFSheet extends POIXMLDocumentPart implements Sheet {
         outlinePr.setShowOutlineSymbols(value);
     }
 
+    /**
+     * Gets the flag indicating whether the window should show 0 (zero) in cells containing zero value.
+     * When false, cells with zero value appear blank instead of showing the number zero.
+     * 
+     * @return whether all zero values on the worksheet are displayed
+     */
+    public boolean isDisplayZeros(){
+        CTSheetView view = getDefaultSheetView();
+        return view == null ? true : view.getShowZeros();
+    }
+
+    /**
+     * Set whether the window should show 0 (zero) in cells containing zero value.
+     * When false, cells with zero value appear blank instead of showing the number zero.
+     *
+     * @param value whether to display or hide all zero values on the worksheet
+     */
+    public void setDisplayZeros(boolean value){
+        CTSheetView view = getSheetTypeSheetView();
+        view.setShowZeros(value);
+    }
+
     /**
      * Gets the first row on the sheet
      *
diff --git a/src/ooxml/testcases/org/apache/poi/xssf/XSSFITestDataProvider.java b/src/ooxml/testcases/org/apache/poi/xssf/XSSFITestDataProvider.java
new file mode 100755 (executable)
index 0000000..c0a17cf
--- /dev/null
@@ -0,0 +1,49 @@
+/* ====================================================================\r
+   Licensed to the Apache Software Foundation (ASF) under one or more\r
+   contributor license agreements.  See the NOTICE file distributed with\r
+   this work for additional information regarding copyright ownership.\r
+   The ASF licenses this file to You under the Apache License, Version 2.0\r
+   (the "License"); you may not use this file except in compliance with\r
+   the License.  You may obtain a copy of the License at\r
+\r
+       http://www.apache.org/licenses/LICENSE-2.0\r
+\r
+   Unless required by applicable law or agreed to in writing, software\r
+   distributed under the License is distributed on an "AS IS" BASIS,\r
+   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\r
+   See the License for the specific language governing permissions and\r
+   limitations under the License.\r
+==================================================================== */\r
+package org.apache.poi.xssf;\r
+\r
+import org.apache.poi.ss.ITestDataProvider;\r
+import org.apache.poi.ss.usermodel.Workbook;\r
+import org.apache.poi.xssf.usermodel.XSSFWorkbook;\r
+\r
+/**\r
+ * @author Yegor Kozlov\r
+ */\r
+public final class XSSFITestDataProvider implements ITestDataProvider {\r
+\r
+    public XSSFWorkbook openSampleWorkbook(String sampleFileName) {\r
+               return XSSFTestDataSamples.openSampleWorkbook(sampleFileName);\r
+       }\r
+\r
+       public XSSFWorkbook writeOutAndReadBack(Workbook original) {\r
+        if(!(original instanceof XSSFWorkbook)) {\r
+            throw new IllegalArgumentException("Expected an instance of XSSFWorkbook");\r
+        }\r
+\r
+        return XSSFTestDataSamples.writeOutAndReadBack((XSSFWorkbook)original);\r
+       }\r
+\r
+    public XSSFWorkbook createWorkbook(){\r
+        return new XSSFWorkbook();\r
+    }\r
+\r
+    private XSSFITestDataProvider(){}\r
+    private static XSSFITestDataProvider inst = new XSSFITestDataProvider();\r
+    public static XSSFITestDataProvider getInstance(){\r
+        return inst;\r
+    }\r
+}\r
index f3a0fb5b111c5e0dabcb4ab722e18430da230f49..212271d0b71f6671b2de260d40e243dde7f398a8 100755 (executable)
@@ -17,8 +17,8 @@
 package org.apache.poi.xssf.usermodel;\r
 \r
 import org.apache.poi.ss.usermodel.BaseTestSheetShiftRows;\r
-import org.apache.poi.ss.usermodel.Workbook;\r
-import org.apache.poi.xssf.XSSFTestDataSamples;\r
+import org.apache.poi.ss.ITestDataProvider;\r
+import org.apache.poi.xssf.XSSFITestDataProvider;\r
 \r
 /**\r
  * @author Yegor Kozlov\r
@@ -26,18 +26,8 @@ import org.apache.poi.xssf.XSSFTestDataSamples;
 public class TestSheetShiftRows  extends BaseTestSheetShiftRows {\r
 \r
     @Override\r
-    protected Workbook openSampleWorkbook(String sampleFileName) {\r
-        return XSSFTestDataSamples.openSampleWorkbook(sampleFileName);\r
-    }\r
-\r
-    @Override\r
-    protected Workbook writeOutAndReadBack(Workbook wb) {\r
-        return XSSFTestDataSamples.writeOutAndReadBack(wb);\r
-    }\r
-\r
-    @Override\r
-    protected Workbook createWorkbook() {\r
-        return new XSSFWorkbook();\r
+    protected ITestDataProvider getTestDataProvider(){\r
+        return XSSFITestDataProvider.getInstance();\r
     }\r
 \r
     public void testShiftRows() {\r
index 38ad08f64ed149444dece1a33ab2f57979de013b..c19ea200f1ec21415e5505f266863006172c2909 100644 (file)
@@ -926,4 +926,32 @@ public class TestXSSFSheet extends TestCase {
         assertEquals(500, sh2.getColumnWidth(0));
         assertEquals(sh1.getColumnWidth(0), sh2.getColumnWidth(0));
     }
+
+    /**
+     * Tests the display of gridlines, formulas, and rowcolheadings.
+     * @author Shawn Laubach (slaubach at apache dot org)
+     */
+    public void testDisplayOptions() {
+        Workbook wb = new XSSFWorkbook();
+        Sheet sheet = wb.createSheet();
+
+        assertEquals(sheet.isDisplayGridlines(), true);
+        assertEquals(sheet.isDisplayRowColHeadings(), true);
+        assertEquals(sheet.isDisplayFormulas(), false);
+        assertEquals(sheet.isDisplayZeros(), true);
+
+        sheet.setDisplayGridlines(false);
+        sheet.setDisplayRowColHeadings(false);
+        sheet.setDisplayFormulas(true);
+        sheet.setDisplayZeros(false);
+
+        wb = XSSFTestDataSamples.writeOutAndReadBack(wb);
+        sheet = wb.getSheetAt(0);
+
+        assertEquals(sheet.isDisplayGridlines(), false);
+        assertEquals(sheet.isDisplayRowColHeadings(), false);
+        assertEquals(sheet.isDisplayFormulas(), true);
+        assertEquals(sheet.isDisplayZeros(), false);
+    }
+
 }
diff --git a/src/testcases/org/apache/poi/hssf/HSSFITestDataProvider.java b/src/testcases/org/apache/poi/hssf/HSSFITestDataProvider.java
new file mode 100755 (executable)
index 0000000..2311cc0
--- /dev/null
@@ -0,0 +1,50 @@
+/* ====================================================================\r
+   Licensed to the Apache Software Foundation (ASF) under one or more\r
+   contributor license agreements.  See the NOTICE file distributed with\r
+   this work for additional information regarding copyright ownership.\r
+   The ASF licenses this file to You under the Apache License, Version 2.0\r
+   (the "License"); you may not use this file except in compliance with\r
+   the License.  You may obtain a copy of the License at\r
+\r
+       http://www.apache.org/licenses/LICENSE-2.0\r
+\r
+   Unless required by applicable law or agreed to in writing, software\r
+   distributed under the License is distributed on an "AS IS" BASIS,\r
+   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\r
+   See the License for the specific language governing permissions and\r
+   limitations under the License.\r
+==================================================================== */\r
+\r
+package org.apache.poi.hssf;\r
+\r
+import org.apache.poi.hssf.usermodel.HSSFWorkbook;\r
+import org.apache.poi.ss.ITestDataProvider;\r
+import org.apache.poi.ss.usermodel.Workbook;\r
+\r
+/**\r
+ * @author Yegor Kozlov\r
+ */\r
+public final class HSSFITestDataProvider implements ITestDataProvider {\r
+\r
+    public HSSFWorkbook openSampleWorkbook(String sampleFileName) {\r
+               return HSSFTestDataSamples.openSampleWorkbook(sampleFileName);\r
+       }\r
+\r
+       public HSSFWorkbook writeOutAndReadBack(Workbook original) {\r
+        if(!(original instanceof HSSFWorkbook)) {\r
+            throw new IllegalArgumentException("Expected an instance of HSSFWorkbook");\r
+        }\r
+\r
+        return HSSFTestDataSamples.writeOutAndReadBack((HSSFWorkbook)original);\r
+       }\r
+\r
+    public HSSFWorkbook createWorkbook(){\r
+        return new HSSFWorkbook();\r
+    }\r
+\r
+    private HSSFITestDataProvider(){}\r
+    private static HSSFITestDataProvider inst = new HSSFITestDataProvider();\r
+    public static HSSFITestDataProvider getInstance(){\r
+        return inst;\r
+    }\r
+}
\ No newline at end of file
index c119daa283547fec03a3d7bacdd138b5dcb47f89..446034ebfef48305df954e0f2ab904be1cc003eb 100644 (file)
@@ -538,16 +538,15 @@ public final class TestHSSFSheet extends TestCase {
         HSSFWorkbook wb = new HSSFWorkbook();
         HSSFSheet sheet = wb.createSheet();
 
-        wb = HSSFTestDataSamples.writeOutAndReadBack(wb);
-        sheet = wb.getSheetAt(0);
-
         assertEquals(sheet.isDisplayGridlines(), true);
         assertEquals(sheet.isDisplayRowColHeadings(), true);
         assertEquals(sheet.isDisplayFormulas(), false);
+        assertEquals(sheet.isDisplayZeros(), true);
 
         sheet.setDisplayGridlines(false);
         sheet.setDisplayRowColHeadings(false);
         sheet.setDisplayFormulas(true);
+        sheet.setDisplayZeros(false);
 
         wb = HSSFTestDataSamples.writeOutAndReadBack(wb);
         sheet = wb.getSheetAt(0);
@@ -555,6 +554,7 @@ public final class TestHSSFSheet extends TestCase {
         assertEquals(sheet.isDisplayGridlines(), false);
         assertEquals(sheet.isDisplayRowColHeadings(), false);
         assertEquals(sheet.isDisplayFormulas(), true);
+        assertEquals(sheet.isDisplayZeros(), false);
     }
 
 
index 962a565503dc886c33223894138322bf2cfdb024..84e0aebdd2080c0b9680a53a943fabb680d06d2c 100755 (executable)
@@ -17,8 +17,7 @@
 
 package org.apache.poi.hssf.usermodel;
 
-import org.apache.poi.hssf.HSSFTestDataSamples;
-import org.apache.poi.ss.usermodel.Workbook;
+import org.apache.poi.hssf.HSSFITestDataProvider;
 import org.apache.poi.ss.usermodel.BaseTestSheetShiftRows;
 
 /**
@@ -30,23 +29,11 @@ import org.apache.poi.ss.usermodel.BaseTestSheetShiftRows;
  */
 public final class TestSheetShiftRows extends BaseTestSheetShiftRows {
 
-       @Override
-       protected Workbook openSampleWorkbook(String sampleFileName) {
-               return HSSFTestDataSamples.openSampleWorkbook(sampleFileName);
-       }
 
-       @Override
-       protected Workbook writeOutAndReadBack(Workbook wb) {
-               if (wb instanceof HSSFWorkbook) {
-                       return HSSFTestDataSamples.writeOutAndReadBack((HSSFWorkbook) wb);
-               }
-               throw new IllegalArgumentException("bad workbook type (" + wb.getClass().getName() + ")");
-       }
-
-       @Override
-       protected Workbook createWorkbook() {
-               return new HSSFWorkbook();
-       }
+    @Override
+    protected HSSFITestDataProvider getTestDataProvider(){
+        return HSSFITestDataProvider.getInstance();
+    }
 
     public void testShiftRows() {
         baseTestShiftRows("SimpleMultiCell.xls");
diff --git a/src/testcases/org/apache/poi/ss/ITestDataProvider.java b/src/testcases/org/apache/poi/ss/ITestDataProvider.java
new file mode 100755 (executable)
index 0000000..f3ca3b2
--- /dev/null
@@ -0,0 +1,47 @@
+/* ====================================================================\r
+   Licensed to the Apache Software Foundation (ASF) under one or more\r
+   contributor license agreements.  See the NOTICE file distributed with\r
+   this work for additional information regarding copyright ownership.\r
+   The ASF licenses this file to You under the Apache License, Version 2.0\r
+   (the "License"); you may not use this file except in compliance with\r
+   the License.  You may obtain a copy of the License at\r
+\r
+       http://www.apache.org/licenses/LICENSE-2.0\r
+\r
+   Unless required by applicable law or agreed to in writing, software\r
+   distributed under the License is distributed on an "AS IS" BASIS,\r
+   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\r
+   See the License for the specific language governing permissions and\r
+   limitations under the License.\r
+==================================================================== */\r
+package org.apache.poi.ss;\r
+\r
+import org.apache.poi.ss.usermodel.Workbook;\r
+\r
+/**\r
+ * Incapsulates a provider of test data for common HSSF / XSSF tests.\r
+ */\r
+public interface ITestDataProvider {\r
+    /**\r
+     * Override to provide HSSF / XSSF specific way for re-serialising a workbook\r
+     *\r
+     * @param wb the workbook to re-serialize\r
+     * @return the re-serialized workbook\r
+     */\r
+    Workbook writeOutAndReadBack(Workbook wb);\r
+\r
+    /**\r
+     * Override to provide way of loading HSSF / XSSF sample workbooks\r
+     *\r
+     * @param sampleFileName the file name to load\r
+     * @return an instance of Workbook loaded from the supplied file name\r
+     */\r
+    Workbook openSampleWorkbook(String sampleFileName);\r
+\r
+    /**\r
+     * Override to provide way of creating HSSF / XSSF workbooks\r
+     * @return an instance of Workbook\r
+     */\r
+    Workbook createWorkbook();\r
+\r
+}\r
index 7b83cbbd532ca26f9cd05aa1e1ed9920733376d1..a9305d8cae7d4a1a8322f6ae0a787984931487a7 100755 (executable)
@@ -18,6 +18,7 @@
 package org.apache.poi.ss.usermodel;\r
 \r
 import junit.framework.TestCase;\r
+import org.apache.poi.ss.ITestDataProvider;\r
 \r
 /**\r
  * Tests row shifting capabilities.\r
@@ -26,19 +27,11 @@ import junit.framework.TestCase;
  * @author Toshiaki Kamoshida (kamoshida.toshiaki at future dot co dot jp)\r
  */\r
 public abstract class BaseTestSheetShiftRows  extends TestCase {\r
+\r
     /**\r
-     * Override to provide HSSF / XSSF specific way for re-serialising a workbook\r
-     */\r
-    protected abstract Workbook writeOutAndReadBack(Workbook wb);\r
-    /**\r
-     * Override to provide way of loading HSSF / XSSF sample workbooks\r
-     * @param sampleFileName without the ".xls" or ".xlsx" suffix\r
-     */\r
-    protected abstract Workbook openSampleWorkbook(String sampleFileName);\r
-    /**\r
-     * Override to provide way of creating HSSF / XSSF workbooks\r
+     * @return an object that provides test data in HSSF / XSSF specific way\r
      */\r
-    protected abstract Workbook createWorkbook();\r
+    protected abstract ITestDataProvider getTestDataProvider();\r
 \r
     /**\r
      * Tests the shiftRows function.  Does three different shifts.\r
@@ -46,17 +39,17 @@ public abstract class BaseTestSheetShiftRows  extends TestCase {
      * check.  This ensures that if some changes code that breaks\r
      * writing or what not, they realize it.\r
      *\r
-     * @author Shawn Laubach (slaubach at apache dot org)\r
+     * @param sampleName the sample file to test against\r
      */\r
     public final void baseTestShiftRows(String sampleName){\r
         // Read initial file in\r
-        Workbook wb = openSampleWorkbook(sampleName);\r
+        Workbook wb = getTestDataProvider().openSampleWorkbook(sampleName);\r
         Sheet s = wb.getSheetAt( 0 );\r
 \r
         // Shift the second row down 1 and write to temp file\r
         s.shiftRows( 1, 1, 1 );\r
 \r
-        wb = writeOutAndReadBack(wb);\r
+        wb = getTestDataProvider().writeOutAndReadBack(wb);\r
 \r
         // Read from temp file and check the number of cells in each\r
         // row (in original file each row was unique)\r
@@ -71,7 +64,7 @@ public abstract class BaseTestSheetShiftRows  extends TestCase {
         // Shift rows 1-3 down 3 in the current one.  This tests when\r
         // 1 row is blank.  Write to a another temp file\r
         s.shiftRows( 0, 2, 3 );\r
-        wb = writeOutAndReadBack(wb);\r
+        wb = getTestDataProvider().writeOutAndReadBack(wb);\r
 \r
         // Read and ensure things are where they should be\r
         s = wb.getSheetAt(0);\r
@@ -83,12 +76,12 @@ public abstract class BaseTestSheetShiftRows  extends TestCase {
         assertEquals(s.getRow(5).getPhysicalNumberOfCells(), 2);\r
 \r
         // Read the first file again\r
-        wb = openSampleWorkbook(sampleName);\r
+        wb = getTestDataProvider().openSampleWorkbook(sampleName);\r
         s = wb.getSheetAt( 0 );\r
 \r
         // Shift rows 3 and 4 up and write to temp file\r
         s.shiftRows( 2, 3, -2 );\r
-        wb = writeOutAndReadBack(wb);\r
+        wb = getTestDataProvider().writeOutAndReadBack(wb);\r
         s = wb.getSheetAt( 0 );\r
         assertEquals(s.getRow(0).getPhysicalNumberOfCells(), 3);\r
         assertEquals(s.getRow(1).getPhysicalNumberOfCells(), 4);\r
@@ -103,11 +96,9 @@ public abstract class BaseTestSheetShiftRows  extends TestCase {
 \r
     /**\r
      * Tests when rows are null.\r
-     *\r
-     * @author Toshiaki Kamoshida (kamoshida.toshiaki at future dot co dot jp)\r
      */\r
     public final void baseTestShiftRow() {\r
-        Workbook b = createWorkbook();\r
+        Workbook b = getTestDataProvider().createWorkbook();\r
         Sheet s        = b.createSheet();\r
         s.createRow(0).createCell(0).setCellValue("TEST1");\r
         s.createRow(3).createCell(0).setCellValue("TEST2");\r
@@ -116,11 +107,9 @@ public abstract class BaseTestSheetShiftRows  extends TestCase {
 \r
     /**\r
      * Tests when shifting the first row.\r
-     *\r
-     * @author Toshiaki Kamoshida (kamoshida.toshiaki at future dot co dot jp)\r
      */\r
     public final void baseTestShiftRow0() {\r
-        Workbook b = createWorkbook();\r
+        Workbook b = getTestDataProvider().createWorkbook();\r
         Sheet s        = b.createSheet();\r
         s.createRow(0).createCell(0).setCellValue("TEST1");\r
         s.createRow(3).createCell(0).setCellValue("TEST2");\r
@@ -132,7 +121,7 @@ public abstract class BaseTestSheetShiftRows  extends TestCase {
      *\r
      */\r
     public final void baseTestShiftRowBreaks() {\r
-        Workbook b = createWorkbook();\r
+        Workbook b = getTestDataProvider().createWorkbook();\r
         Sheet s        = b.createSheet();\r
         Row row = s.createRow(4);\r
         row.createCell(0).setCellValue("test");\r
@@ -144,7 +133,7 @@ public abstract class BaseTestSheetShiftRows  extends TestCase {
 \r
 \r
     public final void baseTestShiftWithComments(String sampleName) {\r
-        Workbook wb = openSampleWorkbook(sampleName);\r
+        Workbook wb = getTestDataProvider().openSampleWorkbook(sampleName);\r
 \r
         Sheet sheet = wb.getSheet("Sheet1");\r
         assertEquals(3, sheet.getLastRowNum());\r
@@ -182,7 +171,7 @@ public abstract class BaseTestSheetShiftRows  extends TestCase {
 \r
         // Write out and read back in again\r
         // Ensure that the changes were persisted\r
-        wb = writeOutAndReadBack(wb);\r
+        wb = getTestDataProvider().writeOutAndReadBack(wb);\r
         sheet = wb.getSheet("Sheet1");\r
         assertEquals(4, sheet.getLastRowNum());\r
 \r
@@ -203,9 +192,11 @@ public abstract class BaseTestSheetShiftRows  extends TestCase {
 \r
     /**\r
      * See bug #34023\r
+     *\r
+     * @param sampleName the sample file to test against\r
      */\r
     public void baseTestShiftWithFormulas(String sampleName) {\r
-        Workbook wb = openSampleWorkbook(sampleName);\r
+        Workbook wb = getTestDataProvider().openSampleWorkbook(sampleName);\r
 \r
         Sheet sheet = wb.getSheet("Sheet1");\r
         assertEquals(20, sheet.getLastRowNum());\r