Bug 58252

Summary: Add new CellReference constructor unit tests
Product: POI Reporter: Javen O'Neal <onealj>
Component: SS CommonAssignee: POI Developers List <dev>
Status: RESOLVED FIXED    
Severity: enhancement    
Priority: P2    
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Linux   
Attachments: TestCellReference.java patch

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.