chore: Remove unused class WaitTwoSeconds

Author: HofmeisterAn

src/Testcontainers.Db2/Db2Container.cs

@@ -42,7 +42,7 @@ public async Task<ExecResult> ExecScriptAsync(string scriptContent, Cancellation
         await CopyAsync(Encoding.Default.GetBytes(scriptContent), scriptFilePath, Unix.FileMode644, ct)
             .ConfigureAwait(false);
 
-        return await ExecAsync(new [] { "/bin/sh", "-c", db2ShellCommand}, ct)
+        return await ExecAsync(new[] { "/bin/sh", "-c", db2ShellCommand}, ct)
             .ConfigureAwait(false);
     }
 }

src/Testcontainers.EventHubs/EventHubsBuilder.cs

@@ -139,7 +139,7 @@ protected override EventHubsBuilder Init()
             .WithPortBinding(EventHubsPort, true)
             .WithPortBinding(EventHubsHttpPort, true)
             .WithWaitStrategy(Wait.ForUnixContainer().UntilHttpRequestIsSucceeded(request =>
-                request.ForPath("/health").ForPort(EventHubsHttpPort)));
+                request.ForPort(EventHubsHttpPort).ForPath("/health")));
     }
 
     /// <inheritdoc />
@@ -159,22 +159,4 @@ protected override EventHubsBuilder Merge(EventHubsConfiguration oldValue, Event
     {
         return new EventHubsBuilder(new EventHubsConfiguration(oldValue, newValue));
     }
-
-    /// <inheritdoc cref="IWaitUntil" />
-    /// <remarks>
-    /// This is a workaround to ensure that the wait strategy does not indicate
-    /// readiness too early:
-    /// https://github.com/Azure/azure-service-bus-emulator-installer/issues/35#issuecomment-2497164533.
-    /// </remarks>
-    private sealed class WaitTwoSeconds : IWaitUntil
-    {
-        /// <inheritdoc />
-        public async Task<bool> UntilAsync(IContainer container)
-        {
-            await Task.Delay(TimeSpan.FromSeconds(2))
-                .ConfigureAwait(false);
-
-            return true;
-        }
-    }
 }

src/Testcontainers.ServiceBus/ServiceBusBuilder.cs

@@ -120,7 +120,7 @@ protected override ServiceBusBuilder Init()
             .WithPortBinding(ServiceBusHttpPort, true)
             .WithEnvironment("SQL_WAIT_INTERVAL", "0")
             .WithWaitStrategy(Wait.ForUnixContainer().UntilHttpRequestIsSucceeded(request =>
-                request.ForPath("/health").ForPort(ServiceBusHttpPort)));
+                request.ForPort(ServiceBusHttpPort).ForPath("/health")));
     }
 
     /// <inheritdoc />
@@ -140,22 +140,4 @@ protected override ServiceBusBuilder Merge(ServiceBusConfiguration oldValue, Ser
     {
         return new ServiceBusBuilder(new ServiceBusConfiguration(oldValue, newValue));
     }
-
-    /// <inheritdoc cref="IWaitUntil" />
-    /// <remarks>
-    /// This is a workaround to ensure that the wait strategy does not indicate
-    /// readiness too early:
-    /// https://github.com/Azure/azure-service-bus-emulator-installer/issues/35#issuecomment-2497164533.
-    /// </remarks>
-    private sealed class WaitTwoSeconds : IWaitUntil
-    {
-        /// <inheritdoc />
-        public async Task<bool> UntilAsync(IContainer container)
-        {
-            await Task.Delay(TimeSpan.FromSeconds(2))
-                .ConfigureAwait(false);
-
-            return true;
-        }
-    }
 }

src/Testcontainers/Containers/DockerContainer.cs

@@ -505,7 +505,7 @@ await _client.AttachAsync(_container.ID, _configuration.OutputConsumer, ct)
       await _client.StartAsync(_container.ID, ct)
         .ConfigureAwait(false);
 
-      _ = await CheckReadinessAsync(new [] { portBindingsMapped }, ct)
+      _ = await CheckReadinessAsync(new[] { portBindingsMapped }, ct)
         .ConfigureAwait(false);
 
       Starting?.Invoke(this, EventArgs.Empty);

src/Testcontainers/Images/DockerfileArchive.cs

@@ -4,7 +4,6 @@ namespace DotNet.Testcontainers.Images
   using System.Collections.Generic;
   using System.IO;
   using System.Linq;
-  using System.Runtime.InteropServices;
   using System.Text;
   using System.Text.RegularExpressions;
   using System.Threading;
@@ -150,7 +149,7 @@ public async Task<string> Tar(CancellationToken ct = default)
               {
                 var entry = TarEntry.CreateTarEntry(relativeFilePath);
                 entry.TarHeader.Size = inputStream.Length;
-                entry.TarHeader.Mode = GetFileMode(absoluteFilePath);
+                entry.TarHeader.Mode = GetUnixFileMode(absoluteFilePath);
 
                 await tarOutputStream.PutNextEntryAsync(entry, ct)
                   .ConfigureAwait(false);
@@ -188,21 +187,21 @@ private static IEnumerable<string> GetFiles(string directory)
     }
 
     /// <summary>
-    /// Gets the file mode for a given file.
+    /// Gets the Unix file mode of the file on the path.
     /// </summary>
-    /// <param name="path">The path to the file.</param>
-    /// <returns>The file mode for the <see cref="TarEntry"/></returns>
-    private static int GetFileMode(string path)
+    /// <param name="filePath">The path to the file.</param>
+    /// <returns>The Unix file mode of the file on the path.</returns>
+    private static int GetUnixFileMode(string filePath)
     {
 #if NET7_0_OR_GREATER
-      if (!RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
+      if (!OperatingSystem.IsWindows())
       {
-        return (int)File.GetUnixFileMode(path);
+        return (int)File.GetUnixFileMode(filePath);
       }
 #endif
 
-      // Default to 755 for Windows and fallback to 755 for Unix when
-      // `GetUnixFileMode` is not available.
+      // Default to 755 for Windows and fall back to 755 for Unix when `GetUnixFileMode`
+      // is not available.
       return (int)Unix.FileMode755;
     }
   }