Use FindElement in polling loop and add explicit width-change assertion

Oxymoron290 · Copilot · Oxymoron290 · commit 5ca1f7f07fbc · 2026-03-12T11:04:39.000-05:00
Replace App.WaitForElement with App.FindElement inside the polling loop
to avoid compounding timeouts. Add an explicit assertion after the loop
to report a clear failure if the width never changed, rather than letting
it surface as a confusing downstream assertion.

Co-authored-by: Copilot &lt;223556219+Copilot@users.noreply.github.com&gt;
diff --git a/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue31109.cs b/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue31109.cs
@@ -40,14 +40,20 @@ public void DynamicWidthRequestUpdatesFlexLayoutItems()
 
 			while (stopwatch.Elapsed < timeout)
 			{
-				var currentRect = App.WaitForElement("Item1").GetRect();
+				var currentRect = App.FindElement("Item1").GetRect();
 				if (System.Math.Abs(currentRect.Width - item1Before.Width) > 1)
 				{
 					break;
 				}
 
 				System.Threading.Thread.Sleep(pollDelay);
 			}
+
+			// Verify the width actually changed; fail here rather than in downstream assertions
+			var item1Check = App.FindElement("Item1").GetRect();
+			Assert.That(item1Check.Width, Is.Not.EqualTo(item1Before.Width).Within(1),
+				"Item1 width should have changed within timeout after tapping ChangeWidths");
+
 			// After changing widths, all items should have equal width
 			var item1After = App.WaitForElement("Item1").GetRect();
 			var item2After = App.WaitForElement("Item2").GetRect();

Original file line number	Diff line number	Diff line change
`@@ -40,14 +40,20 @@ public void DynamicWidthRequestUpdatesFlexLayoutItems()`
`40`	`40`
`41`	`41`	`while (stopwatch.Elapsed < timeout)`
`42`	`42`	`{`
`43`		`- var currentRect = App.WaitForElement("Item1").GetRect();`
	`43`	`+ var currentRect = App.FindElement("Item1").GetRect();`
`44`	`44`	`if (System.Math.Abs(currentRect.Width - item1Before.Width) > 1)`
`45`	`45`	`{`
`46`	`46`	`break;`
`47`	`47`	`}`
`48`	`48`
`49`	`49`	`System.Threading.Thread.Sleep(pollDelay);`
`50`	`50`	`}`
	`51`	`+`
	`52`	`+ // Verify the width actually changed; fail here rather than in downstream assertions`
	`53`	`+ var item1Check = App.FindElement("Item1").GetRect();`
	`54`	`+ Assert.That(item1Check.Width, Is.Not.EqualTo(item1Before.Width).Within(1),`
	`55`	`+ "Item1 width should have changed within timeout after tapping ChangeWidths");`
	`56`	`+`
`51`	`57`	`// After changing widths, all items should have equal width`
`52`	`58`	`var item1After = App.WaitForElement("Item1").GetRect();`
`53`	`59`	`var item2After = App.WaitForElement("Item2").GetRect();`