perf(ErrorLog): improving the error handling logic (#5600)

* refactor: 优化逻辑接管后内部不再弹出 Toast

* test: 更新单元测试

Author: ArgoZhang

src/BootstrapBlazor/Components/ErrorLogger/BootstrapBlazorErrorBoundary.cs

@@ -61,16 +61,14 @@ protected override async Task OnErrorAsync(Exception exception)
         if (OnErrorHandleAsync != null)
         {
             await OnErrorHandleAsync(Logger, exception);
+            return;
         }
-        else
-        {
-            if (ShowToast)
-            {
-                await ToastService.Error(ToastTitle, exception.Message);
-            }
 
-            Logger.LogError(exception, "{BootstrapBlazorErrorBoundary} {OnErrorAsync} log this error occurred at {Page}", nameof(BootstrapBlazorErrorBoundary), nameof(OnErrorAsync), NavigationManager.Uri);
+        if (ShowToast)
+        {
+            await ToastService.Error(ToastTitle, exception.Message);
         }
+        Logger.LogError(exception, "{BootstrapBlazorErrorBoundary} {OnErrorAsync} log this error occurred at {Page}", nameof(BootstrapBlazorErrorBoundary), nameof(OnErrorAsync), NavigationManager.Uri);
     }
 
     /// <summary>
@@ -126,16 +124,14 @@ private MarkupString GetErrorContentMarkupString(Exception ex)
     /// <param name="handler"></param>
     public async Task RenderException(Exception exception, IHandlerException? handler)
     {
-        await OnErrorAsync(exception);
-
         if (handler != null)
         {
             await handler.HandlerException(exception, ExceptionContent);
+            return;
         }
-        else
-        {
-            _exception = exception;
-            StateHasChanged();
-        }
+
+        await OnErrorAsync(exception);
+        _exception = exception;
+        StateHasChanged();
     }
 }

src/BootstrapBlazor/Components/ErrorLogger/ErrorLogger.cs

@@ -56,7 +56,8 @@ public class ErrorLogger : ComponentBase, IErrorLogger
     [Parameter]
     public RenderFragment<Exception>? ErrorContent { get; set; }
 
-    private BootstrapBlazorErrorBoundary _errorBoundary = default!;
+    [NotNull]
+    private BootstrapBlazorErrorBoundary? _errorBoundary = default;
 
     /// <summary>
     /// <inheritdoc/>

test/UnitTest/Components/ErrorHandlerTest.cs

@@ -30,13 +30,31 @@ await cut.InvokeAsync(() => dialog.Show(new DialogOption()
         // 关闭弹窗
         var btn = cut.Find(".modal-header .btn-close");
         await cut.InvokeAsync(() => btn.Click());
+    }
 
+    [Fact]
+    public async Task ShowToast_Ok()
+    {
+        var cut = Context.RenderComponent<BootstrapBlazorRoot>(pb =>
+        {
+            pb.AddChildContent<ErrorComponent>();
+        });
+        var errorButton = cut.Find(".btn-error");
+        await cut.InvokeAsync(() => errorButton.Click());
         cut.Contains("<div class=\"toast-body\">test error logger</div>");
 
         // 关闭 Toast
         var toast = cut.FindComponent<Toast>().Instance;
         await cut.InvokeAsync(() => toast.Close());
         cut.DoesNotContain("<div class=\"toast-body\">test error logger</div>");
+
+        cut.SetParametersAndRender(pb =>
+        {
+            pb.Add(a => a.ShowToast, false);
+        });
+        errorButton = cut.Find(".btn-error");
+        await cut.InvokeAsync(() => errorButton.Click());
+        cut.DoesNotContain("<div class=\"toast-body\">test error logger</div>");
     }
 
     private class MockDialogTest : ComponentBase