Avoid exceptions when console is not available (#644)

* Do not manipulate cursor if Console not available

* Use console.GetTerminal() to test for console

Author: pharring

src/Tools/dotnet-trace/CommandLine/Commands/CollectCommand.cs

@@ -38,7 +38,12 @@ private static async Task<int> Collect(CancellationToken ct, IConsole console, i
             {
                 Debug.Assert(output != null);
                 Debug.Assert(profile != null);
-                Console.Clear();
+
+                bool hasConsole = console.GetTerminal() != null;
+
+                if (hasConsole)
+                    Console.Clear();
+
                 if (processId < 0)
                 {
                     Console.Error.WriteLine("Process ID should not be negative.");
@@ -141,8 +146,13 @@ private static async Task<int> Collect(CancellationToken ct, IConsole console, i
                                     if (nBytesRead <= 0)
                                         break;
                                     fs.Write(buffer, 0, nBytesRead);
-                                    lineToClear = Console.CursorTop-1;
-                                    ResetCurrentConsoleLine(vTermMode.IsEnabled);
+
+                                    if (hasConsole)
+                                    {
+                                        lineToClear = Console.CursorTop - 1;
+                                        ResetCurrentConsoleLine(vTermMode.IsEnabled);
+                                    }
+
                                     Console.Out.WriteLine($"[{stopwatch.Elapsed.ToString(@"dd\:hh\:mm\:ss")}]\tRecording trace {GetSize(fs.Length)}");
                                     Console.Out.WriteLine("Press <Enter> or <Ctrl+C> to exit...");
                                     Debug.WriteLine($"PACKET: {Convert.ToBase64String(buffer, 0, nBytesRead)} (bytes {nBytesRead})");