refactor(#603): Add default console logger

build.cake

@@ -82,7 +82,7 @@ Task("Tests")
       Verbosity = param.Verbosity,
       NoRestore = true,
       NoBuild = true,
-      Loggers = new[] { "trx" },
+      Loggers = new[] { "console;verbosity=normal", "trx" },
       Filter = param.TestFilter,
       ResultsDirectory = param.Paths.Directories.TestResultsDirectoryPath,
       ArgumentCustomization = args => args

src/Testcontainers/Configurations/TestcontainersSettings.cs

@@ -10,7 +10,6 @@ namespace DotNet.Testcontainers.Configurations
   using DotNet.Testcontainers.Images;
   using JetBrains.Annotations;
   using Microsoft.Extensions.Logging;
-  using Microsoft.Extensions.Logging.Abstractions;
 
   /// <summary>
   /// This class represents the Testcontainers settings.
@@ -83,7 +82,7 @@ static TestcontainersSettings()
     [PublicAPI]
     [NotNull]
     public static ILogger Logger { get; set; }
-      = NullLogger.Instance;
+      = new Logger();
 
     /// <summary>
     /// Gets or sets the host operating system.

src/Testcontainers/Logger.cs

@@ -0,0 +1,88 @@
+namespace DotNet.Testcontainers
+{
+  using System;
+  using System.Diagnostics;
+  using System.Globalization;
+  using System.IO;
+  using Microsoft.Extensions.Logging;
+
+  /// <summary>
+  /// An <see cref="ILogger" /> implementation that forwards messages to the console.
+  /// Not every test framework or environment supports this approach.
+  /// Developers may still need to configure their own logging implementation.
+  /// If VSTest.Console.exe loads the test adapter in a deterministic order,
+  /// we can write our own test adapter and intercept the IMessageLogger instance:
+  /// https://github.com/microsoft/vstest/issues/4125#issuecomment-1320880502.
+  /// To debug the test host and runner set the environment variables VSTEST_HOST_DEBUG and VSTEST_RUNNER_DEBUG to 1.
+  /// To enable VSTest logging set VSTEST_DIAG to 1 and VSTEST_DIAG_VERBOSITY to verbose.
+  /// The following example contains the ITestExecutor implementations. It is important that the assembly ends with TestAdapter.dll.
+  /// <code>
+  ///    [FileExtension(DllFileExtension)]
+  ///    [FileExtension(ExeFileExtension)]
+  ///    [DefaultExecutorUri(ExecutorUri)]
+  ///    [ExtensionUri(ExecutorUri)]
+  ///    [Category(Category)]
+  ///    internal sealed class UssDiscovery : ITestDiscoverer, ITestExecutor
+  ///    {
+  ///      private const string DllFileExtension = &quot;.dll&quot;;
+  ///
+  ///      private const string ExeFileExtension = &quot;.exe&quot;;
+  ///
+  ///      private const string ExecutorUri = &quot;executor://testcontainers.org/v1&quot;;
+  ///
+  ///      private const string Category = &quot;managed&quot;;
+  ///
+  ///      public void DiscoverTests(IEnumerable&lt;string&gt; sources, IDiscoveryContext discoveryContext, IMessageLogger logger, ITestCaseDiscoverySink discoverySink)
+  ///      {
+  ///      }
+  ///
+  ///      public void RunTests(IEnumerable&lt;TestCase&gt; tests, IRunContext runContext, IFrameworkHandle frameworkHandle)
+  ///      {
+  ///        SetLogger(frameworkHandle);
+  ///      }
+  ///
+  ///      public void RunTests(IEnumerable&lt;string&gt; sources, IRunContext runContext, IFrameworkHandle frameworkHandle)
+  ///      {
+  ///        SetLogger(frameworkHandle);
+  ///      }
+  ///
+  ///      public void Cancel()
+  ///      {
+  ///      }
+  ///
+  ///      private static void SetLogger(IMessageLogger logger)
+  ///      {
+  ///        // Set the TestcontainersSettings.Logger. Use a semaphore to block the test execution until the logger is set.
+  ///      }
+  ///    }
+  /// </code>
+  /// </summary>
+  internal sealed class Logger : ILogger, IDisposable
+  {
+    private readonly TextWriter stderr = Console.Error;
+
+    private readonly TextWriter stdout = Console.Out;
+
+    private readonly Stopwatch stopwatch = Stopwatch.StartNew();
+
+    public void Dispose()
+    {
+    }
+
+    public void Log<TState>(LogLevel logLevel, EventId eventId, TState state, Exception exception, Func<TState, Exception, string> formatter)
+    {
+      var message = string.Format(CultureInfo.CurrentCulture, "[testcontainers.org {0:hh\\:mm\\:ss\\.ff}] {1}", this.stopwatch.Elapsed, formatter.Invoke(state, exception));
+      this.stdout.WriteLine(message);
+    }
+
+    public bool IsEnabled(LogLevel logLevel)
+    {
+      return true;
+    }
+
+    public IDisposable BeginScope<TState>(TState state)
+    {
+      return this;
+    }
+  }
+}

src/Testcontainers/Networks/NonExistingDockerNetwork.cs

@@ -1,8 +1,6 @@
 namespace DotNet.Testcontainers.Networks
 {
   using System;
-  using System.Collections.Generic;
-  using System.Collections.ObjectModel;
   using System.Threading;
   using System.Threading.Tasks;
   using Docker.DotNet.Models;

tests/Testcontainers.Tests/SkipOnLinuxEngineAttribute.cs

@@ -2,12 +2,18 @@ namespace DotNet.Testcontainers.Tests
 {
   using System;
   using System.Diagnostics;
+  using DotNet.Testcontainers.Configurations;
   using Xunit;
 
   public sealed class SkipOnLinuxEngineAttribute : FactAttribute
   {
     private static readonly bool IsLinuxEngineEnabled = GetIsLinuxEngineEnabled();
 
+    static SkipOnLinuxEngineAttribute()
+    {
+      TestcontainersSettings.ResourceReaperEnabled = IsLinuxEngineEnabled;
+    }
+
     public SkipOnLinuxEngineAttribute()
     {
       if (IsLinuxEngineEnabled)

tests/Testcontainers.Tests/Unit/Configurations/TestcontainersAccessInformationTest.cs

@@ -10,7 +10,6 @@ namespace DotNet.Testcontainers.Tests.Unit
 
   public static class TestcontainersAccessInformationTest
   {
-    [Collection(nameof(Testcontainers))]
     public sealed class AccessDockerInformation
     {
       private const string DoesNotExist = nameof(TestcontainersAccessInformationTest);

tests/Testcontainers.Tests/Unit/Containers/Unix/Modules/Databases/AzuriteTestcontainerTest.cs

@@ -91,7 +91,6 @@ private static async Task EstablishConnection(AzuriteFixture.AzuriteDefaultFixtu
         Assert.Contains(TableServiceDataFileName, execResult.Stdout);
       }
 
-      [Collection(nameof(Testcontainers))]
       public sealed class CommonContainerPorts : IClassFixture<AzuriteFixture.AzuriteDefaultFixture>
       {
         private readonly AzuriteFixture.AzuriteDefaultFixture commonContainerPorts;
@@ -109,7 +108,6 @@ public async Task ConnectionEstablished()
         }
       }
 
-      [Collection(nameof(Testcontainers))]
       public sealed class CustomContainerPorts : IClassFixture<AzuriteFixture.AzuriteWithCustomContainerPortsFixture>
       {
         private readonly AzuriteFixture.AzuriteDefaultFixture customContainerPorts;
@@ -128,7 +126,6 @@ public async Task ConnectionEstablished()
       }
     }
 
-    [Collection(nameof(Testcontainers))]
     public sealed class BlobServiceEnabled : IClassFixture<AzuriteFixture.AzuriteWithBlobOnlyFixture>
     {
       private readonly AzuriteFixture.AzuriteDefaultFixture azurite;
@@ -171,7 +168,6 @@ await Assert.ThrowsAsync<RequestFailedException>(() => tableServiceClient.GetPro
       }
     }
 
-    [Collection(nameof(Testcontainers))]
     public sealed class QueueServiceEnabled : IClassFixture<AzuriteFixture.AzuriteWithQueueOnlyFixture>
     {
       private readonly AzuriteFixture.AzuriteDefaultFixture azurite;
@@ -214,7 +210,6 @@ await Assert.ThrowsAsync<RequestFailedException>(() => tableServiceClient.GetPro
       }
     }
 
-    [Collection(nameof(Testcontainers))]
     public sealed class TableServiceEnabled : IClassFixture<AzuriteFixture.AzuriteWithTableOnlyFixture>
     {
       private readonly AzuriteFixture.AzuriteDefaultFixture azurite;
@@ -257,7 +252,6 @@ await Assert.ThrowsAsync<RequestFailedException>(() => queueServiceClient.GetPro
       }
     }
 
-    [Collection(nameof(Testcontainers))]
     public sealed class CustomLocation : IClassFixture<AzuriteFixture.AzuriteWithCustomWorkspaceFixture>
     {
       private readonly IEnumerable<string> dataFiles;

tests/Testcontainers.Tests/Unit/Containers/Unix/Modules/Databases/CosmosDbTestcontainerTest.cs

@@ -11,7 +11,6 @@ namespace DotNet.Testcontainers.Tests.Unit
 
   public static class CosmosDbTestcontainerTest
   {
-    [Collection(nameof(Testcontainers))]
     public sealed class ConnectionTests : IClassFixture<CosmosDbFixture>, IDisposable
     {
       private const string SkipReason = "The Cosmos DB Linux Emulator Docker image does not run on every CI environment."; // https://github.com/Azure/azure-cosmos-db-emulator-docker/issues/45.

tests/Testcontainers.Tests/Unit/Containers/Unix/Modules/Databases/CouchDbTestcontainerTest.cs

@@ -7,7 +7,6 @@ namespace DotNet.Testcontainers.Tests.Unit
   using DotNet.Testcontainers.Tests.Fixtures;
   using Xunit;
 
-  [Collection(nameof(Testcontainers))]
   public sealed class CouchDbTestcontainerTest : IClassFixture<CouchDbFixture>
   {
     private readonly CouchDbFixture couchDbFixture;

tests/Testcontainers.Tests/Unit/Containers/Unix/Modules/Databases/CouchbaseTestcontainerTest.cs

@@ -7,7 +7,6 @@ namespace DotNet.Testcontainers.Tests.Unit
   using DotNet.Testcontainers.Tests.Fixtures;
   using Xunit;
 
-  [Collection(nameof(Testcontainers))]
   public sealed class CouchbaseTestcontainerTest : IClassFixture<CouchbaseFixture>
   {
     private readonly CouchbaseFixture couchbaseFixture;

tests/Testcontainers.Tests/Unit/Containers/Unix/Modules/Databases/ElasticsearchTestcontainerTest.cs

@@ -6,7 +6,6 @@ namespace DotNet.Testcontainers.Tests.Unit
   using DotNet.Testcontainers.Tests.Fixtures;
   using Xunit;
 
-  [Collection(nameof(Testcontainers))]
   public sealed class ElasticsearchTestcontainerTest : IClassFixture<ElasticsearchFixture>
   {
     private readonly ElasticsearchFixture elasticsearchFixture;

tests/Testcontainers.Tests/Unit/Containers/Unix/Modules/Databases/MariaDbTestcontainerTest.cs

@@ -5,7 +5,6 @@ namespace DotNet.Testcontainers.Tests.Unit
   using DotNet.Testcontainers.Tests.Fixtures;
   using Xunit;
 
-  [Collection(nameof(Testcontainers))]
   public sealed class MariaDbTestcontainerTest : IClassFixture<MariaDbFixture>
   {
     private readonly MariaDbFixture mariaDbFixture;

tests/Testcontainers.Tests/Unit/Containers/Unix/Modules/Databases/MongoDbTestcontainerTest.cs

@@ -6,7 +6,6 @@
   using MongoDB.Bson;
   using Xunit;
 
-  [Collection(nameof(Testcontainers))]
   public sealed class MongoDbTestcontainerTest : IClassFixture<MongoDbFixture>, IClassFixture<MongoDbNoAuthFixture>
   {
     private const string MongoDbScheme = "mongodb";

tests/Testcontainers.Tests/Unit/Containers/Unix/Modules/Databases/MsSqlTestcontainerTest.cs

@@ -7,7 +7,6 @@ namespace DotNet.Testcontainers.Tests.Unit
   using DotNet.Testcontainers.Tests.Fixtures;
   using Xunit;
 
-  [Collection(nameof(Testcontainers))]
   public sealed class MsSqlTestcontainerTest : IClassFixture<MsSqlFixture>
   {
     private readonly MsSqlFixture msSqlFixture;

tests/Testcontainers.Tests/Unit/Containers/Unix/Modules/Databases/MySqlTestcontainerTest.cs

@@ -7,7 +7,6 @@ namespace DotNet.Testcontainers.Tests.Unit
 
   public static class MySqlTestcontainerTest
   {
-    [Collection(nameof(Testcontainers))]
     public sealed class MySqlCustomUsernameTest : IClassFixture<MySqlFixture>
     {
       private readonly MySqlFixture mySqlFixture;
@@ -69,7 +68,6 @@ public async Task ThrowErrorInRunningContainerWithInvalidScript()
       }
     }
 
-    [Collection(nameof(Testcontainers))]
     public sealed class MySqlRootUsernameTest : IClassFixture<MySqlRootUsernameFixture>
     {
       private readonly MySqlFixture mySqlFixture;

tests/Testcontainers.Tests/Unit/Containers/Unix/Modules/Databases/Neo4jTestcontainerTest.cs

@@ -6,7 +6,6 @@
   using DotNet.Testcontainers.Tests.Fixtures;
   using Xunit;
 
-  [Collection(nameof(Testcontainers))]
   public sealed class Neo4jTestcontainerTest : IClassFixture<Neo4jFixture>
   {
     private readonly Neo4jFixture neo4jFixture;

tests/Testcontainers.Tests/Unit/Containers/Unix/Modules/Databases/OracleTestcontainerTest.cs

@@ -7,7 +7,6 @@ namespace DotNet.Testcontainers.Tests.Unit
   using DotNet.Testcontainers.Tests.Fixtures;
   using Xunit;
 
-  [Collection(nameof(Testcontainers))]
   public sealed class OracleTestcontainerTest : IClassFixture<OracleFixture>
   {
     private readonly OracleFixture oracleFixture;

tests/Testcontainers.Tests/Unit/Containers/Unix/Modules/Databases/PostgreSqlTestcontainerTest.cs

@@ -5,7 +5,6 @@ namespace DotNet.Testcontainers.Tests.Unit
   using DotNet.Testcontainers.Tests.Fixtures;
   using Xunit;
 
-  [Collection(nameof(Testcontainers))]
   public sealed class PostgreSqlTestcontainerTest : IClassFixture<PostgreSqlFixture>
   {
     private readonly PostgreSqlFixture postgreSqlFixture;

tests/Testcontainers.Tests/Unit/Containers/Unix/Modules/Databases/RedisTestcontainerTest.cs

@@ -6,7 +6,6 @@ namespace DotNet.Testcontainers.Tests.Unit
   using DotNet.Testcontainers.Tests.Fixtures;
   using Xunit;
 
-  [Collection(nameof(Testcontainers))]
   public sealed class RedisTestcontainerTest : IClassFixture<RedisFixture>
   {
     private readonly RedisFixture redisFixture;

tests/Testcontainers.Tests/Unit/Containers/Unix/Modules/MessageBrokers/KafkaTestcontainerTest.cs

@@ -8,7 +8,6 @@ namespace DotNet.Testcontainers.Tests.Unit
   using DotNet.Testcontainers.Tests.Fixtures;
   using Xunit;
 
-  [Collection(nameof(Testcontainers))]
   public sealed class KafkaTestcontainerTest : IClassFixture<KafkaFixture>
   {
     private readonly KafkaFixture kafkaFixture;