[Test] Fix collect command test race condition with status printer (dotnet#5787)

The CollectCommandFunctionalTests substitute a MemoryStream for the
output file stream, so the trace file is never created on disk. The
interactive status-printing path is non-deterministic in these tests
because the MemoryStream-backed session completes near-instantly, and
whether the status line fires before the loop exits depends on thread
scheduling. When it does fire, FileInfo.Length throws because the file
does not exist.

Set IsOutputRedirected=true on MockConsole to disable the status path.

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: mdh1418

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

@@ -385,7 +385,6 @@ internal async Task<int> Collect(CancellationToken ct, CommandLineConfiguration
                                 }
                             }
 
-                            FileInfo fileInfo = new(output.FullName);
                             bool wroteStatus = false;
                             Action printStatus = () => {
                                 if (printStatusOverTime && rewriter.IsRewriteConsoleLineSupported)
@@ -399,8 +398,7 @@ internal async Task<int> Collect(CancellationToken ct, CommandLineConfiguration
                                         // First time writing status, so don't rewrite console yet.
                                         wroteStatus = true;
                                     }
-                                    fileInfo.Refresh();
-                                    ConsoleWriteLine($"[{stopwatch.Elapsed:dd\\:hh\\:mm\\:ss}]\tRecording trace {GetSize(fileInfo.Length)}");
+                                    ConsoleWriteLine($"[{stopwatch.Elapsed:dd\\:hh\\:mm\\:ss}]\tRecording trace {GetSize(eventStream.Length)}");
                                     ConsoleWriteLine("Press <Enter> or <Ctrl+C> to exit...");
                                 }
 

src/tests/dotnet-trace/CollectCommandFunctionalTests.cs

@@ -91,6 +91,12 @@ public async Task CollectCommand_InvalidProcessSpecifierConfigurations(CollectAr
 
         private static async Task<int> RunAsync(CollectArgs config, MockConsole console, bool hasChildProcess = false)
         {
+            // Disable the interactive status-printing path. It is currently non-deterministic in these tests because the
+            // MemoryStream-backed session completes near-instantly, and whether the status line fires before the loop exits
+            // depends on thread scheduling. The MemoryStream substitution also means no trace file exists on disk, so
+            // FileInfo.Length in the status printer would throw.
+            console.IsOutputRedirected = true;
+
             var handler = new CollectCommandHandler(console);
             handler.StartTraceSessionAsync = (client, cfg, ct) => Task.FromResult<CollectCommandHandler.ICollectSession>(new TestCollectSession());
             handler.ResumeRuntimeAsync = (client, ct) => Task.CompletedTask;