From: Layomi Akinrinade Date: Wed, 13 Nov 2019 05:25:46 +0000 (-0800) Subject: Fix exception tests where we are asserting against the literal message (dotnet/corefx... X-Git-Tag: submit/tizen/20210909.063632~11031^2~19 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=abcbe94bbb73cf4ab4173bd16e1fabdde3468769;p=platform%2Fupstream%2Fdotnet%2Fruntime.git Fix exception tests where we are asserting against the literal message (dotnet/corefx#42569) * Fix up exception tests where we are asserting against the literal message * Remove null assignments Commit migrated from https://github.com/dotnet/corefx/commit/dd1892dc53e28b481bfab364a08573214d3df462 --- diff --git a/src/libraries/System.Text.Json/tests/NewtonsoftTests/CustomObjectConverterTests.cs b/src/libraries/System.Text.Json/tests/NewtonsoftTests/CustomObjectConverterTests.cs index 2340b45..a34830f 100644 --- a/src/libraries/System.Text.Json/tests/NewtonsoftTests/CustomObjectConverterTests.cs +++ b/src/libraries/System.Text.Json/tests/NewtonsoftTests/CustomObjectConverterTests.cs @@ -165,11 +165,8 @@ namespace System.Text.Json.Tests ""Number"": 123 } }]"; - NotSupportedException nse = Assert.Throws(() => - { - JsonSerializer.Deserialize>(validJson); - }); - Assert.Equal("Deserialization of interface types is not supported. Type 'System.Text.Json.Tests.IThing'", nse.Message); + + Assert.Throws(() => JsonSerializer.Deserialize>(validJson)); const string invalidJson = @"{ diff --git a/src/libraries/System.Text.Json/tests/NewtonsoftTests/EnumConverterTests.cs b/src/libraries/System.Text.Json/tests/NewtonsoftTests/EnumConverterTests.cs index 490de14..6594447 100644 --- a/src/libraries/System.Text.Json/tests/NewtonsoftTests/EnumConverterTests.cs +++ b/src/libraries/System.Text.Json/tests/NewtonsoftTests/EnumConverterTests.cs @@ -186,17 +186,13 @@ namespace System.Text.Json.Tests [Fact] public static void DuplicateNameEnumTest() { - JsonException e = Assert.Throws(() => - JsonSerializer.Deserialize("\"foo_bar\"")); + Assert.Throws(() => JsonSerializer.Deserialize("\"foo_bar\"")); } [Fact] public static void InvalidValueStringNumber() { - JsonException ex = Assert.Throws(() => - { - StoreColor s = JsonSerializer.Deserialize("\"1\""); - }); + Assert.Throws(() => JsonSerializer.Deserialize("\"1\"")); } [Fact] @@ -236,10 +232,7 @@ namespace System.Text.Json.Tests [Fact] public static void InvalidValueDash() { - JsonException ex = Assert.Throws(() => - { - JsonSerializer.Deserialize("\"-\""); - }); + Assert.Throws(() => JsonSerializer.Deserialize("\"-\"")); } [Fact] diff --git a/src/libraries/System.Text.Json/tests/NewtonsoftTests/ImmutableCollectionsTests.cs b/src/libraries/System.Text.Json/tests/NewtonsoftTests/ImmutableCollectionsTests.cs index 10e588a..4b8829e 100644 --- a/src/libraries/System.Text.Json/tests/NewtonsoftTests/ImmutableCollectionsTests.cs +++ b/src/libraries/System.Text.Json/tests/NewtonsoftTests/ImmutableCollectionsTests.cs @@ -128,9 +128,8 @@ namespace System.Text.Json.Tests [Fact] public void SerializeDefaultArray() { - InvalidOperationException e = Assert.Throws( - () => JsonSerializer.Serialize(default(ImmutableArray), s_indentedOption)); - Assert.Equal("This operation cannot be performed on a default instance of ImmutableArray. Consider initializing the array, or checking the ImmutableArray.IsDefault property.", e.Message); + // The call to .GetEnumerator() throws this exception. Json.NET fails in the same way. + Assert.Throws(() => JsonSerializer.Serialize(default(ImmutableArray), s_indentedOption)); } #endregion diff --git a/src/libraries/System.Text.Json/tests/NewtonsoftTests/JsonSerializerTests.cs b/src/libraries/System.Text.Json/tests/NewtonsoftTests/JsonSerializerTests.cs index 77d8a22..2826e8b 100644 --- a/src/libraries/System.Text.Json/tests/NewtonsoftTests/JsonSerializerTests.cs +++ b/src/libraries/System.Text.Json/tests/NewtonsoftTests/JsonSerializerTests.cs @@ -69,22 +69,22 @@ namespace System.Text.Json.Tests public void IncompleteContainers() { JsonException e = Assert.Throws(() => JsonSerializer.Deserialize>("[1,")); - Assert.Equal("Expected start of a property name or value, but instead reached end of data. Path: $[1] | LineNumber: 0 | BytePositionInLine: 2.", e.Message); + Assert.Contains("Path: $[1] | LineNumber: 0 | BytePositionInLine: 2.", e.Message); e = Assert.Throws(() => JsonSerializer.Deserialize>("[1,")); - Assert.Equal("Expected start of a property name or value, but instead reached end of data. Path: $[1] | LineNumber: 0 | BytePositionInLine: 2.", e.Message); + Assert.Contains("Path: $[1] | LineNumber: 0 | BytePositionInLine: 2.", e.Message); e = Assert.Throws(() => JsonSerializer.Deserialize>("[1")); - Assert.Equal("'1' is an invalid end of a number. Expected a delimiter. Path: $[0] | LineNumber: 0 | BytePositionInLine: 2.", e.Message); + Assert.Contains("Path: $[0] | LineNumber: 0 | BytePositionInLine: 2.", e.Message); e = Assert.Throws(() => JsonSerializer.Deserialize>("{\"key\":1,")); - Assert.Equal("Expected start of a property name or value, but instead reached end of data. Path: $.key | LineNumber: 0 | BytePositionInLine: 8.", e.Message); + Assert.Contains("Path: $.key | LineNumber: 0 | BytePositionInLine: 8.", e.Message); e = Assert.Throws(() => JsonSerializer.Deserialize>("{\"key\":1")); - Assert.Equal("'1' is an invalid end of a number. Expected a delimiter. Path: $.key | LineNumber: 0 | BytePositionInLine: 8.", e.Message); + Assert.Contains("Path: $.key | LineNumber: 0 | BytePositionInLine: 8.", e.Message); e = Assert.Throws(() => JsonSerializer.Deserialize("{\"key\":1,")); - Assert.Equal("Expected start of a property name or value, but instead reached end of data. Path: $ | LineNumber: 0 | BytePositionInLine: 8.", e.Message); + Assert.Contains("$ | LineNumber: 0 | BytePositionInLine: 8.", e.Message); } [Fact]