Merge pull request #32 from stleary/refactor-test-classes-from-JSONObjectTest

refactor test classes to their own modules

JSONObjectTest.java

@@ -16,57 +16,17 @@ import org.json.CDL;
 import org.json.JSONArray;
 import org.json.JSONException;
 import org.json.JSONObject;
-import org.json.JSONString;
 import org.json.XML;
 import org.junit.Test;
 
 import com.jayway.jsonpath.*;
 
-/**
- * Used in testing when a JSONString is needed
- */
-class MyJsonString implements JSONString {
-
-    @Override
-    public String toJSONString() {
-        return "my string";
-    }
-}
-
-/**
- * Used in testing when Bean behavior is needed
- */
-interface MyBean {
-    public Integer getIntKey();
-    public Double getDoubleKey();
-    public String getStringKey();
-    public String getEscapeStringKey();
-    public Boolean isTrueKey();
-    public Boolean isFalseKey();
-    public StringReader getStringReaderKey();
-};
-
-/**
- * Used in testing when a Bean containing big numbers is needed
- */
-interface MyBigNumberBean {
-    public BigInteger getBigInteger();
-    public BigDecimal getBigDecimal();
-}
-
 /**
  * JSONObject, along with JSONArray, are the central classes of the reference app.
  * All of the other classes interact with them, and JSON functionality would
  * otherwise be impossible.
  */
 public class JSONObjectTest {
-    /**
-     * Need a class with some public data members for testing, so
-     * JSONObjectTest itself will be used for this purpose.
-     * TODO: Why not use MyBigNumberBean or MyBean?
-     */
-    public Integer publicInt = 42;
-    public String publicString = "abc";
 
     /**
      * JSONObject built from a bean, but only using a null value.
@@ -378,8 +338,8 @@ public class JSONObjectTest {
     public void jsonObjectByObjectAndNames() {
         String[] keys = {"publicString", "publicInt"};
         // just need a class that has public data members
-        JSONObjectTest jsonObjectTest = new JSONObjectTest();
+        MyPublicClass myPublicClass = new MyPublicClass();
-        JSONObject jsonObject = new JSONObject(jsonObjectTest, keys);
+        JSONObject jsonObject = new JSONObject(myPublicClass, keys);
 
         // validate JSON
         Object doc = Configuration.defaultConfiguration().jsonProvider().parse(jsonObject.toString());
@@ -1083,10 +1043,10 @@ public class JSONObjectTest {
         /**
          * A bean is also an object. But in order to test the static
          * method getNames(), this particular bean needs some public
-         * data members, which have been added to the class.
+         * data members.
          */
-        JSONObjectTest jsonObjectTest = new JSONObjectTest();
+        MyPublicClass myPublicClass = new MyPublicClass();
-        names = JSONObject.getNames(jsonObjectTest);
+        names = JSONObject.getNames(myPublicClass);
 
         // validate JSON
         jsonArray = new JSONArray(names);

MyBean.java

@@ -0,0 +1,16 @@
+package org.json.junit;
+
+import java.io.*;
+
+/**
+ * Used in testing when Bean behavior is needed
+ */
+interface MyBean {
+    public Integer getIntKey();
+    public Double getDoubleKey();
+    public String getStringKey();
+    public String getEscapeStringKey();
+    public Boolean isTrueKey();
+    public Boolean isFalseKey();
+    public StringReader getStringReaderKey();
+}

MyBigNumberBean.java

@@ -0,0 +1,11 @@
+package org.json.junit;
+
+import java.math.*;
+
+/**
+ * Used in testing when a Bean containing big numbers is needed
+ */
+interface MyBigNumberBean {
+    public BigInteger getBigInteger();
+    public BigDecimal getBigDecimal();
+}

MyJsonString.java

@@ -0,0 +1,14 @@
+package org.json.junit;
+
+import org.json.*;
+
+/**
+ * Used in testing when a JSONString is needed
+ */
+class MyJsonString implements JSONString {
+
+    @Override
+    public String toJSONString() {
+        return "my string";
+    }
+}

MyPublicClass.java

@@ -0,0 +1,9 @@
+package org.json.junit;
+
+/**
+ * Need a class with some public data members for testing
+ */
+public class MyPublicClass {
+    public Integer publicInt = 42;
+    public String publicString = "abc";
+}