Updated logger to use event instead of action property

Author: jschick04

src/EventLogExpert.UI.Tests/Services/DebugLogServiceTests.cs

@@ -147,21 +147,22 @@ public void Constructor_WhenLogLevelIsTrace_ShouldEnableFirstChanceExceptionLogg
     }
 
     [Fact]
-    public void DebugLogLoaded_WhenSet_ShouldBeRetrievable()
+    public void DebugLogLoaded_WhenSubscribed_ShouldBeInvocable()
     {
         // Arrange
         var fileLocationOptions = new FileLocationOptions(_testDirectory);
         var mockSettingsService = CreateMockSettingsService(LogLevel.Information);
 
         using var debugLogService = new DebugLogService(fileLocationOptions, mockSettingsService);
 
-        Action testAction = () => { };
+        var wasInvoked = false;
+        debugLogService.DebugLogLoaded += () => wasInvoked = true;
 
         // Act
-        debugLogService.DebugLogLoaded = testAction;
+        debugLogService.LoadDebugLog();
 
         // Assert
-        Assert.Same(testAction, debugLogService.DebugLogLoaded);
+        Assert.True(wasInvoked);
     }
 
     public void Dispose()
@@ -234,22 +235,21 @@ public async Task LoadAsync_WhenLogFileIsEmpty_ShouldReturnNoLines()
     }
 
     [Fact]
-    public void LoadDebugLog_WhenDebugLogLoadedIsNull_ShouldNotThrow()
+    public void LoadDebugLog_WhenNoSubscribers_ShouldNotThrow()
     {
         // Arrange
         var fileLocationOptions = new FileLocationOptions(_testDirectory);
         var mockSettingsService = CreateMockSettingsService(LogLevel.Information);
 
         using var debugLogService = new DebugLogService(fileLocationOptions, mockSettingsService);
-        debugLogService.DebugLogLoaded = null;
 
-        // Act & Assert
+        // Act & Assert - no subscribers attached
         var exception = Record.Exception(() => debugLogService.LoadDebugLog());
         Assert.Null(exception);
     }
 
     [Fact]
-    public void LoadDebugLog_WhenDebugLogLoadedIsSet_ShouldInvokeAction()
+    public void LoadDebugLog_WhenSubscribed_ShouldInvokeHandler()
     {
         // Arrange
         var fileLocationOptions = new FileLocationOptions(_testDirectory);
@@ -258,7 +258,7 @@ public void LoadDebugLog_WhenDebugLogLoadedIsSet_ShouldInvokeAction()
         using var debugLogService = new DebugLogService(fileLocationOptions, mockSettingsService);
 
         var actionInvoked = false;
-        debugLogService.DebugLogLoaded = () => actionInvoked = true;
+        debugLogService.DebugLogLoaded += () => actionInvoked = true;
 
         // Act
         debugLogService.LoadDebugLog();

src/EventLogExpert.UI/Interfaces/IFileLogger.cs

@@ -7,7 +7,7 @@ namespace EventLogExpert.UI.Interfaces;
 
 public interface IFileLogger
 {
-    Action? DebugLogLoaded { get; set; }
+    event Action? DebugLogLoaded;
 
     Task ClearAsync();
 

src/EventLogExpert.UI/Services/DebugLogService.cs

@@ -34,7 +34,7 @@ public DebugLogService(FileLocationOptions fileLocationOptions, ISettingsService
         AppDomain.CurrentDomain.UnhandledException += OnUnhandledException;
     }
 
-    public Action? DebugLogLoaded { get; set; }
+    public event Action? DebugLogLoaded;
 
     public async Task ClearAsync()
     {