From: Tim Allison <tallison@apache.org>
Date: Tue, 18 Oct 2016 16:54:12 +0000 (+0000)
Subject: BUG 59858 -- avoid npe when there's a collision of record names...doh. Fix unit test.
X-Git-Tag: REL_3_16_BETA1~91
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=2732e72385829cfc1481da94ec0b092ecd01fb88;p=poi.git

BUG 59858 -- avoid npe when there's a collision of record names...doh. Fix unit test.

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

diff --git a/src/testcases/org/apache/poi/poifs/macros/TestVBAMacroReader.java b/src/testcases/org/apache/poi/poifs/macros/TestVBAMacroReader.java
index 3a4c47361d..a001ad9896 100644
--- a/src/testcases/org/apache/poi/poifs/macros/TestVBAMacroReader.java
+++ b/src/testcases/org/apache/poi/poifs/macros/TestVBAMacroReader.java
@@ -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