fix: Set Task as successful when exception should be ignored (Fixes #634)

Author: linkdotnet

src/bunit.core/Rendering/TestRenderer.cs

@@ -82,12 +82,17 @@ public Task DispatchEventAsync(
 			}
 			catch (ArgumentException ex) when (string.Equals(ex.Message, $"There is no event handler associated with this event. EventId: '{eventHandlerId}'. (Parameter 'eventHandlerId')", StringComparison.Ordinal))
 			{
+				if (ignoreUnknownEventHandlers)
+				{
+					return Task.CompletedTask;
+				}
+
 				var betterExceptionMsg = new UnknownEventHandlerIdException(eventHandlerId, fieldInfo, ex);
 				return Task.FromException(betterExceptionMsg);
 			}
 		});
 
-		if (result.IsFaulted && result.Exception is not null && !(ignoreUnknownEventHandlers && result.Exception.InnerException is UnknownEventHandlerIdException))
+		if (result.IsFaulted && result.Exception is not null)
 		{
 			HandleException(result.Exception);
 		}

tests/bunit.web.tests/BlazorE2E/ComponentRenderingTest.cs

@@ -598,4 +598,15 @@ public void CanHandleRemovedParentObjects()
 		cut.WaitForState(() => !cut.FindAll("div").Any());
 		cut.FindAll("div").Count.ShouldBe(0);
 	}
+
+	[Fact]
+	public async Task CanHandleRemovedParentObjectsAsync()
+	{
+		var cut = RenderComponent<DispatcherException>();
+
+		await cut.Find("button").ClickAsync(new MouseEventArgs());
+
+		cut.WaitForState(() => !cut.FindAll("div").Any());
+		cut.FindAll("div").Count.ShouldBe(0);
+	}
 }