Fix for #460 (actually just verification test, release notes; real fi…

…x in jackson-databind)

release-notes/VERSION-2.x

@@ -8,6 +8,7 @@ Project: jackson-dataformat-xml
 
 #456: Fix JsonAlias with unwrapped lists
  (contributed by Westin M)
+#460: Deserialization from blank (not empty) String fails for Collections
 
 2.12.2 (03-Mar-2021)
 

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

@@ -8,7 +8,7 @@
 // for [dataformat-xml#441]
 public class JsonNodeBasicSer441Test extends XmlTestBase
 {
-    final private ObjectMapper XML_MAPPER = newMapper();
+    private final ObjectMapper XML_MAPPER = newMapper();
 
     public void testSimpleNode() throws Exception
     {

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

@@ -0,0 +1,36 @@
+package com.fasterxml.jackson.dataformat.xml.failing;
+
+import java.util.List;
+
+import com.fasterxml.jackson.databind.ObjectMapper;
+import com.fasterxml.jackson.dataformat.xml.XmlTestBase;
+import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlElementWrapper;
+
+public class ListDeser458Test extends XmlTestBase
+{
+    static class Channel458 {
+        public String channelId;
+    }
+
+    static class ChannelSet458 {
+        public String setId;
+
+        // is default but just for readability
+        @JacksonXmlElementWrapper(useWrapping = true)
+        public List<Channel458> channels;
+    }
+
+    private final ObjectMapper XML_MAPPER = newMapper();
+
+    public void testIssue458() throws Exception
+    {
+        String input = "<ChannelSet458>\n" +
+                "<setId>2</setId>\n" +
+                "<channels>\n" +
+                "</channels>\n" +
+                "</ChannelSet458>";
+        ChannelSet458 inputProxyChannelStatus = XML_MAPPER.readValue(input, ChannelSet458.class);
+        assertEquals("List should be empty", 0,
+                inputProxyChannelStatus.channels.size());
+    }
+}

src/test/java/com/fasterxml/jackson/dataformat/xml/lists/EmptyListDeserTest.java

@@ -38,14 +38,27 @@ static class Entry
     static class Value319 {
         public Long orderId, orderTypeId;
     }    
-
+
+    // [dataformat-xml#460]
+    static class Channel460 {
+        public String channelId;
+    }
+
+    static class ChannelSet460 {
+        public String setId;
+
+        // is default but just for readability
+        @JacksonXmlElementWrapper(useWrapping = true)
+        public List<Channel460> channels;
+    }
+
     /*
     /**********************************************************
     /* Test methods
     /**********************************************************
      */
 
-    private final XmlMapper MAPPER = new XmlMapper();
+    private final XmlMapper MAPPER = newMapper();
 
     // [dataformat-xml#124]
     public  void test124() throws Exception {
@@ -101,4 +114,17 @@ public void testEmptyListAsNull435() throws Exception
         });
         assertNull(result);
     }
+
+    // [dataformat-xml#460]
+    public void testWrappedEmptyListWithWhitespace458() throws Exception
+    {
+        String input = "<ChannelSet460>\n" +
+                "<setId>2</setId>\n" +
+                "<channels>\n" +
+                "</channels>\n" +
+                "</ChannelSet460>";
+        ChannelSet460 set = MAPPER.readValue(input, ChannelSet460.class);
+        assertEquals("List should be empty", 0,
+                set.channels.size());
+    }
 }