We often have this pattern: https://github.com/microsoft/testfx/blob/4dc9eebd01e06e50b6a5237890fd6dde16cc8a80/src/Platform/Microsoft.Testing.Platform/OutputDevice/TerminalOutputDevice.cs#L328-L331 where would return immediately on cancellation. This pattern doesn't look right. Instead, we should call `ThrowIfCancellationRequested()`