Skip to content

Commit 1591855

Browse files
Avoid timing issues in CanFocusDuringOnAfterRenderAsyncWithFocusInEvent
1 parent 691b794 commit 1591855

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

src/Components/test/E2ETest/Tests/ComponentRenderingTestBase.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -485,7 +485,7 @@ public void CanFocusDuringOnAfterRenderAsyncWithFocusInEvent(string triggerButto
485485
Browser.Equal("focus-input-onafterrender", () => Browser.SwitchTo().ActiveElement().GetAttribute("id"));
486486

487487
// As well as actually focusing and triggering the onfocusin event, we should not be seeing any errors
488-
var log = Browser.Manage().Logs.GetLog(LogType.Browser);
488+
var log = Browser.Manage().Logs.GetLog(LogType.Browser).ToArray();
489489
Assert.DoesNotContain(log, entry => entry.Level == LogLevel.Severe);
490490
}
491491

src/Components/test/testassets/BasicTestApp/ElementFocusComponent.razor

+7-4
Original file line numberDiff line numberDiff line change
@@ -30,15 +30,18 @@
3030
{
3131
Task.Run(async () =>
3232
{
33-
await Task.Yield();
34-
performFocusDuringOnAfterRender = true;
35-
_ = InvokeAsync(StateHasChanged);
33+
await Task.Delay(500);
34+
_ = InvokeAsync(() =>
35+
{
36+
performFocusDuringOnAfterRender = true;
37+
StateHasChanged();
38+
});
3639
});
3740
}
3841

3942
private async Task FocusDuringOnAfterRenderViaAwait()
4043
{
41-
await Task.Yield();
44+
await Task.Delay(500);
4245
performFocusDuringOnAfterRender = true;
4346
}
4447

0 commit comments

Comments
 (0)