refactor TestHostFixture.cs +semver:skip

Author: Stepami

tests/HydraScript.IntegrationTests/ImplicitTestOutputHelperAccessor.cs

@@ -0,0 +1,12 @@
+using MartinCostello.Logging.XUnit;
+
+namespace HydraScript.IntegrationTests;
+
+public class ImplicitTestOutputHelperAccessor : ITestOutputHelperAccessor
+{
+    public ITestOutputHelper? OutputHelper
+    {
+        get => TestContext.Current.TestOutputHelper;
+        set { }
+    }
+}

tests/HydraScript.IntegrationTests/TestHostFixture.cs

@@ -1,14 +1,14 @@
 using System.IO.Abstractions;
 using HydraScript.Infrastructure;
-using MartinCostello.Logging.XUnit;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Logging;
 using NSubstitute;
 
 namespace HydraScript.IntegrationTests;
 
-public class TestHostFixture : IDisposable, ITestOutputHelperAccessor
+public class TestHostFixture : IDisposable
 {
+    public const string ScriptFileName = "file";
     public record Options(
         string FileName = ScriptFileName + ".js",
         bool Dump = false,
@@ -27,17 +27,8 @@ public void Dispose()
     }
 
     private readonly List<string> _logMessages = [];
-
-    public const string ScriptFileName = "file";
-
     public IReadOnlyCollection<string> LogMessages => _logMessages;
 
-    public ITestOutputHelper? OutputHelper
-    {
-        get => TestContext.Current.TestOutputHelper;
-        set { }
-    }
-
     public Runner GetRunner(Options options, Action<IServiceCollection>? configureTestServices = null)
     {
         var serviceProvider = Program.GetServiceProvider(
@@ -46,7 +37,7 @@ public Runner GetRunner(Options options, Action<IServiceCollection>? configureTe
             services =>
             {
                 services.AddLogging(x => x.ClearProviders()
-                    .AddXUnit(this)
+                    .AddXUnit(new ImplicitTestOutputHelperAccessor())
                     .AddFakeLogging(fakeLogOptions =>
                     {
                         fakeLogOptions.OutputSink = logMessage => _logMessages.Add(logMessage);