Summary: | [patch] Added Excel functions MATCH, NA, SUMPRODUCT and other fixes | ||
---|---|---|---|
Product: | POI | Reporter: | Josh Micich <josh> |
Component: | HSSF | Assignee: | POI Developers List <dev> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | P2 | ||
Version: | 3.0-dev | ||
Target Milestone: | --- | ||
Hardware: | Other | ||
OS: | other | ||
Attachments: |
svn diff
tar bzip2 of newly added files (5 junit classes) new version of TestDataValidation.xls junit report on failing TestFormulasFromSpreadsheet |
Description
Josh Micich
2008-02-05 21:01:55 UTC
Created attachment 21477 [details]
svn diff
Created attachment 21478 [details]
tar bzip2 of newly added files (5 junit classes)
Created attachment 21479 [details]
new version of TestDataValidation.xls
svn diff skips binary files
This spreadsheet belongs here:
src/testcases/org/apache/poi/hssf/data/TestDataValidation.xls
Thanks for this, it's quite the mammoth patch! I've done a visual review of the patch, and it does seem fine from that. Given the size, I'll wait a few days for other people to cast an eye over it, and if no-one shouts I'll then apply it I exercised your code and TestFormulasFromSpreadsheet fails: ant -Dtestcase=org.apache.poi.hssf.record.formula.eval.TestFormulasFromSpreadsheet single-scratchpad-test ..... Test spreadsheet cell empty on row (79). Expected function name or '<END-OF-FUNCTIONS>' junit.framework.AssertionFailedError: Test spreadsheet cell empty on row (79). Expected function name or '<END-OF-FUNCTIONS>' at org.apache.poi.hssf.record.formula.eval.TestFormulasFromSpreadsheet.processFunctionGroup(TestFormulasFromSpreadsheet.java:205) at org.apache.poi.hssf.record.formula.eval.TestFormulasFromSpreadsheet.testFunctionsFromTestSpreadsheet(TestFormulasFromSpreadsheet.java:171) at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) See the attached junit report. All other tests pass. Yegor Created attachment 21492 [details]
junit report on failing TestFormulasFromSpreadsheet
You need to copy the new version of TestDataValidation.xls to src/testcases/org/apache/poi/hssf/data/ I wasn't sure about how to include modified binary files in patch submissions. The last revision of this file seems to be 615859 (Jan 28), and my work was after that so there shouldn't be any need to merge. Request for next time... If you are uploading a new version of TestDataValidation.xls again, any chance you could gzip / bzip it first? It's pretty beasting, and takes an age to download over GPRS, which is an issue when I decide to work on POI from a cafe! (Just doing a final review before applying now) Thanks for this, applied to svn Had to copy the new TestDataValidation.xls to FormulaEvalTestData.xls to get it to work. I think we need to review the fact that we have these two, and also try to figure out why TestDataValidation.xls sometimes gets changed |