chore: add ManagedReference YAML roundtrip tests

dotnet · Sep 26, 2024 · 46ab297 · 46ab297
1 parent 19e7a97
commit 46ab297
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/Docfx.Common/Json/JsonUtility.cs b/src/Docfx.Common/Json/JsonUtility.cs
@@ -94,6 +94,9 @@ private static bool IsSupported()
                 case "Docfx.DataContracts.ManagedReference.PageViewModel":
                     return true;
 
+                case "Docfx.DataContracts.ManagedReference.PageViewModel":
+                    return false; // TODO: Need to support ExtensionData
+
                 // Intermediate types for tests. it's expected to be removed later (And return true by default).
                 case "Docfx.Plugins.MarkdownServiceProperties":
                     return true;

diff --git a/test/docfx.Tests/SerializationTests/YamlSerializationTest.ManagedReference.cs b/test/docfx.Tests/SerializationTests/YamlSerializationTest.ManagedReference.cs
@@ -18,7 +18,7 @@ public void YamlSerializationTest_ManagedReference(string path)
         var model = TestData.Load<PageViewModel>(path);
 
         // Act/Assert
-        ValidateYamlRoundTrip(model);
         ValidateYamlJsonRoundTrip(model);
+        ValidateYamlRoundTripWithJson(model);
     }
 }