Fail when exceptions are not thrown as expected

The idiom was started in the first few methods, but not continued further down where JSONException was expected. False success may have resulted.
BGehrels · Jul 24, 2016 · e57881f · e57881f
1 parent efe33a1
commit e57881f
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/src/test/java/org/json/junit/JSONStringerTest.java b/src/test/java/org/json/junit/JSONStringerTest.java
@@ -60,6 +60,7 @@ public void missplacedArrayException() {
         jsonStringer.object().endObject();
         try {
             jsonStringer.array();
+            assertTrue("Expected an exception", false);
         } catch (JSONException e) {
             assertTrue("Expected an exception message",
                     "Misplaced array.".
@@ -77,6 +78,7 @@ public void missplacedEndArrayException() {
         jsonStringer.object();
         try {
             jsonStringer.endArray();
+            assertTrue("Expected an exception", false);
         } catch (JSONException e) {
             assertTrue("Expected an exception message",
                     "Misplaced endArray.".
@@ -94,6 +96,7 @@ public void missplacedEndObjectException() {
         jsonStringer.array();
         try {
             jsonStringer.endObject();
+            assertTrue("Expected an exception", false);
         } catch (JSONException e) {
             assertTrue("Expected an exception message",
                     "Misplaced endObject.".
@@ -111,6 +114,7 @@ public void missplacedObjectException() {
         jsonStringer.object().endObject();
         try {
             jsonStringer.object();
+            assertTrue("Expected an exception", false);
         } catch (JSONException e) {
             assertTrue("Expected an exception message",
                     "Misplaced object.".