From ec20000df1a5d6242274b8d03739f8357100f9a3 Mon Sep 17 00:00:00 2001 From: Tomas Date: Thu, 22 Apr 2021 17:58:35 +0200 Subject: [PATCH] Fix test 49826 to work after the refactoring in #50074 The change #50074 made rooting of built-in serializers lazier. By analyzing the library code I have figured out that I can root the default serializers by calling Serialize. This fixes the test failure. Thanks Tomas --- src/tests/Regressions/coreclr/GitHub_49826/test49826.cs | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/tests/Regressions/coreclr/GitHub_49826/test49826.cs b/src/tests/Regressions/coreclr/GitHub_49826/test49826.cs index f89706591b5132..ee210fca1c641b 100644 --- a/src/tests/Regressions/coreclr/GitHub_49826/test49826.cs +++ b/src/tests/Regressions/coreclr/GitHub_49826/test49826.cs @@ -11,6 +11,10 @@ class Program static int Main() { JsonSerializerOptions options = new JsonSerializerOptions(); + + // This is needed to internally call RootBuiltInConvertersAndTypeInfoCreator + JsonSerializer.Serialize("a-string"); + JsonConverter converter = options.GetConverter(typeof(DateTime)); Console.WriteLine("Converter type: {0}", converter.GetType()); return converter != null ? 100 : 1;