Add SetVoidResult to Focus handlers with tests (#524)

* add SetVoidResult to Focus hanlders with tests
* Update CHANGELOG.md

Co-authored-by: Egil Hansen <egil@assimilated.dk>

Author: anddrzejb

CHANGELOG.md

@@ -35,6 +35,8 @@ List of added functionality in this release.
 
 - The `Click` and `DoubleClick` extension methods now set the `MouseEventArgs.Detail` property to `1` and `2` respectively by default, unless the user specifies something else. This makes the methods more correctly emulate how Blazor reports single or double clicks on an element in the browser. Thanks to [@David-Moreira](https://github.com/David-Moreira) for the help troubleshooting this issue. By [@egil](https://github.com/egil).
 
+- `FocusAsync()` method handler on `ElementReference` and `<FocusOnNavigate>` js handler return completed `Task`. By [@anddrzejb](https://github.com/anddrzejb).
+
 ## [1.2.49] - 2021-08-09
 
 ### Added

src/bunit.web/JSInterop/InvocationHandlers/Implementation/FocusAsyncInvocationHandler.cs

@@ -1,3 +1,4 @@
+
 #if NET5_0_OR_GREATER
 using System;
 using Microsoft.AspNetCore.Components;
@@ -21,7 +22,9 @@ internal sealed class FocusAsyncInvocationHandler : JSRuntimeInvocationHandler
 		/// </summary>
 		internal FocusAsyncInvocationHandler()
 			: base(inv => inv.Identifier.Equals(FocusIdentifier, StringComparison.Ordinal), isCatchAllHandler: false)
-		{ }
+		{
+			SetVoidResult();
+		}
 	}
 }
 #endif

src/bunit.web/JSInterop/InvocationHandlers/Implementation/FocusOnNavigateHandler.cs

@@ -20,7 +20,9 @@ internal sealed class FocusOnNavigateHandler : JSRuntimeInvocationHandler
 		/// </summary>
 		internal FocusOnNavigateHandler()
 			: base(inv => inv.Identifier.Equals(Identifier, StringComparison.Ordinal), isCatchAllHandler: true)
-		{ }
+		{
+			SetVoidResult();
+		}
 	}
 }
 #endif

tests/bunit.web.tests/JSInterop/InvocationHandlers/FocusAsyncInvocationHandlerTest.cs

@@ -31,14 +31,23 @@ public void Test002()
 			invocations[1].Arguments[0].ShouldBeElementReferenceTo(inputs[1]);
 		}
 
+		[Fact(DisplayName = "Will return completed task")]
+		public void Test003()
+		{
+			var cut = RenderComponent<FocusingComponent>();
+			Assert.True(cut.Instance.AfterFirstRender);
+		}
+
 		private class FocusingComponent : ComponentBase
 		{
 			private ElementReference elmRef;
+			internal bool AfterFirstRender { get; private set; }
 			protected override async Task OnAfterRenderAsync(bool firstRender)
 			{
 				if (firstRender)
 				{
 					await elmRef.FocusAsync();
+					AfterFirstRender = true;
 				}
 			}
 

tests/bunit.web.tests/JSInterop/InvocationHandlers/FocusOnNavigateHandlerTest.cs

@@ -30,6 +30,31 @@ public void Test001()
 				.Arguments[0]
 				.ShouldBe(focusOnNavigateComponent.Instance.Selector);
 		}
+
+		[Fact(DisplayName = "Will return completed task")]
+		public void Test002()
+		{
+			var cut = RenderComponent<App>(ps => ps
+				.Add<FocusOnNavigateInternal, RouteData>(p => p.FoundTemplate, routeData => cps => cps
+					.Add(x => x.RouteData, routeData)
+					.Add(x => x.Selector, "h1")));
+
+			var focusOnNavigateComponent = cut.FindComponent<FocusOnNavigateInternal>();
+			Assert.True(focusOnNavigateComponent.Instance.AfterFirstRender);
+		}
+
+		private class FocusOnNavigateInternal : FocusOnNavigate
+		{
+			internal bool AfterFirstRender { get; private set; }
+			protected override async Task OnAfterRenderAsync(bool firstRender)
+			{
+				await base.OnAfterRenderAsync(firstRender);
+				if (firstRender)
+				{
+					AfterFirstRender = true;
+				}
+			}
+		}
 	}
 }
 #endif