From 334b7f24ea5db6029f62b538c7ab7a8d4c4b29e3 Mon Sep 17 00:00:00 2001
From: Niels <niels.lohmann@gmail.com>
Date: Fri, 13 Feb 2015 18:18:58 +0100
Subject: [PATCH] more systematic test cases

---
 test/unit.cpp | 59 ++++++++++++++++++++++++++++++++++-----------------
 1 file changed, 39 insertions(+), 20 deletions(-)

diff --git a/test/unit.cpp b/test/unit.cpp
index a4621c9c..99788efa 100644
--- a/test/unit.cpp
+++ b/test/unit.cpp
@@ -5218,27 +5218,46 @@ TEST_CASE("parser class")
 
         // unexpected end of string
         CHECK_THROWS_AS(json::parser("\"").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\u").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\u0").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\u01").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\u012").parse(), std::invalid_argument);
+        CHECK_THROWS_AS(json::parser("\"\\\"").parse(), std::invalid_argument);
+        CHECK_THROWS_AS(json::parser("\"\\u\"").parse(), std::invalid_argument);
+        CHECK_THROWS_AS(json::parser("\"\\u0\"").parse(), std::invalid_argument);
+        CHECK_THROWS_AS(json::parser("\"\\u01\"").parse(), std::invalid_argument);
+        CHECK_THROWS_AS(json::parser("\"\\u012\"").parse(), std::invalid_argument);
 
         // invalid escapes
-        CHECK_THROWS_AS(json::parser("\\!").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\#").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\[").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\]").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\.").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\0").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\a").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\c").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\e").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\g").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\m").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\o").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\q").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\s").parse(), std::invalid_argument);
-        CHECK_THROWS_AS(json::parser("\\v").parse(), std::invalid_argument);
+        for (int c = 1; c < 128; ++c)
+        {
+            auto s = std::string("\"\\") + std::string(1, c) + "\"";
+
+            switch (c)
+            {
+                // valid escapes
+                case ('"'):
+                case ('\\'):
+                case ('/'):
+                case ('b'):
+                case ('f'):
+                case ('n'):
+                case ('r'):
+                case ('t'):
+                {
+                    CHECK_NOTHROW(json::parser(s).parse());
+                    break;
+                }
+
+                // \u must be followed with four numbers, so we skip it here
+                case ('u'):
+                {
+                    break;
+                }
+
+                // any other combination of backslash and character is invalid
+                default:
+                {
+                    CHECK_THROWS_AS(json::parser(s).parse(), std::invalid_argument);
+                    break;
+                }
+            }
+        }
     }
 }