fix: move Find into WaitForAssertion to avoid test failing on multi core

egil · egil · commit b46fc0861c17 · 2022-02-15T09:34:19.000Z
diff --git a/tests/bunit.core.tests/Rendering/TestRendererTest.cs b/tests/bunit.core.tests/Rendering/TestRendererTest.cs
@@ -342,8 +342,7 @@ public void Test101()
 		var cut = RenderComponent<AsyncRenderOfSubComponentDuringInit>(parameters =>
 			parameters.Add(p => p.EitherOr, Task.Delay(1)));
 
-		var h1 = cut.Find("h1");
-		cut.WaitForAssertion(() => h1.TextContent.ShouldBe("SECOND"));
+		cut.WaitForAssertion(() => cut.Find("h1").TextContent.ShouldBe("SECOND"));
 	}
 
 	[Fact(DisplayName = "Can render component that awaits completed task in OnInitializedAsync")]

Original file line number	Diff line number	Diff line change
`@@ -342,8 +342,7 @@ public void Test101()`
`342`	`342`	`var cut = RenderComponent<AsyncRenderOfSubComponentDuringInit>(parameters =>`
`343`	`343`	`parameters.Add(p => p.EitherOr, Task.Delay(1)));`
`344`	`344`
`345`		`- var h1 = cut.Find("h1");`
`346`		`- cut.WaitForAssertion(() => h1.TextContent.ShouldBe("SECOND"));`
	`345`	`+ cut.WaitForAssertion(() => cut.Find("h1").TextContent.ShouldBe("SECOND"));`
`347`	`346`	`}`
`348`	`347`
`349`	`348`	`[Fact(DisplayName = "Can render component that awaits completed task in OnInitializedAsync")]`