From db4f8a4d6da05b548a1ed7447d8f42e1342c98b5 Mon Sep 17 00:00:00 2001
From: tareq <tareq.halaby@gmail.com>
Date: Fri, 30 Oct 2026 13:21:01 +0100
Subject: [PATCH] Toolbar: assert Alt-click wrap-around in the flyout cycle
 test.

CodeRabbit nitpick: after the last group tool, the next Alt-click must return to the first and keep the flyout closed.
---
 crates/ui-egui/src/toolbar/tests.rs | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/crates/ui-egui/src/toolbar/tests.rs b/crates/ui-egui/src/toolbar/tests.rs
index 97a3bf6c3..5db130f63 100644
--- a/crates/ui-egui/src/toolbar/tests.rs
+++ b/crates/ui-egui/src/toolbar/tests.rs
@@ -406,4 +406,7 @@ fn alt_click_cycles_the_tool_flyout_group() {
     assert!(h.state().app.ui.flyout.is_none(), "Alt-click must not open the flyout");
     click(&mut h, alt);
     assert_eq!(h.state().app.session.tool_id(), "polygonFrame");
+    click(&mut h, alt);
+    assert_eq!(h.state().app.session.tool_id(), "rectangleFrame", "Alt-click after the last group tool wraps to the first");
+    assert!(h.state().app.ui.flyout.is_none(), "wrap-around Alt-click must not open the flyout");
 }