From d1300845cb73f9672b88b4986281655cf8c8071b Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sat, 20 Dec 2025 15:37:13 +0000 Subject: [PATCH] Add clarifying comments for loop iterations per code review Co-authored-by: tig <585482+tig@users.noreply.github.com> --- .../ViewBase/Navigation/AdornmentNavigationTests.cs | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Tests/UnitTestsParallelizable/ViewBase/Navigation/AdornmentNavigationTests.cs b/Tests/UnitTestsParallelizable/ViewBase/Navigation/AdornmentNavigationTests.cs index 79c070286..e4a68d193 100644 --- a/Tests/UnitTestsParallelizable/ViewBase/Navigation/AdornmentNavigationTests.cs +++ b/Tests/UnitTestsParallelizable/ViewBase/Navigation/AdornmentNavigationTests.cs @@ -502,7 +502,7 @@ public class AdornmentNavigationTests // Test: Navigate through all focusable elements List focusedViews = new (); - // Advance focus 4 times to cycle through all elements + // Advance focus 4 times to cycle through all 4 elements (marginButton, borderButton, paddingButton, contentButton) for (var i = 0; i < 4; i++) { view.AdvanceFocus (NavigationDirection.Forward, TabBehavior.TabStop); @@ -599,6 +599,7 @@ public class AdornmentNavigationTests // Track which views receive focus List focusedIds = new (); + // Navigate multiple times to test nested navigation (extra iteration to allow for wrapping) for (var i = 0; i < 5; i++) { if (parentPaddingButton.HasFocus)