From c88d06eede9a50b2eb0ab8b705abc980b448207c Mon Sep 17 00:00:00 2001
From: stleary <stleary@gmail.com>
Date: Wed, 30 Dec 2015 01:01:43 -0600
Subject: [PATCH] util cleanup

---
 JSONMLTest.java |  4 +++-
 Util.java       | 45 ---------------------------------------------
 2 files changed, 3 insertions(+), 46 deletions(-)

diff --git a/JSONMLTest.java b/JSONMLTest.java
index a16f0ea..953a39d 100644
--- a/JSONMLTest.java
+++ b/JSONMLTest.java
@@ -655,7 +655,9 @@ public class JSONMLTest {
 
         // lastly, confirm the restored JSONObject XML and JSONArray XML look
         // reasonably similar
-        Util.compareXML(jsonObjectXmlToStr, jsonArrayXmlToStr);
+        JSONObject jsonObjectFromObject = JSONML.toJSONObject(jsonObjectXmlToStr);
+        JSONObject jsonObjectFromArray = JSONML.toJSONObject(jsonArrayXmlToStr);
+        Util.compareActualVsExpectedJsonObjects(jsonObjectFromObject, jsonObjectFromArray);
     }
 
     /**
diff --git a/Util.java b/Util.java
index eb33ab5..8446690 100644
--- a/Util.java
+++ b/Util.java
@@ -94,49 +94,4 @@ public class Util {
                 value.toString().equals(expectedValue.toString()));
         }
     }
-
-    /**
-     * Sometimes test completion requires comparison of JSONArray objects that
-     * were produced from a JSONObject, and so unordered. This method is 
-     * imperfect since it only compares the array elements and won't catch
-     * JSON syntax errors but at least it does not rely on ordering
-     * <p>
-     * It is expected that the arrays to be compared come from JSONArray
-     * instances which have been rendered by toString(), and whose syntax
-     * chars have been removed.
-     * <p>
-     * TODO: why are we not simply comparing the JSONArrays?
-     * <p>
-     * @param names an array of strings for comparison
-     * @param expectedNames the other array of strings for comparison
-     */
-    public static void compareActualVsExpectedStringArrays(String[] names,
-            String [] expectedNames) {
-        assertTrue("Array lengths should be equal",
-                names.length == expectedNames.length);
-        List<String> lNames = new ArrayList<String>(Arrays.asList(names));
-        for (int i = 0; i < expectedNames.length; ++i) {
-            String expectedName = expectedNames[i];
-            assertTrue("expected to find "+expectedName, 
-                    lNames.contains(expectedName));
-            lNames.remove(expectedName);
-        }
-    }
-
-    /**
-     * This is stopgap test utility. It is meant to compare strings
-     * of XML, but it does not take ordering into account and should
-     * not be expected to work correctly with complex XML.
-     * @param aXmlStr an XML doc to be compared
-     * @param bXmlStr the other XML doc to be compared
-     */
-    public static void compareXML(String aXmlStr, String bXmlStr) {
-        // TODO For simple tests this may be adequate, but it won't work for
-        // elements with multiple attributes and possibly other cases as well.
-        // Should use XMLUnit or similar.
-        assertTrue("expected equal XML strings \naXmlStr: "+
-                aXmlStr+ "\nbXmlStr: " +bXmlStr, aXmlStr.equals(bXmlStr));
-        
-    }
-
 }