Pass test case for empty tag with forceList

Author: Zach

src/main/java/org/json/XML.java

@@ -415,7 +415,7 @@ private static boolean parse(XMLTokener x, JSONObject context, String name, XMLP
                             if (parse(x, jsonObject, tagName, config)) {
                                 if (config.getForceList().contains(tagName)) {
                                     if (jsonObject.length() == 0) {
-                                        context.append(tagName, "");
+                                        context.put(tagName, new JSONArray());
                                     } else if (jsonObject.length() == 1
                                             && jsonObject.opt(config.getcDataTagName()) != null) {
                                         context.append(tagName, jsonObject.opt(config.getcDataTagName()));

src/test/java/org/json/junit/XMLConfigurationTest.java

@@ -919,7 +919,8 @@ public void testSimpleForceList() {
                 "   </address>\n"+
                 "</addresses>";
 
-        String expectedStr = "{\"addresses\":[{\"address\":{\"name\":\"Sherlock Holmes\"}}]}";
+        String expectedStr = 
+                "{\"addresses\":[{\"address\":{\"name\":\"Sherlock Holmes\"}}]}";
 
         Set<String> forceList = new HashSet<String>();
         forceList.add("addresses");
@@ -949,18 +950,18 @@ public void testLongForceList() {
                 "</servers>";
 
         String expectedStr = 
-            "{"+
-            "\"servers\": ["+
-              "{"+
-                "\"server\": {"+
-                  "\"name\": \"host1\","+
-                  "\"os\": \"Linux\","+
-                  "\"interfaces\": ["+
-                   "{"+
-                     "\"interface\": {"+
-                       "\"name\": \"em0\","+
-                       "\"ip_address\": \"10.0.0.1\""+
-                      "}}]}}]}";
+                "{"+
+                    "\"servers\": ["+
+                        "{"+
+                            "\"server\": {"+
+                            "\"name\": \"host1\","+
+                            "\"os\": \"Linux\","+
+                            "\"interfaces\": ["+
+                                "{"+
+                                    "\"interface\": {"+
+                                    "\"name\": \"em0\","+
+                                    "\"ip_address\": \"10.0.0.1\""+
+                                    "}}]}}]}";
 
         Set<String> forceList = new HashSet<String>();
         forceList.add("servers");
@@ -974,7 +975,25 @@ public void testLongForceList() {
 
         Util.compareActualVsExpectedJsonObjects(jsonObject, expetedJsonObject);
     }
-    
+    @Test
+    public void testEmptyForceList() {
+        String xmlStr = 
+                "<addresses></addresses>";
+
+        String expectedStr = 
+                "{\"addresses\":[]}";
+        
+        Set<String> forceList = new HashSet<String>();
+        forceList.add("addresses");
+
+        XMLParserConfiguration config = 
+                new XMLParserConfiguration()
+                        .withForceList(forceList);
+        JSONObject jsonObject = XML.toJSONObject(xmlStr, config);
+        JSONObject expetedJsonObject = new JSONObject(expectedStr);
+
+        Util.compareActualVsExpectedJsonObjects(jsonObject, expetedJsonObject);
+    }
 
     /**
      * Convenience method, given an input string and expected result,