[dotnet] Make internal console writer more flexible via taking TextWriter only

dotnet/src/webdriver/Internal/Logging/ConsoleHandler.cs

@@ -0,0 +1,31 @@
+// <copyright file="ConsoleHandler.cs" company="Selenium Committers">
+// Licensed to the Software Freedom Conservancy (SFC) under one
+// or more contributor license agreements.  See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership.  The SFC licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License.  You may obtain a copy of the License at
+//
+//   http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied.  See the License for the
+// specific language governing permissions and limitations
+// under the License.
+// </copyright>
+
+using System;
+
+#nullable enable
+
+namespace OpenQA.Selenium.Internal.Logging
+{
+    [Obsolete("Use TextWriterHandler instead, will be removed in v4.32")]
+    /// <summary>
+    /// Represents a log handler that writes log events to the given text writer.
+    /// </summary>
+    public class ConsoleHandler() : TextWriterHandler(Console.Error);
+}

dotnet/src/webdriver/Internal/Logging/LogContextManager.cs

@@ -17,6 +17,7 @@
 // under the License.
 // </copyright>
 
+using System;
 using System.Diagnostics.CodeAnalysis;
 using System.Threading;
 
@@ -26,13 +27,13 @@ namespace OpenQA.Selenium.Internal.Logging
 {
     internal class LogContextManager
     {
-        private readonly AsyncLocal<ILogContext?> _currentAmbientLogContext = new AsyncLocal<ILogContext?>();
+        private readonly AsyncLocal<ILogContext?> _currentAmbientLogContext = new();
 
         public LogContextManager()
         {
-            var defaulConsoleLogHandler = new ConsoleLogHandler();
+            var defaulConsoleLogHandler = new TextWriterHandler(Console.Error);
 
-            GlobalContext = new LogContext(LogEventLevel.Info, null, null, new[] { defaulConsoleLogHandler });
+            GlobalContext = new LogContext(LogEventLevel.Info, null, null, [defaulConsoleLogHandler]);
         }
 
         public ILogContext GlobalContext { get; }

dotnet/src/webdriver/Internal/Logging/ConsoleLogHandler.cs → dotnet/src/webdriver/Internal/Logging/TextWriterHandler.cs

@@ -1,4 +1,4 @@
-// <copyright file="ConsoleLogHandler.cs" company="Selenium Committers">
+// <copyright file="TextWriterHandler.cs" company="Selenium Committers">
 // Licensed to the Software Freedom Conservancy (SFC) under one
 // or more contributor license agreements.  See the NOTICE file
 // distributed with this work for additional information
@@ -17,27 +17,27 @@
 // under the License.
 // </copyright>
 
-using System;
+using System.IO;
 
 #nullable enable
 
 namespace OpenQA.Selenium.Internal.Logging
 {
     /// <summary>
-    /// Represents a log handler that writes log events to the console.
+    /// Represents a log handler that writes log events to the given text writer.
     /// </summary>
-    public class ConsoleLogHandler : ILogHandler
+    public class TextWriterHandler(TextWriter writer) : ILogHandler
     {
         // performance trick to avoid expensive Enum.ToString() with fixed length
         private static readonly string[] _levels = { "TRACE", "DEBUG", " INFO", " WARN", "ERROR" };
 
         /// <summary>
-        /// Handles a log event by writing it to the console.
+        /// Handles a log event by writing it to the text writer.
         /// </summary>
         /// <param name="logEvent">The log event to handle.</param>
         public void Handle(LogEvent logEvent)
         {
-            Console.Error.WriteLine($"{logEvent.Timestamp:HH:mm:ss.fff} {_levels[(int)logEvent.Level]} {logEvent.IssuedBy.Name}: {logEvent.Message}");
+            writer.WriteLine($"{logEvent.Timestamp:HH:mm:ss.fff} {_levels[(int)logEvent.Level]} {logEvent.IssuedBy.Name}: {logEvent.Message}");
         }
     }
 }

dotnet/test/common/Internal/Logging/LogTest.cs

@@ -31,7 +31,7 @@ public class LogTest
         private void ResetGlobalLog()
         {
             Log.SetLevel(LogEventLevel.Info);
-            Log.Handlers.Clear().Handlers.Add(new ConsoleLogHandler());
+            Log.Handlers.Clear().Handlers.Add(new TextWriterHandler(Console.Error));
         }
 
         [SetUp]