test: fix serialization tests on Windows

Serialization tests were relying on string comparison, which fails because of line endings on Windows (string literals in source have LF newlines). Fixed by using JsonNode to parse and compare the serialization output with the expected text.
chickensoft-games · Jun 20, 2024 · cff68ff · cff68ff
1 parent 9b195d6
commit cff68ff
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 10 deletions.
diff --git a/Chickensoft.LogicBlocks.Tests/test/src/LogicBlock.SerializationTest.cs b/Chickensoft.LogicBlocks.Tests/test/src/LogicBlock.SerializationTest.cs
@@ -1,6 +1,7 @@
 namespace Chickensoft.LogicBlocks.Tests.Serialization;
 
 using System.Text.Json;
+using System.Text.Json.Nodes;
 using Chickensoft.Collections;
 using Chickensoft.Introspection;
 using Chickensoft.LogicBlocks.Tests.Fixtures;
@@ -31,10 +32,10 @@ public partial record B {
 
     var options = CreateOptions();
 
-    var json = JsonSerializer.Serialize(logic, options);
+    var jsonText = JsonSerializer.Serialize(logic, options);
+    var jsonNode = JsonNode.Parse(jsonText);
 
-    json.ShouldBe(
-      /*lang=json,strict*/
+    var jsonExpectedText = /*lang=json,strict*/
       """
       {
         "$type": "serializable_logic_block",
@@ -60,8 +61,9 @@ public partial record B {
           }
         }
       }
-      """
-    );
+      """;
+    var jsonExpectedNode = JsonNode.Parse(jsonExpectedText);
+    JsonNode.DeepEquals(jsonNode, jsonExpectedNode).ShouldBeTrue();
   }
 
   [Fact]

diff --git a/Chickensoft.LogicBlocks.Tutorial.Tests/SerializableLogicBlockTest.cs b/Chickensoft.LogicBlocks.Tutorial.Tests/SerializableLogicBlockTest.cs
@@ -1,6 +1,7 @@
 namespace Chickensoft.LogicBlocks.Tutorial.Tests;
 
 using System.Text.Json;
+using System.Text.Json.Nodes;
 using Chickensoft.Serialization;
 using Shouldly;
 using Xunit;
@@ -19,10 +20,10 @@ public class SerializableLogicBlockTest {
 
     var logic = new SerializableLogicBlock();
 
-    var json = JsonSerializer.Serialize(logic, options);
+    var jsonText = JsonSerializer.Serialize(logic, options);
+    var jsonNode = JsonNode.Parse(jsonText);
 
-    json.ShouldBe(
-      /*language=json*/
+    var jsonExpectedText = /*language=json*/
       """
       {
         "$type": "serializable_logic",
@@ -37,8 +38,10 @@ public class SerializableLogicBlockTest {
           "values": {}
         }
       }
-      """
-    );
+      """;
+    var jsonExpectedNode = JsonNode.Parse(jsonExpectedText);
+
+    JsonNode.DeepEquals(jsonNode, jsonExpectedNode).ShouldBeTrue();
   }
 
   [Fact]