From: Javen O'Neal <onealj@apache.org>
Date: Thu, 19 Jan 2017 08:51:04 +0000 (+0000)
Subject: bug 60601: allow SXSSFITestDataProvider.writeOutAndReadBack(Workbook wb) to work... 
X-Git-Tag: REL_3_16_BETA2~31
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=797048ff1ed6ad165f8094f5700e9e73f586a7b9;p=poi.git

bug 60601: allow SXSSFITestDataProvider.writeOutAndReadBack(Workbook wb) to work on SXSSFWorkbooks and XSSFWorkbooks

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

diff --git a/src/ooxml/testcases/org/apache/poi/xssf/SXSSFITestDataProvider.java b/src/ooxml/testcases/org/apache/poi/xssf/SXSSFITestDataProvider.java
index 4f2eb40f91..ab84979d09 100644
--- a/src/ooxml/testcases/org/apache/poi/xssf/SXSSFITestDataProvider.java
+++ b/src/ooxml/testcases/org/apache/poi/xssf/SXSSFITestDataProvider.java
@@ -64,7 +64,10 @@ public final class SXSSFITestDataProvider implements ITestDataProvider {
      */
     @Override
     public XSSFWorkbook writeOutAndReadBack(Workbook wb) {
-        if(!(wb instanceof SXSSFWorkbook)) {
+        // wb is usually an SXSSFWorkbook, but must also work on an XSSFWorkbook
+        // since workbooks must be able to be written out and read back
+        // several times in succession
+        if(!(wb instanceof SXSSFWorkbook || wb instanceof XSSFWorkbook)) {
             throw new IllegalArgumentException("Expected an instance of SXSSFWorkbook");
         }