fix(tests): visit `flow_confirm_summary` menus
What the developer wrote
Authored by Ioan Bizău
fix(tests): visit `flow_confirm_summary` menus
[no changelog]
What changed, and why it matters
This commit is a test-only UI fix that enables the on-screen menu during automated test runs for two wallet interface layouts. It does not change how real transactions are signed or approved; it only makes sure automated tests can navigate the confirmation screens properly. There is no indication this fixes a security vulnerability.
No security action required. Treat as a normal test-maintenance commit.
Security signals we found
Commit title and message explicitly describe the change as a test fix
No changes to signing, validation, or authorization code
Only UI flow configuration for test coverage is touched
No changelog entry requested by the author
Evidence from the diff
The patch adds .with_flow_menu(true) / .with_flow_menu() calls in the new_confirm_summary flows for the Delizia and Eckhart UI layouts. These calls expose the flow’s menu button so automated tests can visit it. The change is gated behind test infrastructure and the commit title/message explicitly label it as a test fix with ‘[no changelog]’. No cryptographic, authorization, or transaction-approval logic is modified.
Changed components
core/embed/rust/src/ui/layout_delizia/flow/confirm_summary.rscore/embed/rust/src/ui/layout_eckhart/flow/confirm_summary.rsInspect captured patch +3 / −0
diff --git a/core/embed/rust/src/ui/layout_delizia/flow/confirm_summary.rs b/core/embed/rust/src/ui/layout_delizia/flow/confirm_summary.rs
index b828259b..98afafd4 100644
--- a/core/embed/rust/src/ui/layout_delizia/flow/confirm_summary.rs
+++ b/core/embed/rust/src/ui/layout_delizia/flow/confirm_summary.rs
@@ -74,6 +74,7 @@ pub fn new_confirm_summary(
) -> Result<SwipeFlow, error::Error> {
// Summary
let content_summary = summary_params
+ .with_flow_menu(true)
.into_layout()?
// Summary(1) + Hold(1)
.with_pages(|summary_pages| summary_pages + 1);
@@ -83,6 +84,7 @@ pub fn new_confirm_summary(
TR::send__sign_transaction.into(),
SwipeContent::new(PromptScreen::new_hold_to_confirm()),
)
+ .with_flow_menu()
.with_menu_button()
.with_footer(TR::instructions__hold_to_sign.into(), None)
.with_swipe(Direction::Down, SwipeSettings::Default)
diff --git a/core/embed/rust/src/ui/layout_eckhart/flow/confirm_summary.rs b/core/embed/rust/src/ui/layout_eckhart/flow/confirm_summary.rs
index 0c1647e0..30680a3a 100644
--- a/core/embed/rust/src/ui/layout_eckhart/flow/confirm_summary.rs
+++ b/core/embed/rust/src/ui/layout_eckhart/flow/confirm_summary.rs
@@ -123,6 +123,7 @@ pub fn new_confirm_summary(
.with_placement(LinearPlacement::vertical()),
)
.with_header(Header::new(title).with_menu_button())
+ .with_flow_menu()
.with_action_bar(if back_button {
ActionBar::new_double(Button::with_icon(theme::ICON_CHEVRON_UP), confirm_button)
} else {
Why this scored 17/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.