[#242] #REMOVE 'assemblyName: DotNet.Testcontainers; function: PurgeO…

…rphanedContainersArgs'

{Remove PurgeOrphanedContainersArgs.}

build.cake

@@ -85,7 +85,7 @@ Task("Tests")
       Verbosity = param.Verbosity,
       NoRestore = true,
       NoBuild = true,
-      Logger = "trx",
+      Loggers = new[] { "trx" },
       Filter = param.TestFilter,
       ResultsDirectory = param.Paths.Directories.TestResults,
       ArgumentCustomization = args => args

src/DotNet.Testcontainers/Clients/DockerApiClient.cs

@@ -9,7 +9,7 @@ internal abstract class DockerApiClient
     private static readonly ConcurrentDictionary<Uri, IDockerClient> Clients = new ConcurrentDictionary<Uri, IDockerClient>();
 
     protected DockerApiClient(Uri endpoint) : this(
-      Clients.GetOrAdd(endpoint, key => new DockerClientConfiguration(endpoint).CreateClient()))
+      Clients.GetOrAdd(endpoint, _ => new DockerClientConfiguration(endpoint).CreateClient()))
     {
     }
 

src/DotNet.Testcontainers/Clients/TestcontainersClient.cs

@@ -53,9 +53,6 @@ internal sealed class TestcontainersClient : ITestcontainersClient
       this.containers = containerOperations;
       this.images = imageOperations;
       this.system = systemOperations;
-
-      AppDomain.CurrentDomain.ProcessExit += this.PurgeOrphanedContainers;
-      Console.CancelKeyPress += this.PurgeOrphanedContainers;
     }
 
     ~TestcontainersClient()
@@ -73,8 +70,6 @@ public bool IsRunningInsideDocker
 
     public void Dispose()
     {
-      AppDomain.CurrentDomain.ProcessExit -= this.PurgeOrphanedContainers;
-      Console.CancelKeyPress -= this.PurgeOrphanedContainers;
       GC.SuppressFinalize(this);
     }
 
@@ -125,7 +120,7 @@ await this.containers.RemoveAsync(id, ct)
         }
         catch (DockerApiException e)
         {
-          // The Docker daemon may already start the progress to remove the container (AutoRemove).
+          // The Docker daemon may already start the progress to removes the container (AutoRemove).
           // https://docs.docker.com/engine/api/v1.41/#operation/ContainerCreate.
           if (!e.Message.Contains($"removal of container {id} is already in progress"))
           {
@@ -199,11 +194,5 @@ public Task<string> BuildAsync(IImageFromDockerfileConfiguration configuration,
     {
       return this.images.BuildAsync(configuration, ct);
     }
-
-    private void PurgeOrphanedContainers(object sender, EventArgs args)
-    {
-      var arguments = new PurgeOrphanedContainersArgs(this.endpoint, this.registryService.GetRegisteredContainers());
-      _ = new Process { StartInfo = { FileName = "docker", Arguments = arguments.ToString() } }.Start();
-    }
   }
 }

src/DotNet.Testcontainers/Containers/Modules/TestcontainersContainer.cs

@@ -233,7 +233,7 @@ await Task.WhenAll(attachTask, startTask)
       await this.configuration.StartupCallback(this, ct)
         .ConfigureAwait(false);
 
-      // Do not use a to small frequency. Especially with a lot of containers,
+      // Do not use a too small frequency. Especially with a lot of containers,
       // we send many operations to the Docker endpoint. The endpoint may cancel operations.
       foreach (var waitStrategy in this.configuration.WaitStrategies)
       {