Should the pipe connection loop be `await`ed?

[Youssef1313]

should we await it? the VS client uses the following vs-threading API https://github.com/microsoft/vs-threading/blob/849676611f654d73c84a9468b0ecc205294250e0/src/Microsoft.VisualStudio.Threading/ThreadingTools.cs#L105

also, when would we expect a timeout? if test process exits, then this method should hopefully completely pretty much immediately

Originally posted by @drognanar in #50727 (comment)

[Youssef1313]

@mariam-abdulla Can you answer this please?