Merge pull request #1544 from microsoft/dev/andarno/testStabilization

Fix flaky GC-based tests in AsyncLazyTests

Author: AArnott

test/Microsoft.VisualStudio.Threading.Tests/AsyncLazyTests.cs

@@ -153,10 +153,11 @@ public async Task ValueFactoryReleasedAfterExecution()
     {
         WeakReference collectible = await this.ValueFactoryReleasedAfterExecution_Helper();
 
+        await Task.Yield();
         for (int i = 0; i < 3; i++)
         {
-            await Task.Yield();
-            GC.Collect();
+            GC.Collect(GC.MaxGeneration, GCCollectionMode.Forced, blocking: true);
+            GC.WaitForPendingFinalizers();
         }
 
         Assert.False(collectible.IsAlive);
@@ -167,10 +168,11 @@ public async Task AsyncPumpReleasedAfterExecution(bool throwInValueFactory)
     {
         WeakReference collectible = await this.AsyncPumpReleasedAfterExecution_Helper(throwInValueFactory);
 
+        await Task.Yield();
         for (int i = 0; i < 3; i++)
         {
-            await Task.Yield();
-            GC.Collect();
+            GC.Collect(GC.MaxGeneration, GCCollectionMode.Forced, blocking: true);
+            GC.WaitForPendingFinalizers();
         }
 
         Assert.False(collectible.IsAlive);