Put back comments

Author: JooHyukKim

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

@@ -14,6 +14,10 @@
 
 import static org.junit.jupiter.api.Assertions.*;
 
+/**
+ * Tests for verifying that Lists (and arrays) can be serialized even
+ * when they are root values.
+ */
 public class RootListHandlingTest extends XmlTestUtil
 {
     @JsonRootName("SR")

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

@@ -13,6 +13,7 @@
 import static java.util.Arrays.asList;
 import static org.junit.Assert.*;
 
+// [dataformat-xml#584]
 public class StringListRoundtripTest
 {
     private final static String[] TEST_DATA = new String[] {"", "test", null, "test2"};

src/test/java/com/fasterxml/jackson/dataformat/xml/misc/ArrayConversionsTest.java

@@ -13,6 +13,10 @@
 
 import static org.junit.jupiter.api.Assertions.*;
 
+/**
+ * Conversion tests to ensure that standard ObjectMapper conversions
+ * work despite XmlMapper having to add XML-specific work-arounds.
+ */
 public class ArrayConversionsTest extends XmlTestUtil
 {
     static class IntListWrapper {

src/test/java/com/fasterxml/jackson/dataformat/xml/misc/BadEncodingTest.java

@@ -9,6 +9,7 @@
 
 import static org.junit.jupiter.api.Assertions.assertNotNull;
 
+// [dataformat-xml#428]: problem with an encoding supported via JDK
 public class BadEncodingTest extends XmlTestUtil
 {
 //    private static final String xml = "<?xml version='1.0' encoding='WINDOWS-1252'?><x/>";

src/test/java/com/fasterxml/jackson/dataformat/xml/misc/CustomAnnotationIntrospectorNoWrapperTest.java

@@ -16,6 +16,9 @@
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
+/**
+ * A regression test for https://github.com/FasterXML/jackson-databind/issues/4595
+ */
 public class CustomAnnotationIntrospectorNoWrapperTest extends XmlTestUtil
 {
     public static class Foo {

src/test/java/com/fasterxml/jackson/dataformat/xml/misc/DTDSupportTest.java

@@ -9,6 +9,7 @@
 
 import static org.junit.jupiter.api.Assertions.fail;
 
+// for [databind-xml#211]
 public class DTDSupportTest extends XmlTestUtil
 {
     @Test

src/test/java/com/fasterxml/jackson/dataformat/xml/misc/ObjectId104Test.java

@@ -13,6 +13,9 @@
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertSame;
 
+/**
+ * Unit test to showcase issue #104, based on TestObjectIdDeserialization unit test in databind package.
+ */
 public class ObjectId104Test extends XmlTestUtil {
     // // Classes for external id from property annotations:
 

src/test/java/com/fasterxml/jackson/dataformat/xml/misc/RootNameTest.java

@@ -15,6 +15,9 @@
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.fail;
 
+// NOTE: even tho `@JacksonXmlRootElement` will be deprecated in near
+// future (possibly in 2.13) -- to be replaced by `@JsonRootName` -- this
+// test will use it to ensure we handle both annotations as expected
 public class RootNameTest extends XmlTestUtil
 {
     static class RootBeanBase

src/test/java/com/fasterxml/jackson/dataformat/xml/misc/RootNameWrapping374Test.java

@@ -11,6 +11,8 @@
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertNotNull;
 
+// Test originally for [dataformat-xml#374] but later (2.13)
+// for [dataformat-xml#485]
 public class RootNameWrapping374Test extends XmlTestUtil
 {
     @JsonRootName("Root")

src/test/java/com/fasterxml/jackson/dataformat/xml/misc/UnwrappedJsonIdentityConflict286Test.java

@@ -12,6 +12,8 @@
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertNotNull;
 
+// for [dataformat-xml#286]: parser getting confused with unwrapped lists,
+// object id
 public class UnwrappedJsonIdentityConflict286Test extends XmlTestUtil
 {
     static class Town