From 16fa69c0f60d13c371aa0fd42d4bd493b503ff61 Mon Sep 17 00:00:00 2001
From: stleary <stleary@gmail.com>
Date: Fri, 17 Jul 2015 22:04:01 -0500
Subject: [PATCH] investigate XML.toString() behavior with null JSONObject
 values

---
 JSONObjectTest.java | 19 ++++++++++++-------
 1 file changed, 12 insertions(+), 7 deletions(-)

diff --git a/JSONObjectTest.java b/JSONObjectTest.java
index 7efd2d8..b818a9e 100644
--- a/JSONObjectTest.java
+++ b/JSONObjectTest.java
@@ -1498,7 +1498,8 @@ public class JSONObjectTest {
          *      These are three literal name tokens: ...
          *      null 
          * 
-         * There seems to be no best practice to follow, it's all about what we want the code to do.
+         * There seems to be no best practice to follow, it's all about what we
+         * want the code to do.
          */
 
         // add JSONObject.NULL then convert to string in the manner of XML.toString() 
@@ -1506,14 +1507,17 @@ public class JSONObjectTest {
         Object obj = JSONObject.NULL;
         jsonObjectJONull.put("key", obj);
         Object value = jsonObjectJONull.opt("key");
-        assertTrue("opt() JSONObject.NULL should find JSONObject.NULL", obj.equals(value));
+        assertTrue("opt() JSONObject.NULL should find JSONObject.NULL",
+                obj.equals(value));
         value = jsonObjectJONull.get("key");
-        assertTrue("get() JSONObject.NULL should find JSONObject.NULL", obj.equals(value));
+        assertTrue("get() JSONObject.NULL should find JSONObject.NULL",
+                obj.equals(value));
         if (value == null) {
             value = "";
         }
         String string = value instanceof String ? (String)value : null;
-        assertTrue("XML toString() should convert JSONObject.NULL to null", string == null);
+        assertTrue("XML toString() should convert JSONObject.NULL to null",
+                string == null);
 
         // now try it with null
         JSONObject jsonObjectNull = new JSONObject();
@@ -1536,11 +1540,12 @@ public class JSONObjectTest {
          * if the key val is "content", then value.toString() will be 
          * called. This will evaluate to "null" for JSONObject.NULL,
          * and the empty string for null.
-         * But if the key is anything else, then JSONObject.NULL will be emitted as <key>null</key> 
-         * and null will be emitted as ""
+         * But if the key is anything else, then JSONObject.NULL will be emitted
+         * as <key>null</key> and null will be emitted as ""
          */
         String sJONull = XML.toString(jsonObjectJONull);
-        assertTrue("JSONObject.NULL should emit a null value", "<key>null</key>".equals(sJONull));
+        assertTrue("JSONObject.NULL should emit a null value", 
+                "<key>null</key>".equals(sJONull));
         String sNull = XML.toString(jsonObjectNull);
         assertTrue("null should emit an empty string", "".equals(sNull));
     }