add test case for per-client logging

Author: mus65

Directory.Packages.props

@@ -10,6 +10,7 @@
     <PackageVersion Include="coverlet.msbuild" Version="6.0.4" />
     <PackageVersion Include="GitHubActionsTestLogger" Version="2.4.1" />
     <PackageVersion Include="Meziantou.Analyzer" Version="2.0.205" />
+    <PackageVersion Include="Microsoft.Extensions.Diagnostics.Testing" Version="8.4.0" />
     <!-- Should stay on LTS .NET releases. -->
     <PackageVersion Include="Microsoft.Extensions.Logging.Abstractions" Version="8.0.3" />
     <PackageVersion Include="Microsoft.Extensions.Logging.Console" Version="9.0.7" />

test/Renci.SshNet.IntegrationTests/LoggingTest.cs

@@ -0,0 +1,65 @@
+#if NET
+using Microsoft.Extensions.Logging;
+using Microsoft.Extensions.Logging.Testing;
+
+namespace Renci.SshNet.IntegrationTests
+{
+    [TestClass]
+    public class LoggingTest : IntegrationTestBase
+    {
+        private readonly SftpClient _sftpClient;
+
+        public LoggingTest()
+        {
+            _sftpClient = new SftpClient(SshServerHostName, SshServerPort, User.UserName, User.Password);
+        }
+
+        [TestMethod]
+        public async Task SessionLoggerFactoryTest()
+        {
+            // remember the original logger factory and restore it later
+            ILoggerFactory originalTestLoggerFactory = SshNetLoggingConfiguration.LoggerFactory;
+
+            try
+            {
+                FakeLogCollector globalLogCollector = new();
+                FakeLogCollector sessionLogCollector = new();
+
+                ILoggerFactory globalLoggerFactory = CreateFakeLoggerFactory(globalLogCollector);
+                SshNetLoggingConfiguration.InitializeLogging(globalLoggerFactory);
+
+                _sftpClient.ConnectionInfo.LoggerFactory = CreateFakeLoggerFactory(sessionLogCollector);
+
+                await _sftpClient.ConnectAsync(CancellationToken.None).ConfigureAwait(false);
+
+                // all logs should have been written to the session logger only
+                Assert.AreEqual(0, globalLogCollector.Count);
+                Assert.IsTrue(sessionLogCollector.Count > 0);
+            }
+            finally
+            {
+                SshNetLoggingConfiguration.InitializeLogging(originalTestLoggerFactory);
+            }
+        }
+
+
+        private ILoggerFactory CreateFakeLoggerFactory(FakeLogCollector fakeLogCollector)
+        {
+            FakeLoggerProvider fakeLogProvider = new(fakeLogCollector);
+
+            return LoggerFactory.Create(builder =>
+            {
+                builder.AddProvider(fakeLogProvider);
+                builder.SetMinimumLevel(LogLevel.Trace);
+            });
+        }
+
+        [TestCleanup]
+        public void Cleanup()
+        {
+            _sftpClient.Disconnect();
+            _sftpClient.Dispose();
+        }
+    }
+}
+#endif

test/Renci.SshNet.IntegrationTests/Renci.SshNet.IntegrationTests.csproj

@@ -30,6 +30,10 @@
     </PackageReference>
   </ItemGroup>
 
+  <ItemGroup Condition="$([MSBuild]::IsTargetFrameworkCompatible('$(TargetFramework)', 'net8.0'))">
+    <PackageReference Include="Microsoft.Extensions.Diagnostics.Testing" />
+  </ItemGroup>
+
   <ItemGroup>
     <ProjectReference Include="..\Renci.SshNet.TestTools.OpenSSH\Renci.SshNet.TestTools.OpenSSH.csproj" />
     <ProjectReference Include="..\..\src\Renci.SshNet\Renci.SshNet.csproj" />