Fix TestExited_SynchronizingObject test (dotnet#37639)

Fix race condition.
paul1956 · Jun 9, 2020 · 8d3e718 · 8d3e718
1 parent 3b5a51a
commit 8d3e718
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/libraries/System.Diagnostics.Process/tests/ProcessTests.cs b/src/libraries/System.Diagnostics.Process/tests/ProcessTests.cs
@@ -139,7 +139,7 @@ public void TestEnableRaiseEvents(bool? enable)
         [InlineData(true)]
         public void TestExited_SynchronizingObject(bool invokeRequired)
         {
-            bool exitedInvoked = false;
+            var mres = new ManualResetEventSlim();
             Task beginInvokeTask = null;
 
             Process p = CreateProcessLong();
@@ -154,20 +154,20 @@ public void TestExited_SynchronizingObject(bool invokeRequired)
                 }
             };
             p.EnableRaisingEvents = true;
-            p.Exited += delegate { exitedInvoked = true; };
+            p.Exited += delegate { mres.Set(); };
             StartSleepKillWait(p);
 
+            Assert.True(mres.Wait(WaitInMS));
+
             if (invokeRequired)
             {
                 Assert.NotNull(beginInvokeTask);
-                beginInvokeTask.Wait();
+                Assert.True(beginInvokeTask.Wait(WaitInMS));
             }
             else
             {
                 Assert.Null(beginInvokeTask);
             }
-
-            Assert.True(exitedInvoked);
         }
 
         [Fact]