Merge pull request #67 from LBHackney-IT/feature/error-handling-ddb-f…

…ixture Ignore ConditionalCheckFailedException during entity deletion in DynamoDbFixture
LBHackney-IT · Feb 4, 2025 · fc38e54 · fc38e54
2 parents 49ec337 + 2d09609
commit fc38e54
Showing 1 changed file with 12 additions and 1 deletion.
diff --git a/Hackney.Core/Hackney.Core.Testing/Hackney.Core.Testing.DynamoDb/DynamoDbFixture.cs b/Hackney.Core/Hackney.Core.Testing/Hackney.Core.Testing.DynamoDb/DynamoDbFixture.cs
@@ -112,7 +112,18 @@ public void EnsureTablesExist(List<TableDef> tables)
         public async Task SaveEntityAsync<T>(T entity) where T : class
         {
             await DynamoDbContext.SaveAsync<T>(entity).ConfigureAwait(false);
-            _cleanup.Add(async () => await DynamoDbContext.DeleteAsync(entity).ConfigureAwait(false));
+            _cleanup.Add(async () =>
+            {
+                try
+                {
+                    await DynamoDbContext.DeleteAsync(entity).ConfigureAwait(false);
+                }
+                catch (ConditionalCheckFailedException ex)
+                {
+                    // Ignore this exception as it means the entity has already been deleted
+                    Console.WriteLine($"Delete failed: {ex.Message}");
+                }
+            });
         }
     }
 }