Bug 58252 - Add new CellReference constructor unit tests
Summary: Add new CellReference constructor unit tests
Alias: None
Product: POI
Classification: Unclassified
Component: SS Common (show other bugs)
Version: unspecified
Hardware: PC Linux
: P2 enhancement (vote)
Target Milestone: ---
Assignee: POI Developers List
Depends on:
Reported: 2015-08-18 03:17 UTC by Javen O'Neal
Modified: 2015-08-18 14:33 UTC (History)
0 users

TestCellReference.java patch (2.00 KB, patch)
2015-08-18 03:17 UTC, Javen O'Neal
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Javen O'Neal 2015-08-18 03:17:04 UTC
Created attachment 33005 [details]
TestCellReference.java patch

It looks like the org.apache.poi.ss.util.TestCellReference unit test only covers 3 of the constructors, CellReference(String), CellReference(int, int), and formatAsString is not tested for all possible inputs.

I've added these two cases to the TestCellReference in the attached patch.
Comment 1 Nick Burch 2015-08-18 14:33:15 UTC
Thanks, patch committed in r1696427.