fix: typo in extension methods

changes:
- renamed the method to `FireAndForgetSafeAsync`
- added some unit testing

closes #107

Author: BoBoBaSs84

src/BB84.Notifications/Commands/AsyncActionCommand.cs

@@ -46,7 +46,7 @@ public bool CanExecute(object? parameter)
 
   /// <inheritdoc/>
   public void Execute(object? parameter)
-    => ExecuteAsync().ToSaveVoid(handler);
+    => ExecuteAsync().FireAndForgetSafeAsync(handler);
 
   /// <inheritdoc/>
   public void RaiseCanExecuteChanged()
@@ -80,7 +80,7 @@ public bool CanExecute(object? parameter)
 
   /// <inheritdoc/>
   public void Execute(object? parameter)
-    => ExecuteAsync((T)parameter!).ToSaveVoid(handler);
+    => ExecuteAsync((T)parameter!).FireAndForgetSafeAsync(handler);
 
   /// <inheritdoc/>
   public async Task ExecuteAsync(T parameter)

src/BB84.Notifications/Extensions/TaskExtensions.cs

@@ -9,19 +9,19 @@ public static class TaskExtensions
 {
   /// <summary>
   /// Awaits the provided <paramref name="task"/>, uses the exception <paramref name="handler"/>
-  /// if an <see cref="Exception"/> occured and returns <see cref="void"/>.
+  /// if an <see cref="Exception"/> occurs and returns <see cref="void"/>.
   /// </summary>
   /// <param name="task">The task to await.</param>
   /// <param name="handler">Sends the exception to an exception handler.</param>
-  public static async void ToSaveVoid(this Task task, IExceptionHandler? handler = null)
+  public static async void FireAndForgetSafeAsync(this Task task, IExceptionHandler? handler = null)
   {
     try
     {
       await task;
     }
     catch (Exception ex)
     {
-      handler?.Handle(ex);
+      handler?.HandleError(ex);
     }
   }
 }

src/BB84.Notifications/Interfaces/Components/IExceptionHandler.cs

@@ -9,5 +9,5 @@ public interface IExceptionHandler
   /// If an error occurs, it send the exception to an error handler.
   /// </summary>
   /// <param name="exception">The exception to handle.</param>
-  void Handle(Exception exception);
+  void HandleError(Exception exception);
 }

tests/BB84.NotificationsTests/Extensions/TaskExtensionsTests.cs

@@ -0,0 +1,33 @@
+using BB84.Notifications.Extensions;
+using BB84.Notifications.Interfaces.Components;
+
+namespace BB84.NotificationsTests.Extensions;
+
+[TestClass]
+[SuppressMessage("Usage", "CA2201", Justification = "UnitTest")]
+public sealed class TaskExtensionsTests
+{
+  [TestMethod]
+  public void FireAndForgetSafeAsyncTest()
+  {
+    Task task = Task.Factory.StartNew(() => { });
+
+    task.FireAndForgetSafeAsync();
+
+    Assert.IsNotNull(task);
+  }
+
+  [TestMethod]
+  public void FireAndForgetSafeAsyncExceptionHandledTest()
+  {
+    VoidHandler handler = new();
+    Task task = Task.Factory.StartNew(() => throw new Exception(""));
+
+    task.FireAndForgetSafeAsync(handler);
+  }
+
+  private sealed class VoidHandler : IExceptionHandler
+  {
+    public void HandleError(Exception exception) { }
+  }
+}