Make the logger configurable with a new WithLogger method on the buil…

…ders

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

Fixes #996

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;