Removes some tests that are now bogus

Last week I changed the implementation of encoders such that they now don't guarantee that their Default property always returns the same instance of the default encoder. Access to the property’s backing field used to be synchronized using memory berries; I now removed these synchronizations. This change was made to simplify code and improve performance of the common scenario: retrieving the default encoder, at the cost of having to create (and collect) more than one instance of the encoder in rare cases (once in a blue moon).
This change just remove tests that test behavior that we don’t guarantee anymore.


Commit migrated from dotnet/corefx@0bfa870

Author: KrzysztofCwalina

src/libraries/System.Text.Encodings.Web/tests/HtmlEncoderTests.cs

@@ -71,17 +71,6 @@ public void Default_EquivalentToBasicLatin()
             }
         }
 
-        [Fact]
-        public void Default_ReturnsSingletonInstance()
-        {
-            // Act
-            HtmlEncoder encoder1 = HtmlEncoder.Default;
-            HtmlEncoder encoder2 = HtmlEncoder.Default;
-
-            // Assert
-            Assert.Same(encoder1, encoder2);
-        }
-
         [Theory]
         [InlineData("<", "&lt;")]
         [InlineData(">", "&gt;")]

src/libraries/System.Text.Encodings.Web/tests/JavaScriptStringEncoderTests.cs

@@ -71,17 +71,6 @@ public void Default_EquivalentToBasicLatin()
             }
         }
 
-        [Fact]
-        public void Default_ReturnsSingletonInstance()
-        {
-            // Act
-            JavaScriptStringEncoder encoder1 = JavaScriptStringEncoder.Default;
-            JavaScriptStringEncoder encoder2 = JavaScriptStringEncoder.Default;
-
-            // Assert
-            Assert.Same(encoder1, encoder2);
-        }
-
         [Fact]
         public void JavaScriptStringEncode_AllRangesAllowed_StillEncodesForbiddenChars_Simple_Escaping() {
             // The following two calls could be simply InlineData to the Theory below

src/libraries/System.Text.Encodings.Web/tests/UrlEncoderTests.cs

@@ -75,17 +75,6 @@ public void Default_EquivalentToBasicLatin()
             }
         }
 
-        [Fact]
-        public void Default_ReturnsSingletonInstance()
-        {
-            // Act
-            UrlEncoder encoder1 = UrlEncoder.Default;
-            UrlEncoder encoder2 = UrlEncoder.Default;
-
-            // Assert
-            Assert.Same(encoder1, encoder2);
-        }
-
         [Fact]
         public void UrlEncode_AllRangesAllowed_StillEncodesForbiddenChars()
         {