From: Javen O'Neal <onealj@apache.org>
Date: Sat, 11 Jun 2016 11:28:55 +0000 (+0000)
Subject: bug 57840: add disabled unit test with large number of structured reference formulas
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=be9b615048952e78a22891a181a89849ef31e4cf;p=poi.git

bug 57840: add disabled unit test with large number of structured reference formulas
patch and spreadsheet contributed by Mark Woolsey

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

diff --git a/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFFormulaEvaluation.java b/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFFormulaEvaluation.java
index a6d5bdb966..b64ef25013 100644
--- a/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFFormulaEvaluation.java
+++ b/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFFormulaEvaluation.java
@@ -609,10 +609,20 @@ public final class TestXSSFFormulaEvaluation extends BaseTestFormulaEvaluator {
     // bug 57721
     @Test
     public void structuredReferences() throws IOException {
-        XSSFWorkbook wb = XSSFTestDataSamples.openSampleWorkbook("evaluate_formula_with_structured_table_references.xlsx");
-        
-        XSSFFormulaEvaluator.evaluateAllFormulaCells(wb);
+        verifyAllFormulasInWorkbookCanBeEvaluated("evaluate_formula_with_structured_table_references.xlsx");
+    }
+    
+    // bug 57840
+    @Ignore("Takes over a minute to evaluate all formulas in this large workbook. Run this test when profiling for formula evaluation speed.")
+    @Test
+    public void testLotsOfFormulasWithStructuredReferencesToCalculatedTableColumns() throws IOException {
+        verifyAllFormulasInWorkbookCanBeEvaluated("StructuredRefs-lots-with-lookups.xlsx");
+    }
 
+    // FIXME: use junit4 parameterization
+    private static void verifyAllFormulasInWorkbookCanBeEvaluated(String sampleWorkbook) throws IOException {
+        XSSFWorkbook wb = XSSFTestDataSamples.openSampleWorkbook(sampleWorkbook);
+        XSSFFormulaEvaluator.evaluateAllFormulaCells(wb);
         wb.close();
     }
 
diff --git a/test-data/spreadsheet/StructuredRefs-lots-with-lookups.xlsx b/test-data/spreadsheet/StructuredRefs-lots-with-lookups.xlsx
new file mode 100644
index 0000000000..11845f82fa
Binary files /dev/null and b/test-data/spreadsheet/StructuredRefs-lots-with-lookups.xlsx differ