Skip to content

Commit e137e2d

Browse files
committed
chore: add ManagedReference YAML roundtrip tests
1 parent 19e7a97 commit e137e2d

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

src/Docfx.Common/Json/JsonUtility.cs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,9 +91,11 @@ private static bool IsSupported()
9191
switch (fullName)
9292
{
9393
case "Docfx.Build.Engine.XRefMap":
94-
case "Docfx.DataContracts.ManagedReference.PageViewModel":
9594
return true;
9695

96+
case "Docfx.DataContracts.ManagedReference.PageViewModel":
97+
return false; // TODO: Need to support ExtensionData
98+
9799
// Intermediate types for tests. it's expected to be removed later (And return true by default).
98100
case "Docfx.Plugins.MarkdownServiceProperties":
99101
return true;

test/docfx.Tests/SerializationTests/YamlSerializationTest.ManagedReference.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,6 @@ public void YamlSerializationTest_ManagedReference(string path)
1919

2020
// Act/Assert
2121
ValidateYamlRoundTrip(model);
22-
ValidateYamlJsonRoundTrip(model);
22+
ValidateYamlJsonRoundTrip(model);
2323
}
2424
}

0 commit comments

Comments
 (0)