From 37f1ca936bd4651ac040d4c929a4ee04f1325111 Mon Sep 17 00:00:00 2001 From: Levi Neuwirth Date: Tue, 28 Jul 2026 17:34:40 -0400 Subject: [PATCH] Keep the passive-panel fixture inside its buffer `clippy::format_collect` rejected the generated body; the replacement was shorter than the hard-coded caret offset, so the caret landed past the end and the focused half stopped exercising the clamp. Derive the offset from the body instead of restating it. Co-Authored-By: Claude Opus 5 (1M context) Claude-Session: https://claude.ai/code/session_01Lv428Fth9LRtffwJSsqH7T --- tests/bottom_panel_stage2b_daemon_acceptance.rs | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tests/bottom_panel_stage2b_daemon_acceptance.rs b/tests/bottom_panel_stage2b_daemon_acceptance.rs index f7f5ada..adf3774 100644 --- a/tests/bottom_panel_stage2b_daemon_acceptance.rs +++ b/tests/bottom_panel_stage2b_daemon_acceptance.rs @@ -907,14 +907,15 @@ fn a_passive_panel_keeps_its_view_top_while_a_focused_one_scrolls_to_its_caret() session.declare(1, ROWS, COLS); open_panel(&session, "*panel*", 4); let panel = session.side_window().expect("side window"); - let body: String = (0..40).map(|n| format!("line {n}\n")).collect(); + let body = vec!["line-of-text"; 40].join("\n"); + let last_byte = body.len() as u64; set_panel_text(&session, &body); // Put the panel's caret far below its viewport while it is PASSIVE. { let mut core = session.state.core.borrow_mut(); let window = core.windows.get_mut(&panel).expect("panel window"); - window.cursor = 200; + window.cursor = last_byte; window.view_top = 0; } let _ = session.present();