Merge pull request #169 from koenbeuk/issue-168

Ensure that we delist the trigger session on exception in the BeforeSave cycle

Author: koenbeuk

src/EntityFrameworkCore.Triggered/Internal/TriggerSessionSaveChangesInterceptor.cs

@@ -80,14 +80,19 @@ public InterceptionResult<int> SavingChanges(DbContextEventData eventData, Inter
 
             try
             {
-
                 eventData.Context.ChangeTracker.AutoDetectChangesEnabled = false;
 
                 _triggerSession.RaiseBeforeSaveStartingTriggers();
                 _triggerSession.RaiseBeforeSaveTriggers();
                 _triggerSession.CaptureDiscoveredChanges();
                 _triggerSession.RaiseBeforeSaveCompletedTriggers();
             }
+            catch
+            {
+                // We're aborting the SaveChanges call, delist the trigger session now
+                DelistTriggerSession(eventData);
+                throw;
+            }
             finally
             {
                 eventData.Context.ChangeTracker.AutoDetectChangesEnabled = defaultAutoDetectChangesEnabled;
@@ -118,6 +123,12 @@ public async ValueTask<InterceptionResult<int>> SavingChangesAsync(DbContextEven
                 _triggerSession.RaiseBeforeSaveCompletedTriggers();
                 await _triggerSession.RaiseBeforeSaveCompletedAsyncTriggers(cancellationToken).ConfigureAwait(false);
             }
+            catch
+            {
+                // We're aborting the SaveChanges call, delist the trigger session now
+                DelistTriggerSession(eventData);
+                throw;
+            }
             finally
             {
                 eventData.Context.ChangeTracker.AutoDetectChangesEnabled = defaultAutoDetectChangesEnabled;