fix: Remove Sonar code smells

testcontainers · Oct 3, 2024 · f4966d8 · f4966d8
1 parent e96d4e6
commit f4966d8
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git a/src/Testcontainers.Neo4j/Neo4jBuilder.cs b/src/Testcontainers.Neo4j/Neo4jBuilder.cs
@@ -78,10 +78,10 @@ public Neo4jBuilder WithEnterpriseEdition(bool acceptLicenseAgreement)
         {
             tag = image.Tag;
         }
-        else if (image.MatchVersion(v => operatingSystems.Any(v.Contains)))
+        else if (image.MatchVersion(v => Array.Exists(operatingSystems, v.Contains)))
         {
             MatchEvaluator evaluator = match => $"{enterpriseSuffix}-{match.Value}";
-            tag = Regex.Replace(image.Tag, string.Join("|", operatingSystems), evaluator);
+            tag = Regex.Replace(image.Tag, string.Join("|", operatingSystems), evaluator, RegexOptions.None, TimeSpan.FromSeconds(1));
         }
         else
         {

diff --git a/src/Testcontainers/Clients/DockerImageOperations.cs b/src/Testcontainers/Clients/DockerImageOperations.cs
@@ -59,7 +59,7 @@ public async Task CreateAsync(IImage image, IDockerRegistryAuthenticationConfigu
     {
       var createParameters = new ImagesCreateParameters
       {
-        FromImage = image.FullName
+        FromImage = image.FullName,
       };
 
       var authConfig = new AuthConfig

diff --git a/tests/Testcontainers.Platform.Linux.Tests/Usings.cs b/tests/Testcontainers.Platform.Linux.Tests/Usings.cs
@@ -8,7 +8,6 @@
 global using System.Text;
 global using System.Threading;
 global using System.Threading.Tasks;
-global using Docker.DotNet;
 global using Docker.DotNet.Models;
 global using DotNet.Testcontainers;
 global using DotNet.Testcontainers.Builders;