Clean up after merge

Author: AArnott

test/StreamJsonRpc.Tests.ExternalAssembly/StreamJsonRpc.Tests.ExternalAssembly.csproj

@@ -2,7 +2,6 @@
 
   <PropertyGroup>
     <TargetFramework>netstandard2.0</TargetFramework>
-    <DisablePolyfills>true</DisablePolyfills>
     <IsTestProject>false</IsTestProject>
   </PropertyGroup>
 

test/StreamJsonRpc.Tests/TestBase.cs

@@ -42,8 +42,6 @@ protected TestBase(ITestOutputHelper logger)
 
     protected static CancellationToken UnexpectedTimeoutToken => new CancellationTokenSource(UnexpectedTimeout).Token;
 
-    protected static bool IsTestRunOnAzurePipelines => !string.IsNullOrEmpty(Environment.GetEnvironmentVariable("SYSTEM_DEFINITIONID"));
-
     protected ITestOutputHelper Logger { get; }
 
     protected CancellationToken TimeoutToken => Debugger.IsAttached ? CancellationToken.None : this.timeoutTokenSource.Token;
@@ -63,12 +61,6 @@ protected static void AssertCollectedObject(WeakReference weakReference)
         GC.Collect();
         GC.WaitForPendingFinalizers();
 
-        // For some reason the assertion tends to be sketchy when running on Azure Pipelines.
-        if (IsTestRunOnAzurePipelines)
-        {
-            Assert.SkipWhen(weakReference.IsAlive, "Weak reference is still alive.");
-        }
-
         Assert.False(weakReference.IsAlive);
     }