Make the logger configurable with a new WithLogger method on the builders

This will enable a seamless integration with xUnit.net (both ITestOutputHelper and IMessageSink)

Fixes testcontainers#996

Author: 0xced

src/Templates/CSharp/Testcontainers.ModuleName/ModuleNameContainer.cs

@@ -8,9 +8,8 @@ public sealed class ModuleNameContainer : DockerContainer
     /// Initializes a new instance of the <see cref="ModuleNameContainer" /> class.
     /// </summary>
     /// <param name="configuration">The container configuration.</param>
-    /// <param name="logger">The logger.</param>
-    public ModuleNameContainer(ModuleNameConfiguration configuration, ILogger logger)
-        : base(configuration, logger)
+    public ModuleNameContainer(ModuleNameConfiguration configuration)
+        : base(configuration)
     {
     }
 }

src/Testcontainers.ActiveMq/ArtemisBuilder.cs

@@ -62,7 +62,7 @@ public ArtemisBuilder WithPassword(string password)
     public override ArtemisContainer Build()
     {
         Validate();
-        return new ArtemisContainer(DockerResourceConfiguration, TestcontainersSettings.Logger);
+        return new ArtemisContainer(DockerResourceConfiguration);
     }
 
     /// <inheritdoc />

src/Testcontainers.ActiveMq/ArtemisContainer.cs

@@ -10,9 +10,8 @@ public sealed class ArtemisContainer : DockerContainer
     /// Initializes a new instance of the <see cref="ArtemisContainer" /> class.
     /// </summary>
     /// <param name="configuration">The container configuration.</param>
-    /// <param name="logger">The logger.</param>
-    public ArtemisContainer(ActiveMqConfiguration configuration, ILogger logger)
-        : base(configuration, logger)
+    public ArtemisContainer(ActiveMqConfiguration configuration)
+        : base(configuration)
     {
         _configuration = configuration;
     }

src/Testcontainers.ActiveMq/Usings.cs

@@ -4,5 +4,4 @@
 global using DotNet.Testcontainers.Builders;
 global using DotNet.Testcontainers.Configurations;
 global using DotNet.Testcontainers.Containers;
-global using JetBrains.Annotations;
-global using Microsoft.Extensions.Logging;
+global using JetBrains.Annotations;

src/Testcontainers.ArangoDb/ArangoDbBuilder.cs

@@ -49,7 +49,7 @@ public ArangoDbBuilder WithPassword(string password)
     public override ArangoDbContainer Build()
     {
         Validate();
-        return new ArangoDbContainer(DockerResourceConfiguration, TestcontainersSettings.Logger);
+        return new ArangoDbContainer(DockerResourceConfiguration);
     }
 
     /// <inheritdoc />

src/Testcontainers.ArangoDb/ArangoDbContainer.cs

@@ -8,9 +8,8 @@ public sealed class ArangoDbContainer : DockerContainer
     /// Initializes a new instance of the <see cref="ArangoDbContainer" /> class.
     /// </summary>
     /// <param name="configuration">The container configuration.</param>
-    /// <param name="logger">The logger.</param>
-    public ArangoDbContainer(ArangoDbConfiguration configuration, ILogger logger)
-        : base(configuration, logger)
+    public ArangoDbContainer(ArangoDbConfiguration configuration)
+        : base(configuration)
     {
     }
 

src/Testcontainers.ArangoDb/Usings.cs

@@ -4,5 +4,4 @@
 global using DotNet.Testcontainers.Builders;
 global using DotNet.Testcontainers.Configurations;
 global using DotNet.Testcontainers.Containers;
-global using JetBrains.Annotations;
-global using Microsoft.Extensions.Logging;
+global using JetBrains.Annotations;

src/Testcontainers.Azurite/AzuriteBuilder.cs

@@ -86,7 +86,7 @@ public override AzuriteContainer Build()
         }
 
         var azuriteBuilder = DockerResourceConfiguration.WaitStrategies.Count() > 1 ? this : WithWaitStrategy(waitStrategy);
-        return new AzuriteContainer(azuriteBuilder.DockerResourceConfiguration, TestcontainersSettings.Logger);
+        return new AzuriteContainer(azuriteBuilder.DockerResourceConfiguration);
     }
 
     /// <inheritdoc />

src/Testcontainers.Azurite/AzuriteContainer.cs

@@ -12,9 +12,8 @@ public sealed class AzuriteContainer : DockerContainer
     /// Initializes a new instance of the <see cref="AzuriteContainer" /> class.
     /// </summary>
     /// <param name="configuration">The container configuration.</param>
-    /// <param name="logger">The logger.</param>
-    public AzuriteContainer(AzuriteConfiguration configuration, ILogger logger)
-        : base(configuration, logger)
+    public AzuriteContainer(AzuriteConfiguration configuration)
+        : base(configuration)
     {
     }
 

src/Testcontainers.Azurite/Usings.cs

@@ -5,5 +5,4 @@
 global using DotNet.Testcontainers.Builders;
 global using DotNet.Testcontainers.Configurations;
 global using DotNet.Testcontainers.Containers;
-global using JetBrains.Annotations;
-global using Microsoft.Extensions.Logging;
+global using JetBrains.Annotations;

src/Testcontainers.BigQuery/BigQueryBuilder.cs

@@ -46,7 +46,7 @@ public BigQueryBuilder WithProject(string projectId)
     public override BigQueryContainer Build()
     {
         Validate();
-        return new BigQueryContainer(DockerResourceConfiguration, TestcontainersSettings.Logger);
+        return new BigQueryContainer(DockerResourceConfiguration);
     }
 
     /// <inheritdoc />

src/Testcontainers.BigQuery/BigQueryContainer.cs

@@ -8,9 +8,8 @@ public sealed class BigQueryContainer : DockerContainer
     /// Initializes a new instance of the <see cref="BigQueryContainer" /> class.
     /// </summary>
     /// <param name="configuration">The container configuration.</param>
-    /// <param name="logger">The logger.</param>
-    public BigQueryContainer(BigQueryConfiguration configuration, ILogger logger)
-        : base(configuration, logger)
+    public BigQueryContainer(BigQueryConfiguration configuration)
+        : base(configuration)
     {
     }
 

src/Testcontainers.BigQuery/Usings.cs

@@ -3,5 +3,4 @@
 global using DotNet.Testcontainers.Builders;
 global using DotNet.Testcontainers.Configurations;
 global using DotNet.Testcontainers.Containers;
-global using JetBrains.Annotations;
-global using Microsoft.Extensions.Logging;
+global using JetBrains.Annotations;

src/Testcontainers.Bigtable/BigtableBuilder.cs

@@ -34,7 +34,7 @@ private BigtableBuilder(BigtableConfiguration resourceConfiguration)
     public override BigtableContainer Build()
     {
         Validate();
-        return new BigtableContainer(DockerResourceConfiguration, TestcontainersSettings.Logger);
+        return new BigtableContainer(DockerResourceConfiguration);
     }
 
     /// <inheritdoc />

src/Testcontainers.Bigtable/BigtableContainer.cs

@@ -8,9 +8,8 @@ public sealed class BigtableContainer : DockerContainer
     /// Initializes a new instance of the <see cref="BigtableContainer" /> class.
     /// </summary>
     /// <param name="configuration">The container configuration.</param>
-    /// <param name="logger">The logger.</param>
-    public BigtableContainer(IContainerConfiguration configuration, ILogger logger)
-        : base(configuration, logger)
+    public BigtableContainer(IContainerConfiguration configuration)
+        : base(configuration)
     {
     }
 

src/Testcontainers.Bigtable/Usings.cs

@@ -3,5 +3,4 @@
 global using DotNet.Testcontainers.Builders;
 global using DotNet.Testcontainers.Configurations;
 global using DotNet.Testcontainers.Containers;
-global using JetBrains.Annotations;
-global using Microsoft.Extensions.Logging;
+global using JetBrains.Annotations;

src/Testcontainers.ClickHouse/ClickHouseBuilder.cs

@@ -75,7 +75,7 @@ public ClickHouseBuilder WithPassword(string password)
     public override ClickHouseContainer Build()
     {
         Validate();
-        return new ClickHouseContainer(DockerResourceConfiguration, TestcontainersSettings.Logger);
+        return new ClickHouseContainer(DockerResourceConfiguration);
     }
 
     /// <inheritdoc />

src/Testcontainers.ClickHouse/ClickHouseContainer.cs

@@ -10,9 +10,8 @@ public sealed class ClickHouseContainer : DockerContainer, IDatabaseContainer
     /// Initializes a new instance of the <see cref="ClickHouseContainer" /> class.
     /// </summary>
     /// <param name="configuration">The container configuration.</param>
-    /// <param name="logger">The logger.</param>
-    public ClickHouseContainer(ClickHouseConfiguration configuration, ILogger logger)
-        : base(configuration, logger)
+    public ClickHouseContainer(ClickHouseConfiguration configuration)
+        : base(configuration)
     {
         _configuration = configuration;
     }

src/Testcontainers.ClickHouse/Usings.cs

@@ -11,5 +11,4 @@
 global using DotNet.Testcontainers.Builders;
 global using DotNet.Testcontainers.Configurations;
 global using DotNet.Testcontainers.Containers;
-global using JetBrains.Annotations;
-global using Microsoft.Extensions.Logging;
+global using JetBrains.Annotations;

src/Testcontainers.CockroachDb/CockroachDbBuilder.cs

@@ -75,7 +75,7 @@ public CockroachDbBuilder WithPassword(string password)
     public override CockroachDbContainer Build()
     {
         Validate();
-        return new CockroachDbContainer(DockerResourceConfiguration, TestcontainersSettings.Logger);
+        return new CockroachDbContainer(DockerResourceConfiguration);
     }
 
     /// <inheritdoc />

src/Testcontainers.CockroachDb/CockroachDbContainer.cs

@@ -10,9 +10,8 @@ public sealed class CockroachDbContainer : DockerContainer, IDatabaseContainer
     /// Initializes a new instance of the <see cref="CockroachDbContainer" /> class.
     /// </summary>
     /// <param name="configuration">The container configuration.</param>
-    /// <param name="logger">The logger.</param>
-    public CockroachDbContainer(CockroachDbConfiguration configuration, ILogger logger)
-        : base(configuration, logger)
+    public CockroachDbContainer(CockroachDbConfiguration configuration)
+        : base(configuration)
     {
         _configuration = configuration;
     }

src/Testcontainers.CockroachDb/Usings.cs

@@ -10,5 +10,4 @@
 global using DotNet.Testcontainers.Builders;
 global using DotNet.Testcontainers.Configurations;
 global using DotNet.Testcontainers.Containers;
-global using JetBrains.Annotations;
-global using Microsoft.Extensions.Logging;
+global using JetBrains.Annotations;

src/Testcontainers.Consul/ConsulBuilder.cs

@@ -36,7 +36,7 @@ private ConsulBuilder(ConsulConfiguration resourceConfiguration)
     public override ConsulContainer Build()
     {
         Validate();
-        return new ConsulContainer(DockerResourceConfiguration, TestcontainersSettings.Logger);
+        return new ConsulContainer(DockerResourceConfiguration);
     }
 
     /// <inheritdoc />

src/Testcontainers.Consul/ConsulContainer.cs

@@ -8,9 +8,8 @@ public sealed class ConsulContainer : DockerContainer
     /// Initializes a new instance of the <see cref="ConsulContainer" /> class.
     /// </summary>
     /// <param name="configuration">The container configuration.</param>
-    /// <param name="logger">The logger.</param>
-    public ConsulContainer(ConsulConfiguration configuration, ILogger logger)
-        : base(configuration, logger)
+    public ConsulContainer(ConsulConfiguration configuration)
+        : base(configuration)
     {
     }
 

src/Testcontainers.Consul/Usings.cs

@@ -3,5 +3,4 @@
 global using DotNet.Testcontainers.Builders;
 global using DotNet.Testcontainers.Configurations;
 global using DotNet.Testcontainers.Containers;
-global using JetBrains.Annotations;
-global using Microsoft.Extensions.Logging;
+global using JetBrains.Annotations;

src/Testcontainers.CosmosDb/CosmosDbBuilder.cs

@@ -36,7 +36,7 @@ private CosmosDbBuilder(CosmosDbConfiguration resourceConfiguration)
     public override CosmosDbContainer Build()
     {
         Validate();
-        return new CosmosDbContainer(DockerResourceConfiguration, TestcontainersSettings.Logger);
+        return new CosmosDbContainer(DockerResourceConfiguration);
     }
 
     /// <inheritdoc />

src/Testcontainers.CosmosDb/CosmosDbContainer.cs

@@ -8,9 +8,8 @@ public sealed class CosmosDbContainer : DockerContainer
     /// Initializes a new instance of the <see cref="CosmosDbContainer" /> class.
     /// </summary>
     /// <param name="configuration">The container configuration.</param>
-    /// <param name="logger">The logger.</param>
-    public CosmosDbContainer(CosmosDbConfiguration configuration, ILogger logger)
-        : base(configuration, logger)
+    public CosmosDbContainer(CosmosDbConfiguration configuration)
+        : base(configuration)
     {
     }
 

src/Testcontainers.CosmosDb/Usings.cs

@@ -8,5 +8,4 @@
 global using DotNet.Testcontainers.Builders;
 global using DotNet.Testcontainers.Configurations;
 global using DotNet.Testcontainers.Containers;
-global using JetBrains.Annotations;
-global using Microsoft.Extensions.Logging;
+global using JetBrains.Annotations;

src/Testcontainers.CouchDb/CouchDbBuilder.cs

@@ -60,7 +60,7 @@ public CouchDbBuilder WithPassword(string password)
     public override CouchDbContainer Build()
     {
         Validate();
-        return new CouchDbContainer(DockerResourceConfiguration, TestcontainersSettings.Logger);
+        return new CouchDbContainer(DockerResourceConfiguration);
     }
 
     /// <inheritdoc />

src/Testcontainers.CouchDb/CouchDbContainer.cs

@@ -10,9 +10,8 @@ public sealed class CouchDbContainer : DockerContainer
     /// Initializes a new instance of the <see cref="CouchDbContainer" /> class.
     /// </summary>
     /// <param name="configuration">The container configuration.</param>
-    /// <param name="logger">The logger.</param>
-    public CouchDbContainer(CouchDbConfiguration configuration, ILogger logger)
-        : base(configuration, logger)
+    public CouchDbContainer(CouchDbConfiguration configuration)
+        : base(configuration)
     {
         _configuration = configuration;
     }

src/Testcontainers.CouchDb/Usings.cs

@@ -4,5 +4,4 @@
 global using DotNet.Testcontainers.Builders;
 global using DotNet.Testcontainers.Configurations;
 global using DotNet.Testcontainers.Containers;
-global using JetBrains.Annotations;
-global using Microsoft.Extensions.Logging;
+global using JetBrains.Annotations;

src/Testcontainers.Couchbase/CouchbaseBuilder.cs

@@ -122,7 +122,7 @@ public override CouchbaseContainer Build()
         }
 
         var couchbaseBuilder = DockerResourceConfiguration.WaitStrategies.Count() > 1 ? this : WithWaitStrategy(waitStrategy);
-        return new CouchbaseContainer(couchbaseBuilder.DockerResourceConfiguration, TestcontainersSettings.Logger);
+        return new CouchbaseContainer(couchbaseBuilder.DockerResourceConfiguration);
     }
 
     /// <inheritdoc />

src/Testcontainers.Couchbase/CouchbaseContainer.cs

@@ -10,9 +10,8 @@ public sealed class CouchbaseContainer : DockerContainer
     /// Initializes a new instance of the <see cref="CouchbaseContainer" /> class.
     /// </summary>
     /// <param name="configuration">The container configuration.</param>
-    /// <param name="logger">The logger.</param>
-    public CouchbaseContainer(CouchbaseConfiguration configuration, ILogger logger)
-        : base(configuration, logger)
+    public CouchbaseContainer(CouchbaseConfiguration configuration)
+        : base(configuration)
     {
         _configuration = configuration;
         Starting += (_, _) => Logger.LogInformation("Couchbase container is starting, performing configuration.");

src/Testcontainers.DynamoDb/DynamoDbBuilder.cs

@@ -34,7 +34,7 @@ private DynamoDbBuilder(DynamoDbConfiguration dockerResourceConfiguration)
     public override DynamoDbContainer Build()
     {
         Validate();
-        return new DynamoDbContainer(DockerResourceConfiguration, TestcontainersSettings.Logger);
+        return new DynamoDbContainer(DockerResourceConfiguration);
     }
 
     /// <inheritdoc />

src/Testcontainers.DynamoDb/DynamoDbContainer.cs

@@ -8,9 +8,8 @@ public sealed class DynamoDbContainer : DockerContainer
     /// Initializes a new instance of the <see cref="DynamoDbContainer" /> class.
     /// </summary>
     /// <param name="configuration">The container configuration.</param>
-    /// <param name="logger">The logger.</param>
-    public DynamoDbContainer(DynamoDbConfiguration configuration, ILogger logger)
-        : base(configuration, logger)
+    public DynamoDbContainer(DynamoDbConfiguration configuration)
+        : base(configuration)
     {
     }
 

src/Testcontainers.DynamoDb/Usings.cs

@@ -4,5 +4,4 @@
 global using DotNet.Testcontainers.Builders;
 global using DotNet.Testcontainers.Configurations;
 global using DotNet.Testcontainers.Containers;
-global using JetBrains.Annotations;
-global using Microsoft.Extensions.Logging;
+global using JetBrains.Annotations;

src/Testcontainers.Elasticsearch/ElasticsearchBuilder.cs

@@ -59,7 +59,7 @@ public ElasticsearchBuilder WithPassword(string password)
     public override ElasticsearchContainer Build()
     {
         Validate();
-        return new ElasticsearchContainer(DockerResourceConfiguration, TestcontainersSettings.Logger);
+        return new ElasticsearchContainer(DockerResourceConfiguration);
     }
 
     /// <inheritdoc />

src/Testcontainers.Elasticsearch/ElasticsearchContainer.cs

@@ -10,9 +10,8 @@ public sealed class ElasticsearchContainer : DockerContainer
     /// Initializes a new instance of the <see cref="ElasticsearchContainer" /> class.
     /// </summary>
     /// <param name="configuration">The container configuration.</param>
-    /// <param name="logger">The logger.</param>
-    public ElasticsearchContainer(ElasticsearchConfiguration configuration, ILogger logger)
-        : base(configuration, logger)
+    public ElasticsearchContainer(ElasticsearchConfiguration configuration)
+        : base(configuration)
     {
         _configuration = configuration;
     }

src/Testcontainers.Elasticsearch/Usings.cs

@@ -8,5 +8,4 @@
 global using DotNet.Testcontainers.Builders;
 global using DotNet.Testcontainers.Configurations;
 global using DotNet.Testcontainers.Containers;
-global using JetBrains.Annotations;
-global using Microsoft.Extensions.Logging;
+global using JetBrains.Annotations;