+++ /dev/null
-/* ====================================================================
- 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 org.apache.poi.ss.usermodel.BaseTestSheetShiftRows;
-import org.apache.poi.ss.ITestDataProvider;
-import org.apache.poi.xssf.XSSFITestDataProvider;
-
-/**
- * @author Yegor Kozlov
- */
-public final class TestSheetShiftRows extends BaseTestSheetShiftRows {
-
- @Override
- protected ITestDataProvider getTestDataProvider(){
- return XSSFITestDataProvider.getInstance();
- }
-
- public void testShiftRows() {
- baseTestShiftRows("SimpleMultiCell.xlsx");
- }
-
- public void testShiftRow() {
- baseTestShiftRow();
- }
-
- public void testShiftNames() {
- baseTestShiftWithNames();
- }
-
- //TODO support shifting of page breaks
- public void $testShiftRowBreaks() {
- baseTestShiftRowBreaks();
- }
-
- //TODO support shifting of comments.
- public void $testShiftWithComments() {
- baseTestShiftWithComments("comments.xlsx");
- }
-
- public void testShiftWithFormulas() {
- baseTestShiftWithFormulas("ForShifting.xlsx");
- }
-
- public void testShiftWithMergedRegions() {
- baseTestShiftWithMergedRegions();
- }
-}
--- /dev/null
+/* ====================================================================
+ 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 org.apache.poi.ss.usermodel.BaseTestSheetShiftRows;
+import org.apache.poi.xssf.XSSFITestDataProvider;
+
+/**
+ * @author Yegor Kozlov
+ */
+public final class TestXSSFSheetShiftRows extends BaseTestSheetShiftRows {
+
+ public TestXSSFSheetShiftRows(){
+ super(XSSFITestDataProvider.instance);
+ }
+
+ public void testShiftRowBreaks() { // disabled test from superclass
+ // TODO - support shifting of page breaks
+ }
+
+ public void testShiftWithComments() { // disabled test from superclass
+ // TODO - support shifting of comments.
+ }
+}
result.addTestSuite(TestReadWriteChart.class);
result.addTestSuite(TestSanityChecker.class);
result.addTestSuite(TestSheetHiding.class);
- result.addTestSuite(TestSheetShiftRows.class);
+ result.addTestSuite(TestHSSFSheetShiftRows.class);
if (false) { // deliberately avoiding this one
result.addTestSuite(TestUnfixedBugs.class);
}
--- /dev/null
+/* ====================================================================
+ 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.hssf.usermodel;
+
+import org.apache.poi.hssf.HSSFITestDataProvider;
+import org.apache.poi.ss.usermodel.BaseTestSheetShiftRows;
+
+/**
+ * @author Yegor Kozlov
+ */
+public final class TestHSSFSheetShiftRows extends BaseTestSheetShiftRows {
+
+ public TestHSSFSheetShiftRows() {
+ super(HSSFITestDataProvider.instance);
+ }
+}
+++ /dev/null
-/* ====================================================================
- 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.hssf.usermodel;
-
-import org.apache.poi.hssf.HSSFITestDataProvider;
-import org.apache.poi.ss.usermodel.BaseTestSheetShiftRows;
-
-/**
- * Tests row shifting capabilities.
- *
- *
- * @author Shawn Laubach (slaubach at apache dot com)
- * @author Toshiaki Kamoshida (kamoshida.toshiaki at future dot co dot jp)
- */
-public final class TestSheetShiftRows extends BaseTestSheetShiftRows {
-
-
- @Override
- protected HSSFITestDataProvider getTestDataProvider(){
- return HSSFITestDataProvider.getInstance();
- }
-
- public void testShiftRows() {
- baseTestShiftRows("SimpleMultiCell.xls");
- }
-
- public void testShiftRow() {
- baseTestShiftRow();
- }
-
- public void testShiftNames() {
- baseTestShiftWithNames();
- }
-
- public void testShiftRowBreaks() {
- baseTestShiftRowBreaks();
- }
-
- public void testShiftWithComments() {
- baseTestShiftWithComments("comments.xls");
- }
-
- public void testShiftWithFormulas() {
- baseTestShiftWithFormulas("ForShifting.xls");
- }
-
- public void testShiftWithMergedRegions() {
- baseTestShiftWithMergedRegions();
- }
-}
package org.apache.poi.ss.usermodel;
import junit.framework.TestCase;
+
import org.apache.poi.ss.ITestDataProvider;
import org.apache.poi.ss.util.CellRangeAddress;
* @author Shawn Laubach (slaubach at apache dot com)
* @author Toshiaki Kamoshida (kamoshida.toshiaki at future dot co dot jp)
*/
-public abstract class BaseTestSheetShiftRows extends TestCase {
+public abstract class BaseTestSheetShiftRows extends TestCase {
- /**
- * @return an object that provides test data in HSSF / XSSF specific way
- */
- protected abstract ITestDataProvider getTestDataProvider();
+ private final ITestDataProvider _testDataProvider;
+
+ protected BaseTestSheetShiftRows(ITestDataProvider testDataProvider) {
+ _testDataProvider = testDataProvider;
+ }
/**
* Tests the shiftRows function. Does three different shifts.
*
* @param sampleName the sample file to test against
*/
- public final void baseTestShiftRows(String sampleName){
+ public final void testShiftRows(){
// Read initial file in
- Workbook wb = getTestDataProvider().openSampleWorkbook(sampleName);
+ String sampleName = "SimpleMultiCell." + _testDataProvider.getStandardFileNameExtension();
+ Workbook wb = _testDataProvider.openSampleWorkbook(sampleName);
Sheet s = wb.getSheetAt( 0 );
// Shift the second row down 1 and write to temp file
s.shiftRows( 1, 1, 1 );
- wb = getTestDataProvider().writeOutAndReadBack(wb);
+ wb = _testDataProvider.writeOutAndReadBack(wb);
// Read from temp file and check the number of cells in each
// row (in original file each row was unique)
// Shift rows 1-3 down 3 in the current one. This tests when
// 1 row is blank. Write to a another temp file
s.shiftRows( 0, 2, 3 );
- wb = getTestDataProvider().writeOutAndReadBack(wb);
+ wb = _testDataProvider.writeOutAndReadBack(wb);
// Read and ensure things are where they should be
s = wb.getSheetAt(0);
assertEquals(s.getRow(5).getPhysicalNumberOfCells(), 2);
// Read the first file again
- wb = getTestDataProvider().openSampleWorkbook(sampleName);
+ wb = _testDataProvider.openSampleWorkbook(sampleName);
s = wb.getSheetAt( 0 );
// Shift rows 3 and 4 up and write to temp file
s.shiftRows( 2, 3, -2 );
- wb = getTestDataProvider().writeOutAndReadBack(wb);
+ wb = _testDataProvider.writeOutAndReadBack(wb);
s = wb.getSheetAt( 0 );
assertEquals(s.getRow(0).getPhysicalNumberOfCells(), 3);
assertEquals(s.getRow(1).getPhysicalNumberOfCells(), 4);
/**
* Tests when rows are null.
*/
- public final void baseTestShiftRow() {
- Workbook b = getTestDataProvider().createWorkbook();
+ public final void testShiftRow() {
+ Workbook b = _testDataProvider.createWorkbook();
Sheet s = b.createSheet();
s.createRow(0).createCell(0).setCellValue("TEST1");
s.createRow(3).createCell(0).setCellValue("TEST2");
/**
* Tests when shifting the first row.
*/
- public final void baseTestActiveCell() {
- Workbook b = getTestDataProvider().createWorkbook();
+ public final void testActiveCell() {
+ Workbook b = _testDataProvider.createWorkbook();
Sheet s = b.createSheet();
s.createRow(0).createCell(0).setCellValue("TEST1");
/**
* When shifting rows, the page breaks should go with it
- *
*/
- public final void baseTestShiftRowBreaks() {
- Workbook b = getTestDataProvider().createWorkbook();
+ public void testShiftRowBreaks() { // TODO - enable XSSF test
+ Workbook b = _testDataProvider.createWorkbook();
Sheet s = b.createSheet();
Row row = s.createRow(4);
row.createCell(0).setCellValue("test");
assertTrue("Row number 6 should have a pagebreak", s.isRowBroken(6));
}
+ public void testShiftWithComments() { // TODO - enable XSSF test
- public final void baseTestShiftWithComments(String sampleName) {
- Workbook wb = getTestDataProvider().openSampleWorkbook(sampleName);
+ Workbook wb = _testDataProvider.openSampleWorkbook("comments." + _testDataProvider.getStandardFileNameExtension());
Sheet sheet = wb.getSheet("Sheet1");
assertEquals(3, sheet.getLastRowNum());
// Write out and read back in again
// Ensure that the changes were persisted
- wb = getTestDataProvider().writeOutAndReadBack(wb);
+ wb = _testDataProvider.writeOutAndReadBack(wb);
sheet = wb.getSheet("Sheet1");
assertEquals(4, sheet.getLastRowNum());
assertEquals(comment4,comment4_shifted);
}
- public final void baseTestShiftWithNames() {
- Workbook wb = getTestDataProvider().createWorkbook();
+ public final void testShiftWithNames() {
+ Workbook wb = _testDataProvider.createWorkbook();
Sheet sheet1 = wb.createSheet("Sheet1");
wb.createSheet("Sheet2");
Row row = sheet1.createRow(0);
assertEquals("A1", name4.getRefersToFormula());
}
- public final void baseTestShiftWithMergedRegions() {
- Workbook wb = getTestDataProvider().createWorkbook();
+ public final void testShiftWithMergedRegions() {
+ Workbook wb = _testDataProvider.createWorkbook();
Sheet sheet = wb.createSheet();
Row row = sheet.createRow(0);
row.createCell(0).setCellValue(1.1);
/**
* See bug #34023
- *
- * @param sampleName the sample file to test against
*/
- public void baseTestShiftWithFormulas(String sampleName) {
- Workbook wb = getTestDataProvider().openSampleWorkbook(sampleName);
+ public final void testShiftWithFormulas() {
+ Workbook wb = _testDataProvider.openSampleWorkbook("ForShifting." + _testDataProvider.getStandardFileNameExtension());
Sheet sheet = wb.getSheet("Sheet1");
assertEquals(20, sheet.getLastRowNum());