Author: tallison
Date: Tue Oct 18 16:54:12 2016
New Revision: 1765482
URL: http://svn.apache.org/viewvc?rev=1765482&view=rev
Log:
BUG 59858 -- avoid npe when there's a collision of record names...doh. Fix unit
test.
Modified:
poi/trunk/src/testcases/org/apache/poi/poifs/macros/TestVBAMacroReader.java
Modified:
poi/trunk/src/testcases/org/apache/poi/poifs/macros/TestVBAMacroReader.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/poifs/macros/TestVBAMacroReader.java?rev=1765482&r1=1765481&r2=1765482&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/poifs/macros/TestVBAMacroReader.java
(original)
+++ poi/trunk/src/testcases/org/apache/poi/poifs/macros/TestVBAMacroReader.java
Tue Oct 18 16:54:12 2016
@@ -257,11 +257,11 @@ public class TestVBAMacroReader {
@Test
public void bug59858() throws IOException {
- File f = POIDataSamples.getSpreadSheetInstance().getFile("59830.xls");
+ File f = POIDataSamples.getSpreadSheetInstance().getFile("59858.xls");
VBAMacroReader r = new VBAMacroReader(f);
Map<String, String> macros = r.readMacros();
- assertNotNull(macros.get("Module20"));
- assertContains(macros.get("Module20"), "start of superscripting");
+ assertNotNull(macros.get("Sheet4"));
+ assertContains(macros.get("Sheet4"), "intentional constituent");
}
@Test
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@poi.apache.org
For additional commands, e-mail: commits-h...@poi.apache.org
