Added additional test cases for ReadJson.

TensionDev.UUID.Serialization.JsonNet.Tests/ReadJsonTestData.cs

@@ -0,0 +1,38 @@
+using Newtonsoft.Json;
+using System;
+using Xunit;
+
+namespace TensionDev.UUID.Serialization.JsonNet.Tests
+{
+    public class ReadJsonTestData : TheoryData<JsonToken, Object, Boolean>
+    {
+        public ReadJsonTestData()
+        {
+            Add(JsonToken.String, "00000000-0000-0000-0000-000000000000", false);
+            Add(JsonToken.String, "ffffffff-ffff-ffff-ffff-ffffffffffff", false);
+            Add(JsonToken.String, "164a714c-0c79-11ec-82a8-0242ac130003", false);
+            Add(JsonToken.String, "550e8400-e29b-41d4-a716-446655440000", false);
+            Add(JsonToken.String, "1bf6935b-49e6-54cf-a9c8-51fb21c41b46", false);
+
+            Add(JsonToken.Boolean, "550e8400-e29b-41d4-a716-446655440000", true);
+            Add(JsonToken.Bytes, "550e8400-e29b-41d4-a716-446655440000", true);
+            Add(JsonToken.Date, "550e8400-e29b-41d4-a716-446655440000", true);
+            Add(JsonToken.Float, "550e8400-e29b-41d4-a716-446655440000", true);
+            Add(JsonToken.Integer, "550e8400-e29b-41d4-a716-446655440000", true);
+            Add(JsonToken.Null, "550e8400-e29b-41d4-a716-446655440000", true);
+            Add(JsonToken.Raw, "550e8400-e29b-41d4-a716-446655440000", true);
+            Add(JsonToken.Undefined, "550e8400-e29b-41d4-a716-446655440000", true);
+
+            Add(JsonToken.String, Int16.MaxValue, true);
+            Add(JsonToken.String, Int32.MaxValue, true);
+            Add(JsonToken.String, Int64.MaxValue, true);
+            Add(JsonToken.String, Single.MaxValue, true);
+            Add(JsonToken.String, Double.MaxValue, true);
+            Add(JsonToken.String, DateTime.MinValue, true);
+            Add(JsonToken.String, DateTime.MaxValue, true);
+            Add(JsonToken.String, Guid.Empty, true);
+
+            Add(JsonToken.Raw, new Object(), true);
+        }
+    }
+}

TensionDev.UUID.Serialization.JsonNet.Tests/UuidJsonNetConverterTests.cs

@@ -17,48 +17,32 @@
         }
 
         [Theory]
-        [InlineData("00000000-0000-0000-0000-000000000000")]
-        [InlineData("ffffffff-ffff-ffff-ffff-ffffffffffff")]
-        [InlineData("164a714c-0c79-11ec-82a8-0242ac130003")]
-        [InlineData("550e8400-e29b-41d4-a716-446655440000")]
-        [InlineData("1bf6935b-49e6-54cf-a9c8-51fb21c41b46")]
-        public void TestReadJson(string validUuidString)
+        [ClassData(typeof(ReadJsonTestData))]
+        public void TestReadJson(JsonToken jsonToken, object value, bool throwsException)
         {
             // Arrange
             var readerMock = new Mock<JsonReader>(MockBehavior.Strict);
-            readerMock.SetupGet(r => r.TokenType).Returns(JsonToken.String);
-            readerMock.SetupGet(r => r.Value).Returns((object)validUuidString);
+            readerMock.SetupGet(r => r.TokenType).Returns(jsonToken);
+            readerMock.SetupGet(r => r.Value).Returns(value);
 
             // existingValue must be non-nullable; obtain an instance via Parse to satisfy parameter constraints.
-            Uuid existingValue = Uuid.Parse(validUuidString);
+            Uuid existingValue = Uuid.Max;
             var serializer = new JsonSerializer();
 
-            // Act
-            Uuid result = _converter.ReadJson(readerMock.Object, typeof(Uuid), existingValue, hasExistingValue: false, serializer: serializer);
-
-            // Assert
-            Assert.NotNull(result);
-            Assert.Equal(validUuidString, result.ToString());
-        }
-
-        [Fact]
-        public void TestReadJsonInvalidType()
-        {
-            // Arrange
-            const string validUuidString = "00000000-0000-0000-0000-000000000000";
-            var readerMock = new Mock<JsonReader>(MockBehavior.Strict);
-            readerMock.SetupGet(r => r.TokenType).Returns(JsonToken.Undefined);
-            readerMock.SetupGet(r => r.Value).Returns((object)validUuidString);
-
-            // existingValue must be non-nullable; obtain an instance via Parse to satisfy parameter constraints.
-            Uuid existingValue = Uuid.Parse(validUuidString);
-            var serializer = new JsonSerializer();
-
-            Assert.Throws<JsonSerializationException>(() =>
+            if (throwsException)
+            {
+                // Act and Assert
+                Assert.Throws<JsonSerializationException>(() => _converter.ReadJson(readerMock.Object, typeof(Uuid), existingValue, hasExistingValue: false, serializer: serializer));
+            }
+            else
             {
                 // Act
-                _converter.ReadJson(readerMock.Object, typeof(Uuid), existingValue, hasExistingValue: false, serializer: serializer);
-            });
+                Uuid result = _converter.ReadJson(readerMock.Object, typeof(Uuid), existingValue, hasExistingValue: false, serializer: serializer);
+
+                // Assert
+                Assert.NotNull(result);
+                Assert.Equal(value.ToString(), result.ToString());
+            }
         }
 
         [Theory]
@@ -70,10 +54,10 @@
         public void TestWriteJson(string validUuidString)
         {
             var writerMock = new Mock<JsonWriter>(MockBehavior.Strict);
             object? capturedValue = null;
             // Setup to accept any object and capture it
             writerMock.Setup(w => w.WriteValue(It.IsAny<string?>()))
                       .Callback<string?>(v => capturedValue = v)
                       .Verifiable();
 
             var serializerMock = new Mock<JsonSerializer>(MockBehavior.Loose);
@@ -84,7 +68,7 @@
             _converter.WriteJson(writerMock.Object, value, serializerMock.Object);
 
             // Assert
             writerMock.Verify(w => w.WriteValue(It.IsAny<string?>()), Times.Once, "WriteValue should be called exactly once.");
             // Ensure the captured value matches the value.ToString() result
             var expected = value.ToString();
             Assert.NotNull(capturedValue);