CatalogIntegrationEventService Refactored

Author: CESARDELATORRE

src/Services/Catalog/Catalog.API/Controllers/CatalogController.cs

@@ -152,13 +152,16 @@ public async Task<IActionResult> UpdateProduct([FromBody]CatalogItem productToUp
             catalogItem = productToUpdate;
             _catalogContext.CatalogItems.Update(catalogItem);
 
-            if (raiseProductPriceChangedEvent) // Save and publish event if price has changed
-            {                               
+            if (raiseProductPriceChangedEvent) // Save and publish integration event if price has changed
+            {
+                //Create Integration Event to be published through the Event Bus
                 var priceChangedEvent = new ProductPriceChangedIntegrationEvent(catalogItem.Id, productToUpdate.Price, oldPrice);
+                
                 // Achieving atomicity between original Catalog database operation and the IntegrationEventLog thanks to a local transaction
-                await _catalogIntegrationEventService.SaveEventAsync(priceChangedEvent);
-                // Publish to Event Bus only if product price changed
-                await _catalogIntegrationEventService.PublishAsync(priceChangedEvent);
+                await _catalogIntegrationEventService.SaveEventAndCatalogContextChangesAsync(priceChangedEvent);
+                
+                // Publish through the Event Bus and mark the saved event as published
+                await _catalogIntegrationEventService.PublishThroughEventBusAsync(priceChangedEvent);
             }
             else // Save updated product
             {

src/Services/Catalog/Catalog.API/IntegrationEvents/CatalogIntegrationEventService.cs

@@ -27,13 +27,13 @@ public CatalogIntegrationEventService(IEventBus eventBus, CatalogContext catalog
             _eventLogService = _integrationEventLogServiceFactory(_catalogContext.Database.GetDbConnection());
         }
 
-        public async Task PublishAsync(IntegrationEvent evt)
+        public async Task PublishThroughEventBusAsync(IntegrationEvent evt)
         {
             _eventBus.Publish(evt);
             await _eventLogService.MarkEventAsPublishedAsync(evt);
         }
 
-        public async Task SaveEventAsync(IntegrationEvent evt)
+        public async Task SaveEventAndCatalogContextChangesAsync(IntegrationEvent evt)
         {
             //Use of an EF Core resiliency strategy when using multiple DbContexts within an explicit BeginTransaction():
             //See: https://docs.microsoft.com/en-us/ef/core/miscellaneous/connection-resiliency            

src/Services/Catalog/Catalog.API/IntegrationEvents/ICatalogIntegrationEventService.cs

@@ -8,7 +8,7 @@ namespace Catalog.API.IntegrationEvents
 {
     public interface ICatalogIntegrationEventService
     {
-        Task SaveEventAsync(IntegrationEvent evt);
-        Task PublishAsync(IntegrationEvent evt);
+        Task SaveEventAndCatalogContextChangesAsync(IntegrationEvent evt);
+        Task PublishThroughEventBusAsync(IntegrationEvent evt);
     }
 }