[DX-388] Fix BigIntegerJsonConverters (#14)

Author: v16Studios

src/Enjin.Platform.Sdk/Enjin.Platform.Sdk.Tests/Unit/Json/BigIntegerJsonConverterTest.cs

@@ -37,7 +37,6 @@ public void DeserializeWhenGivenValidJsonStringReturnsExpected(string json)
 
     [Test]
     [TestCase(@"null")]
-    [TestCase(@"0")]
     [TestCase(@"""1.1579E+77""")]
     [TestCase(@"""1.0""")]
     [TestCase(@"[]")]

src/Enjin.Platform.Sdk/Enjin.Platform.Sdk.Tests/Unit/Json/NullableBigIntegerJsonConverterTest.cs

@@ -53,7 +53,6 @@ public void DeserializeWhenGivenNullJsonValueReturnsNull()
     }
 
     [Test]
-    [TestCase(@"0")]
     [TestCase(@"""1.1579E+77""")]
     [TestCase(@"""1.0""")]
     [TestCase(@"[]")]

src/Enjin.Platform.Sdk/Enjin.Platform.Sdk/Json/BigIntegerJsonConverter.cs

@@ -23,14 +23,15 @@ public class BigIntegerJsonConverter : JsonConverter<BigInteger>
     /// <inheritdoc/>
     /// </para>
     /// <para>
-    /// When returned in a response, the <c>BigInt</c> type on the platform is expected to be returned as a string and
-    /// as such this converter only expects string.
+    /// When returned in a response, the <c>BigInt</c> type on the platform is expected to be returned as an int or a
+    /// string and as such this converter only expects number or string types.
     /// </para>
     /// </remarks>
     public override BigInteger Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
     {
         return reader.TokenType switch
         {
+            JsonTokenType.Number => new BigInteger(reader.GetInt32()),
             JsonTokenType.String => BigInteger.Parse(reader.GetString()!, NumberStyles.Integer),
             _ => throw new FormatException($"Invalid {nameof(JsonTokenType)} for {nameof(BigInteger)} field")
         };

src/Enjin.Platform.Sdk/Enjin.Platform.Sdk/Json/NullableBigIntegerJsonConverter.cs

@@ -24,14 +24,15 @@ public class NullableBigIntegerJsonConverter : JsonConverter<BigInteger?>
     /// <inheritdoc/>
     /// </para>
     /// <para>
-    /// When returned in a response, the <c>BigInt</c> type on the platform is expected to be returned as a string and
-    /// as such this converter only expects string or null JSON types.
+    /// When returned in a response, the <c>BigInt</c> type on the platform is expected to be returned as an int or a
+    ///  string and as such this converter only expects number, string or null JSON types.
     /// </para>
     /// </remarks>
     public override BigInteger? Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
     {
         return reader.TokenType switch
         {
+            JsonTokenType.Number => new BigInteger(reader.GetInt32()),
             JsonTokenType.String => BigInteger.Parse(reader.GetString()!, NumberStyles.Integer),
             JsonTokenType.Null => null,
             _ => throw new FormatException($"Invalid {nameof(JsonTokenType)} for {nameof(Nullable<BigInteger>)} field")