Wait for serverTask before asserting IsCompletedSuccessfully (#79071)

jozkee · adamsitnik · web-flow · commit 8e07dd415ec6 · 2022-12-02T11:17:44.000-06:00
* Wait for serverTask before asserting IsCompletedSuccessfully

* Apply suggestions from code review

Co-authored-by: Adam Sitnik &lt;adam.sitnik@gmail.com&gt;
diff --git a/src/libraries/System.IO.Pipes/tests/NamedPipeTests/NamedPipeTest.CurrentUserOnly.Windows.cs b/src/libraries/System.IO.Pipes/tests/NamedPipeTests/NamedPipeTest.CurrentUserOnly.Windows.cs
@@ -197,6 +197,7 @@ public void Connection_UnderDifferentUsers_BehavesAsExpected(
                 {
                     // When CurrentUserOnly is only on client side and asks for ReadOnly access, the connection is not rejected
                     // but we get the UnauthorizedAccessException on the client regardless.
+                    Assert.True(serverTask.Wait(TimeSpan.FromSeconds(10)));
                     Assert.True(serverTask.IsCompletedSuccessfully);
                 }
                 else

Original file line number	Diff line number	Diff line change
`@@ -197,6 +197,7 @@ public void Connection_UnderDifferentUsers_BehavesAsExpected(`
`197`	`197`	`{`
`198`	`198`	`// When CurrentUserOnly is only on client side and asks for ReadOnly access, the connection is not rejected`
`199`	`199`	`// but we get the UnauthorizedAccessException on the client regardless.`
	`200`	`+ Assert.True(serverTask.Wait(TimeSpan.FromSeconds(10)));`
`200`	`201`	`Assert.True(serverTask.IsCompletedSuccessfully);`
`201`	`202`	`}`
`202`	`203`	`else`