refactoring, move test for #47 out of 'failing' as it now passes

src/test/java/com/fasterxml/jackson/dataformat/xml/TestListDeserialization.java

@@ -66,7 +66,7 @@ static class ListBeanUnwrapped
         @JacksonXmlElementWrapper(useWrapping=false)
         public List<Integer> values;
     }
-    
+
     /*
     /**********************************************************
     /* Unit tests

src/test/java/com/fasterxml/jackson/dataformat/xml/TestListRoundtrip.java

@@ -0,0 +1,92 @@
+package com.fasterxml.jackson.dataformat.xml;
+
+import java.util.ArrayList;
+import java.util.List;
+
+import com.fasterxml.jackson.annotation.JsonPropertyOrder;
+
+import com.fasterxml.jackson.databind.SerializationFeature;
+import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlElementWrapper;
+import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlProperty;
+import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlRootElement;
+
+public class TestListRoundtrip extends XmlTestBase
+{
+    @JacksonXmlRootElement(localName="parents")
+    public static class Parents {
+      @JacksonXmlElementWrapper(useWrapping=false)
+      public List<Parent> parent = new ArrayList<Parent>();
+    }
+
+    @JsonPropertyOrder({ "name", "desc", "prop" })
+    public static class Parent {
+      @JacksonXmlProperty(isAttribute=true)
+      public String name;
+
+      public String description;
+
+      @JacksonXmlElementWrapper(useWrapping=false)
+      public List<Prop> prop = new ArrayList<Prop>();
+
+      public Parent() { }
+      public Parent(String name, String desc) {
+          this.name = name;
+          description = desc;
+      }
+    }
+
+    static class Prop {
+      @JacksonXmlProperty(isAttribute=true)
+      public String name;
+
+      public String value;
+
+      public Prop() { }
+      public Prop(String name, String value) {
+          this.name = name;
+          this.value = value;
+      }
+    }
+
+    /*
+    /**********************************************************
+    /* Unit tests
+    /**********************************************************
+     */
+
+    private final XmlMapper MAPPER = new XmlMapper();
+    { // easier for eye:
+        MAPPER.enable(SerializationFeature.INDENT_OUTPUT);
+    }
+
+    public void testParentListRoundtrip() throws Exception
+    {
+        Parents root = new Parents();
+        Parent parent1 = new Parent("a", "First");
+        root.parent.add(parent1);
+        parent1.prop.add(new Prop("width", "13"));
+        parent1.prop.add(new Prop("height", "10"));
+        Parent parent2 = new Parent("b", "Second");
+        parent2.prop.add(new Prop("x", "1"));
+        parent2.prop.add(new Prop("y", "2"));
+        root.parent.add(parent2);
+
+        String xml = MAPPER.writeValueAsString(root);
+        assertNotNull(xml);
+
+        // then bring it back
+        Parents result = MAPPER.readValue(xml, Parents.class);
+        assertNotNull(result.parent);
+        assertEquals(2, result.parent.size());
+        Parent p2 = result.parent.get(1);
+        assertNotNull(p2);
+        assertEquals("b", p2.name);
+        assertEquals("Second", p2.description);
+
+        assertEquals(2, p2.prop.size());
+        Prop prop2 = p2.prop.get(1);
+        assertNotNull(prop2);
+        assertEquals("2", prop2.value);
+    }
+
+}

src/test/java/com/fasterxml/jackson/dataformat/xml/failing/TestAdapterIssue37.java → src/test/java/com/fasterxml/jackson/dataformat/xml/adapters/TestAdapterIssue37.java

@@ -1,4 +1,4 @@
-package com.fasterxml.jackson.dataformat.xml.failing;
+package com.fasterxml.jackson.dataformat.xml.adapters;
 
 import java.io.UnsupportedEncodingException;
 import java.net.*;