diff --git a/src/integrationtest/org/apache/poi/TestAllFiles.java b/src/integrationtest/org/apache/poi/TestAllFiles.java index f3d79eb17..14e346bd3 100644 --- a/src/integrationtest/org/apache/poi/TestAllFiles.java +++ b/src/integrationtest/org/apache/poi/TestAllFiles.java @@ -299,8 +299,7 @@ public class TestAllFiles { "spreadsheet/poc-xmlbomb.xlsx", // contains xml-entity-expansion "spreadsheet/poc-xmlbomb-empty.xlsx", // contains xml-entity-expansion "spreadsheet/poc-shared-strings.xlsx", // contains shared-string-entity-expansion - "spreadsheet/60255_extra_drawingparts.xlsx", // Non-drawing drawing - + // old Excel files, which we only support simple text extraction of "spreadsheet/testEXCEL_2.xls", "spreadsheet/testEXCEL_3.xls", @@ -321,12 +320,8 @@ public class TestAllFiles { "ddf/47143.dat", // sheet cloning errors - "spreadsheet/47813.xlsx", "spreadsheet/56450.xls", - "spreadsheet/OddStyleRecord.xls", - "spreadsheet/WithChartSheet.xlsx", - "spreadsheet/chart_sheet.xlsx", - "spreadsheet/SimpleScatterChart.xlsx" + "spreadsheet/OddStyleRecord.xls" ); private static final Set IGNORED = unmodifiableHashSet( diff --git a/src/integrationtest/org/apache/poi/stress/SpreadsheetHandler.java b/src/integrationtest/org/apache/poi/stress/SpreadsheetHandler.java index 854dbf2fb..2b5c6295f 100644 --- a/src/integrationtest/org/apache/poi/stress/SpreadsheetHandler.java +++ b/src/integrationtest/org/apache/poi/stress/SpreadsheetHandler.java @@ -31,6 +31,7 @@ import org.apache.poi.ss.usermodel.Sheet; import org.apache.poi.ss.usermodel.Workbook; import org.apache.poi.ss.usermodel.WorkbookFactory; import org.apache.poi.util.RecordFormatException; +import org.apache.poi.xssf.usermodel.XSSFChartSheet; public abstract class SpreadsheetHandler extends AbstractFileHandler { public void handleWorkbook(Workbook wb) throws IOException { @@ -125,6 +126,11 @@ public abstract class SpreadsheetHandler extends AbstractFileHandler { }*/ for (int i=wb.getNumberOfSheets()-1; i>=0; i--) { + if(wb.getSheetAt(i) instanceof XSSFChartSheet) { + // clone for chart-sheets is not supported + continue; + } + try { wb.cloneSheet(i); } catch (RecordFormatException e) { diff --git a/src/ooxml/testcases/org/apache/poi/xssf/streaming/TestSXSSFWorkbook.java b/src/ooxml/testcases/org/apache/poi/xssf/streaming/TestSXSSFWorkbook.java index 2a4653173..9179362fa 100644 --- a/src/ooxml/testcases/org/apache/poi/xssf/streaming/TestSXSSFWorkbook.java +++ b/src/ooxml/testcases/org/apache/poi/xssf/streaming/TestSXSSFWorkbook.java @@ -580,7 +580,7 @@ public final class TestSXSSFWorkbook extends BaseTestXWorkbook { @Test public void test56557() throws IOException, InvalidFormatException { - Workbook wb = WorkbookFactory.create(XSSFTestDataSamples.getSampleFile("56557.xlsx")); + Workbook wb = WorkbookFactory.create(XSSFTestDataSamples.getSampleFile("56557.xlsx"), null, false); // Using streaming XSSFWorkbook makes the output file invalid wb = new SXSSFWorkbook(((XSSFWorkbook) wb)); diff --git a/test-data/spreadsheet/56557.xlsx b/test-data/spreadsheet/56557.xlsx index d092ffa6b..7ca891cf5 100644 Binary files a/test-data/spreadsheet/56557.xlsx and b/test-data/spreadsheet/56557.xlsx differ