add cmd line test for json output, use similar()

src/main/java/edu/sc/seis/TauP/Arrival.java

@@ -614,7 +614,7 @@ public void writeJSON(PrintWriter pw, String indent) throws IOException {
             } else {
                 pw.write(innerIndent+"  "+JSONWriter.valueToString("error")+": "+JSONWriter.valueToString("geometrical speading not finite")+","+NL);
             }
-            pw.write(innerIndent+"  "+JSONWriter.valueToString("refltranpsv")+": "+JSONWriter.valueToString((float)getReflTransPSV())+NL);
+            pw.write(innerIndent+"  "+JSONWriter.valueToString("refltranpsv")+": "+JSONWriter.valueToString((float)getReflTransPSV())+", "+NL);
             pw.write(innerIndent+"  "+JSONWriter.valueToString("refltransh")+": "+JSONWriter.valueToString((float)getReflTransSH())+NL);
             pw.write(innerIndent+"}");
         } catch (TauPException e) {

src/test/java/edu/sc/seis/TauP/CmdLineOutputTest.java

@@ -5,6 +5,8 @@
 import static org.junit.jupiter.api.Assertions.assertNotNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
+import org.json.JSONObject;
+import org.json.JSONTokener;
 import org.junit.jupiter.api.Test;
 
 import java.io.BufferedOutputStream;
@@ -112,6 +114,10 @@ public class CmdLineOutputTest {
                                           "taup wkbj --help",
     };
 
+    String[] jsonTestCmds = new String[] {
+            "taup time -h 10 -ph P -deg 35 --json",
+    };
+
     String versionCmd = "taup --version";
 
     String[] docCmds = new String[] {
@@ -144,6 +150,7 @@ public void regenSavedOutput() throws Exception {
         allList.addAll(Arrays.asList(velplotTestCmds));
         allList.addAll(Arrays.asList(reflTransPlotTestCmds));
         allList.addAll(Arrays.asList(phaseDescribeTestCmds));
+        allList.addAll(Arrays.asList(jsonTestCmds));
         for (String cmd : allList) {
             System.err.println(cmd);
             saveTestOutputToFile(cmd);
@@ -318,6 +325,11 @@ public void testTauPReflTransplot() throws Exception {
         runTests(reflTransPlotTestCmds);
     }
 
+
+    @Test
+    public void testTauPJSON() throws Exception {
+        runJsonTests(jsonTestCmds);
+    }
     @Test
     public void testTauPTable() throws Exception {
         // this one takes a lot of memory
@@ -330,6 +342,12 @@ public void runTests(String[] cmds) throws Exception {
         }
     }
 
+    public void runJsonTests(String[] cmds) throws Exception {
+        for (int i = 0; i < cmds.length; i++) {
+            testJsonCmd(cmds[i]);
+        }
+    }
+
     public void runCmd(String cmd) throws Exception {
         String[] s = cmd.split(" +");
         String tool = s[0];
@@ -371,6 +389,19 @@ public void testCmd(String cmd) throws Exception {
         origErr.println("Done with " + cmd);
     }
 
+    public void testJsonCmd(String cmd) throws Exception {
+        setUpStreams();
+        assertEquals(0, outContent.toByteArray().length, "sysout is not empty");
+        runCmd(cmd);
+        BufferedReader prior = getPriorOutput(cmd);
+        BufferedReader current = new BufferedReader(new InputStreamReader(new ByteArrayInputStream(outContent.toByteArray())));
+        JSONTokener jsonIn = new JSONTokener(prior);
+        JSONObject priorJson = new JSONObject(jsonIn);
+        JSONTokener currentIn = new JSONTokener(current);
+        JSONObject currentJson = new JSONObject(currentIn);
+        assertTrue(priorJson.similar(currentJson), currentJson.toString(2));
+    }
+
     /**
      * test loading prior results text file from test resources. Kind of a meta-test... :)
      * @throws Exception

src/test/resources/edu/sc/seis/TauP/cmdLineTest/taup_time_-h_10_-ph_P_-deg_35_--json

@@ -0,0 +1,25 @@
+{
+  "model": "iasp91",
+  "sourcedepth": 10,
+  "receiverdepth": 0,
+  "phases": [ "P" ],
+  "arrivals": [
+    {
+      "distdeg": 35,
+      "phase": "P",
+      "time": 412.4329,
+      "rayparam": 8.613012,
+      "takeoff": 26.741543,
+      "incident": 26.69624,
+      "puristdist": 35,
+      "puristname": "P",
+      "amp": {
+        "factorpsv": 2.0197284E-9,
+        "factorsh": 0,
+        "geospread": 8.546455E-9,
+        "refltranpsv": 0.9386537, 
+        "refltransh": 0
+      }
+    }
+  ]
+}