From 055fcba35d3322c4d05425b632cca3d9e4849dd3 Mon Sep 17 00:00:00 2001 From: Jeremy Barton Date: Sat, 13 Jul 2019 14:34:29 -0700 Subject: [PATCH] Remove the WriteNullStringAsProperty test The writer has standardized on not allowing null -> Empty coercion, and this test no longer is needed since JsonElement doesn't have a WriteProperty method, and a JsonProperty can't ever have a null property name. Commit migrated from https://github.com/dotnet/corefx/commit/5294f0da4695162603d24dba351c985181042796 --- .../System.Text.Json/tests/JsonElementWriteTests.cs | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/src/libraries/System.Text.Json/tests/JsonElementWriteTests.cs b/src/libraries/System.Text.Json/tests/JsonElementWriteTests.cs index 204dc6b..c69cf67 100644 --- a/src/libraries/System.Text.Json/tests/JsonElementWriteTests.cs +++ b/src/libraries/System.Text.Json/tests/JsonElementWriteTests.cs @@ -385,21 +385,6 @@ null, [Theory] [InlineData(false)] [InlineData(true)] - public static void WriteNullStringAsProperty(bool indented) - { - WritePropertyValueBothForms( - indented, - null, - "\"\"", - @"{ - """": """" -}", - "{\"\":\"\"}"); - } - - [Theory] - [InlineData(false)] - [InlineData(true)] public static void WriteNumberAsProperty(bool indented) { WritePropertyValueBothForms( -- 2.7.4