Add null handling for process output/error data in UiTestHelpers (#3184)

* null handling for process output/error data * apply suggestions!
AzureAD · Jan 7, 2025 · 1710b64 · 1710b64
1 parent d83d251
commit 1710b64
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/tests/E2E Tests/WebAppUiTests/UiTestHelpers.cs b/tests/E2E Tests/WebAppUiTests/UiTestHelpers.cs
@@ -182,8 +182,17 @@ public static Process StartProcessLocally(
             else
             {
                 // Log the output and error streams
-                process.OutputDataReceived += (sender, e) => output.WriteLine(e.Data);
-                process.ErrorDataReceived += (sender, e) => output.WriteLine(e.Data);
+                process.OutputDataReceived += (sender, e) =>
+                {
+                    output.WriteLine($"{process.Id} ");
+                    output.WriteLine(e?.Data ?? "null output data received.");
+                };
+
+                process.ErrorDataReceived += (sender, e) =>
+                {
+                    output.WriteLine($"{process.Id} ");
+                    output.WriteLine(e?.Data ?? "null error data received.");
+                };
 
                 process.BeginOutputReadLine();
                 process.BeginErrorReadLine();