Add a failing test for #498

Author: cowtowncoder

src/test/java/com/fasterxml/jackson/dataformat/xml/deser/MapDeserializationTest.java

@@ -1,5 +1,7 @@
 package com.fasterxml.jackson.dataformat.xml.deser;
 
+import java.util.Collections;
+import java.util.LinkedHashMap;
 import java.util.Map;
 
 import com.fasterxml.jackson.dataformat.xml.XmlMapper;
@@ -13,13 +15,13 @@ public class MapDeserializationTest extends XmlTestBase
     /**********************************************************
      */
 
-    private final XmlMapper MAPPER = newMapper();
+    private final XmlMapper XML_MAPPER = newMapper();
 
     // [dataformat-xml#14]
     public void testMapWithAttr() throws Exception
     {
         final String xml = "<order><person lang='en'>John Smith</person></order>";
-        Map<?,?> map = MAPPER.readValue(xml, Map.class);
+        Map<?,?> map = XML_MAPPER.readValue(xml, Map.class);
 
      // Will result in equivalent of:
      // { "person" : {
@@ -32,6 +34,10 @@ public void testMapWithAttr() throws Exception
      // we would just have '{ "person" : "John Smith" }'
 
          assertNotNull(map);
+         assertEquals(1, map.size());
+         Map<String,Object> inner = new LinkedHashMap<>();
+         inner.put("lang", "en");
+         inner.put("", "John Smith");
+         assertEquals(Collections.singletonMap("person", inner), map);
     }
-
 }

src/test/java/com/fasterxml/jackson/dataformat/xml/failing/MapDeserialization498Test.java

@@ -0,0 +1,53 @@
+package com.fasterxml.jackson.dataformat.xml.failing;
+
+import java.util.Map;
+
+import com.fasterxml.jackson.databind.ObjectMapper;
+import com.fasterxml.jackson.databind.json.JsonMapper;
+
+import com.fasterxml.jackson.dataformat.xml.XmlMapper;
+import com.fasterxml.jackson.dataformat.xml.XmlTestBase;
+
+public class MapDeserialization498Test extends XmlTestBase
+{
+    /*
+    /**********************************************************************
+    /* Test methods
+    /**********************************************************************
+     */
+
+    private final XmlMapper XML_MAPPER = newMapper();
+
+    private final ObjectMapper JSON_MAPPER = new JsonMapper();
+
+    // [dataformat-xml#498]
+    public void testRootLevelMap() throws Exception
+    {
+        final String xml = "<result>\n"
+                + "  <hello>world</hello>\n"
+                + "  <lists>1</lists>\n"
+                + "  <lists>2</lists>\n"
+                + "  <lists></lists>\n"
+                + "  <lists>\n"
+                + "    <inner>internal</inner>\n"
+                + "    <time>123</time>\n"
+                + "  </lists>\n"
+                + "  <lists>3</lists>\n"
+                + "  <lists>test</lists>\n"
+                + "  <lists></lists>\n"
+                + "  <single>one</single>\n"
+                + "</result>";
+
+        Map<?,?> expMap = JSON_MAPPER.readValue(a2q(
+"{'hello':'world','lists':['1','2','',"
++"{'inner':'internal','time':'123'},'3','test',''],'single':'one'}"),
+                Map.class);
+
+        Map<?,?> map = XML_MAPPER.readValue(xml, Map.class);
+
+// Work around: nominal target type of Object
+//        Map<?,?> map = (Map<?,?> ) XML_MAPPER.readValue(xml, Object.class);
+
+        assertEquals(expMap, map);
+    }
+}