Merge pull request skyscreamer#28 from carterpage/dependency_test

carterpage · carterpage · commit 99b9a783c6f8 · 2013-03-17T06:55:23.000-07:00
Added dependency test (but commented it out) for issue skyscreamer#25
diff --git a/src/test/java/org/skyscreamer/jsonassert/DependencyTest.java b/src/test/java/org/skyscreamer/jsonassert/DependencyTest.java
@@ -0,0 +1,24 @@
+package org.skyscreamer.jsonassert;
+
+import org.json.JSONObject;
+import org.junit.Assert;
+
+/**
+ * Unit tests for our external/third-party dependencies.
+ *
+ * @author Carter Page <carter@skyscreamer.org>
+ */
+public class DependencyTest {
+    //@Test  // For https://github.com/skyscreamer/JSONassert/issues/25
+    public void testJSonGetLong() throws Exception {
+        Long target = -4611686018427386614L;
+        String targetString = target.toString();
+
+        JSONObject value = new JSONObject().put("id", target);
+        Assert.assertEquals(target, (Long) value.getLong("id"));  //Correct: when put as long getLong is correct
+
+        value = new JSONObject().put("id", targetString);
+        Assert.assertEquals(target, (Long) Long.parseLong(value.getString("id"))); //Correct: when put as String getString is correct
+        Assert.assertEquals(target, (Long) value.getLong("id")); //Bug: Having json convert the string to long fails
+    }
+}
