add decde overload for JsonString (#127)

Author: arnetheduck

json_serialization/format.nim

@@ -100,6 +100,14 @@ when declared(macrocache.hasKey): # Nim 1.6 have no macrocache.hasKey
   generateJsonAutoSerializationAddon(DummyFlavor)
   DummyFlavor.automaticBuiltinSerialization(false)
 
+template decode*(
+    Format: type Json,
+    inputParam: JsonString,
+    RecordType: type,
+    params: varargs[untyped],
+): auto =
+  decode(Format, string(inputParam), RecordType, params)
+
 template createJsonFlavor*(FlavorName: untyped,
                            mimeTypeValue = "application/json",
                            automaticObjectSerialization = false,
@@ -118,6 +126,14 @@ template createJsonFlavor*(FlavorName: untyped,
     template PreferredOutputType*(T: type FlavorName): type = string
     template mimeType*(T: type FlavorName): string = mimeTypeValue
 
+  template decode*(
+      Format: type FlavorName,
+      inputParam: JsonString,
+      RecordType: type,
+      params: varargs[untyped],
+  ): auto =
+    decode(Format, string(inputParam), RecordType, params)
+
   template flavorUsesAutomaticObjectSerialization*(T: type FlavorName): bool = automaticObjectSerialization
   template flavorOmitsOptionalFields*(T: type FlavorName): bool = omitOptionalFields
   template flavorRequiresAllFields*(T: type FlavorName): bool = requireAllFields

tests/test_json_flavor.nim

@@ -130,8 +130,9 @@ suite "Test JsonFlavor":
 
   test "object with null fields":
     expect JsonReaderError:
-      let x = Json.decode(jsonTextWithNullFields, Container)
-      discard x
+      discard Json.decode(jsonTextWithNullFields, Container)
+    expect JsonReaderError:
+      discard Json.decode(JsonString(jsonTextWithNullFields), Container)
 
     let x = NullyFields.decode(jsonTextWithNullFields, Container)
     check x.list.len == 0
@@ -203,7 +204,7 @@ suite "Test JsonFlavor":
     NullyFields.flavorEnumRep(EnumAsStringifiedNumber)
     let x = NullyFields.encode(JackFruit)
     check x == "\"2\""
-    
+
     NullyFields.flavorEnumRep(EnumAsNumber)
     let z = NullyFields.encode(Banana)
     check z == "0"

tests/test_serialization.nim

@@ -608,8 +608,11 @@ suite "toJson tests":
     """
 
     expect IncompleteObjectError:
-      let shouldNotDecode = Json.decode(json, Simple, requireAllFields = true)
-      echo "This should not have decoded ", shouldNotDecode
+      discard Json.decode(json, Simple, requireAllFields = true)
+
+    expect IncompleteObjectError:
+      # JsonString overload parameters forwarding..
+      discard Json.decode(JsonString(json), Simple, requireAllFields = true)
 
   test "all fields were required, but not all were provided (additional fields present instead)":
     let json = test_dedent"""