Add test for non-string map key format on .NET 8

cmeeren · Mar 22, 2024 · c5706fe · c5706fe
1 parent 2cbb43b
commit c5706fe
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/src/Faqt.Tests/Formatting.fs b/src/Faqt.Tests/Formatting.fs
@@ -1107,6 +1107,19 @@ Value:
 """
 
 
+    [<Fact>]
+    let ``Map format with non-string keys`` () =
+        fun () -> ().Should().FailWith("Value", Map.ofList [ 1, "a"; 2, "b" ])
+        |> assertExnMsg
+            """
+Subject: ()
+Should: FailWith
+Value:
+  '1': a
+  '2': b
+"""
+
+
 module YamlFormatterBuilder =
 
 

diff --git a/src/Faqt/Formatting.fs b/src/Faqt/Formatting.fs
@@ -409,7 +409,6 @@ type YamlFormatterBuilder = private {
                     .WithUnionUnwrapSingleFieldCases()
                     .WithUnwrapOption(false)
 #if NET8_0_OR_GREATER
-                    // TODO: Test when upgrading test project to .NET 8
                     .WithMapFormat(MapFormat.Object)
 #endif
             )