From 702f846f33e9d0b0338a97e7f47939b53c6dd911 Mon Sep 17 00:00:00 2001 From: Michael Render Date: Fri, 8 Nov 2024 15:04:00 -0500 Subject: [PATCH] [dotnet] Add reflection-based JSON serialization fallback to `Command`s Fixes #14731 --- dotnet/src/webdriver/Command.cs | 3 ++- dotnet/test/common/CommandTests.cs | 22 ++++++++++++++++++++++ 2 files changed, 24 insertions(+), 1 deletion(-) create mode 100644 dotnet/test/common/CommandTests.cs diff --git a/dotnet/src/webdriver/Command.cs b/dotnet/src/webdriver/Command.cs index 38abb85480a3e..9cb96055cc952 100644 --- a/dotnet/src/webdriver/Command.cs +++ b/dotnet/src/webdriver/Command.cs @@ -21,6 +21,7 @@ using System.Collections.Generic; using System.Text.Json; using System.Text.Json.Serialization; +using System.Text.Json.Serialization.Metadata; namespace OpenQA.Selenium { @@ -35,7 +36,7 @@ public class Command private readonly static JsonSerializerOptions s_jsonSerializerOptions = new() { - TypeInfoResolver = CommandJsonSerializerContext.Default, + TypeInfoResolver = JsonTypeInfoResolver.Combine(CommandJsonSerializerContext.Default, new DefaultJsonTypeInfoResolver()), Converters = { new ResponseValueJsonConverter() } }; diff --git a/dotnet/test/common/CommandTests.cs b/dotnet/test/common/CommandTests.cs new file mode 100644 index 0000000000000..5e657ac4170c8 --- /dev/null +++ b/dotnet/test/common/CommandTests.cs @@ -0,0 +1,22 @@ +using NUnit.Framework; +using System.Collections.Generic; + +namespace OpenQA.Selenium +{ + [TestFixture] + public class CommandTests + { + [Test] + public void CommandSerializesAnonymousType() + { + var parameters = new Dictionary + { + ["arg"] = new { param1 = true, param2 = false }, + }; + + var command = new Command(new SessionId("session"), "test command", parameters); + + Assert.That(command.ParametersAsJsonString, Is.EqualTo("""{"arg":{"param1":true,"param2":false}}""")); + } + } +}