Fix #405

Author: cowtowncoder

release-notes/VERSION-2.x

@@ -19,6 +19,7 @@ Project: jackson-dataformat-xml
  (reported by Ghenadii B)
 #397: `XmlReadContext` does not keep track of array index
 #403: Make `JsonNode` implicitly create `ArrayNode`s for repeated XML Elements
+#405: Mixed content not exposed through `FromXmlParser`, lost by `JsonNode`
 
 2.11.1 (not yet released)
 

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

@@ -2,8 +2,10 @@
 
 import com.fasterxml.jackson.databind.JsonNode;
 import com.fasterxml.jackson.databind.ObjectMapper;
+import com.fasterxml.jackson.databind.ObjectWriter;
+import com.fasterxml.jackson.databind.json.JsonMapper;
 import com.fasterxml.jackson.databind.node.JsonNodeType;
-
+import com.fasterxml.jackson.databind.node.ObjectNode;
 import com.fasterxml.jackson.dataformat.xml.XmlTestBase;
 
 public class JsonNodeBasicDeserTest extends XmlTestBase
@@ -30,4 +32,22 @@ public void testRepeated() throws Exception
         assertEquals("a", root.at("/value/0").asText());
         assertEquals("b", root.at("/value/1").asText());
     }
+
+    // [dataformat-xml#405]: support mixed content
+    public void testMixedContent() throws Exception
+    {
+        JsonNode fromXml = XML_MAPPER.readTree("<root>first<a>123</a>second<b>abc</b>last</root>");
+        final ObjectNode exp = XML_MAPPER.createObjectNode();
+        exp.putArray("")
+            .add("first")
+            .add("second")
+            .add("last");
+        exp.put("a", "123");
+        exp.put("b", "abc");
+        
+        if (!fromXml.equals(exp)) {
+            ObjectWriter w = new JsonMapper().writerWithDefaultPrettyPrinter();
+            fail("Expected:\n"+w.writeValueAsString(exp)+"\ngot:\n"+w.writeValueAsString(fromXml));
+        }
+    }
 }

src/test/java/com/fasterxml/jackson/dataformat/xml/deser/UntypedObjectDeser205Test.java renamed to src/test/java/com/fasterxml/jackson/dataformat/xml/deser/UntypedObjectDeserTest.java

@@ -8,14 +8,12 @@
 import com.fasterxml.jackson.databind.node.ObjectNode;
 import com.fasterxml.jackson.dataformat.xml.XmlTestBase;
 
-// for [dataformat-xml#205], handling "untyped" ({@code java.lang.Object}-targeted)
-// deserialization, including handling of element sequences
-public class UntypedObjectDeser205Test extends XmlTestBase
+public class UntypedObjectDeserTest extends XmlTestBase
 {
     private final ObjectMapper XML_MAPPER = newMapper();
 
-    private final ObjectMapper JSON_MAPPER = new JsonMapper();
-
+    // for [dataformat-xml#205], handling "untyped" ({@code java.lang.Object}-targeted)
+    // deserialization, including handling of element sequences
     public void testRepeatingElements() throws Exception
     {
         final String XML =
@@ -39,8 +37,8 @@ public void testRepeatingElements() throws Exception
                 "      </dog>\n" +
                 "   </dogs>\n" +
                 "</person>";
-        final JsonNode fromXml = JSON_MAPPER.valueToTree(XML_MAPPER.readValue(XML, Object.class));
-        final ObjectNode exp = JSON_MAPPER.createObjectNode();
+        final JsonNode fromXml = XML_MAPPER.valueToTree(XML_MAPPER.readValue(XML, Object.class));
+        final ObjectNode exp = XML_MAPPER.createObjectNode();
         exp.put("name", "John");
         {
             exp.putArray("parent")
@@ -60,7 +58,26 @@ public void testRepeatingElements() throws Exception
                 .put("age", "14");
         }
         if (!fromXml.equals(exp)) {
-            ObjectWriter w = JSON_MAPPER.writerWithDefaultPrettyPrinter();
+            ObjectWriter w = new JsonMapper().writerWithDefaultPrettyPrinter();
+            fail("Expected:\n"+w.writeValueAsString(exp)+"\ngot:\n"+w.writeValueAsString(fromXml));
+        }
+    }
+
+    // [dataformat-xml#405]: support mixed content
+    public void testMixedContent() throws Exception
+    {
+        final String XML = "<root>first<a>123</a>second<b>abc</b>last</root>";
+        final JsonNode fromXml = XML_MAPPER.valueToTree(XML_MAPPER.readValue(XML, Object.class));
+        final ObjectNode exp = XML_MAPPER.createObjectNode();
+        exp.putArray("")
+            .add("first")
+            .add("second")
+            .add("last");
+        exp.put("a", "123");
+        exp.put("b", "abc");
+        
+        if (!fromXml.equals(exp)) {
+            ObjectWriter w = new JsonMapper().writerWithDefaultPrettyPrinter();
             fail("Expected:\n"+w.writeValueAsString(exp)+"\ngot:\n"+w.writeValueAsString(fromXml));
         }
     }