- fix localstack integration tests

Author: NinjaRocks

tests/SourceFlow.Cloud.AWS.Tests/Integration/KmsSecurityAndPerformanceTests.cs

@@ -169,7 +169,7 @@ public async Task IamPermissions_WithInvalidKey_ShouldThrowException()
         var plaintext = "Test message";
 
         // Act & Assert - Should fail with invalid key
-        await Assert.ThrowsAsync<Exception>(async () =>
+        await Assert.ThrowsAnyAsync<Exception>(async () =>
         {
             await encryption.EncryptAsync(plaintext);
         });

tests/SourceFlow.Cloud.AWS.Tests/TestHelpers/LocalStackManager.cs

@@ -22,6 +22,7 @@ public class LocalStackManager : ILocalStackManager
     private IContainer? _container;
     private LocalStackConfiguration? _configuration;
     private bool _disposed;
+    private bool _isExternalInstance;
     private readonly Dictionary<string, DateTime> _serviceReadyTimes = new();
     private readonly object _lockObject = new();
 
@@ -31,7 +32,7 @@ public LocalStackManager(ILogger<LocalStackManager> logger)
     }
 
     /// <inheritdoc />
-    public bool IsRunning => _container?.State == TestcontainersStates.Running;
+    public bool IsRunning => _container?.State == TestcontainersStates.Running || _isExternalInstance;
 
     /// <inheritdoc />
     public string Endpoint => _configuration?.Endpoint ?? "http://localhost:4566";
@@ -54,7 +55,7 @@ public async Task StartAsync(LocalStackConfiguration config)
         if (await IsExternalLocalStackAvailableAsync(config.Endpoint))
         {
             _logger.LogInformation("Detected existing LocalStack instance at {Endpoint}, using it instead of starting new container", config.Endpoint);
-            // Don't start a new container, just use the existing one
+            _isExternalInstance = true;
             return;
         }
 
@@ -152,14 +153,22 @@ public async Task StartAsync(LocalStackConfiguration config)
     /// <inheritdoc />
     public async Task StopAsync()
     {
+        if (_isExternalInstance)
+        {
+            _logger.LogInformation("Using external LocalStack instance — skipping stop");
+            _isExternalInstance = false;
+            _configuration = null;
+            return;
+        }
+
         if (_container == null)
             return;
-        
+
         _logger.LogInformation("Stopping LocalStack container");
-        
+
         try
         {
-            if (IsRunning)
+            if (_container.State == TestcontainersStates.Running)
             {
                 await _container.StopAsync();
             }
@@ -174,7 +183,7 @@ public async Task StopAsync()
             _container = null;
             _configuration = null;
         }
-        
+
         _logger.LogInformation("LocalStack container stopped");
     }