mirror of
https://github.com/pchuan98/codex.git
synced 2026-07-01 00:31:56 +08:00
pakrym-oai
e752f7b4ae
The workspace denies `clippy::expect_used` in production. Although `clippy.toml` allows `expect` in tests, Bazel Clippy compiles integration-test helper code in a way that does not receive that exemption, which encouraged verbose `unwrap_or_else(... panic!(...))` and equivalent `match`/`let else` forms. This allows `clippy::expect_used` once at each integration-test crate root (including aggregated suites and test-support libraries), then replaces manual panic-based Result and Option unwraps with `expect`/`expect_err`. Standalone `tests/*.rs` files remain their own crate roots. Intentional assertion and unexpected-variant panics remain unchanged, and the production `expect_used = "deny"` lint remains in place. The cleanup is mechanical and net-negative in line count.
44 lines
1.5 KiB
Rust
44 lines
1.5 KiB
Rust
use crate::test_backend::VT100Backend;
|
|
use ratatui::layout::Rect;
|
|
use ratatui::text::Line;
|
|
|
|
#[test]
|
|
fn live_001_commit_on_overflow() {
|
|
let backend = VT100Backend::new(/*width*/ 20, /*height*/ 6);
|
|
let mut term =
|
|
codex_tui::Terminal::with_options(backend).expect("terminal should be constructed");
|
|
let area = Rect::new(
|
|
/*x*/ 0, /*y*/ 5, /*width*/ 20, /*height*/ 1,
|
|
);
|
|
term.set_viewport_area(area);
|
|
|
|
// Build 5 explicit rows at width 20.
|
|
let mut rb = codex_tui::RowBuilder::new(/*target_width*/ 20);
|
|
rb.push_fragment("one\n");
|
|
rb.push_fragment("two\n");
|
|
rb.push_fragment("three\n");
|
|
rb.push_fragment("four\n");
|
|
rb.push_fragment("five\n");
|
|
|
|
// Keep the last 3 in the live ring; commit the first 2.
|
|
let commit_rows = rb.drain_commit_ready(/*max_keep*/ 3);
|
|
let lines: Vec<Line<'static>> = commit_rows.into_iter().map(|r| r.text.into()).collect();
|
|
|
|
codex_tui::insert_history_lines(&mut term, lines)
|
|
.expect("Failed to insert history lines in test");
|
|
|
|
let screen = term.backend().vt100().screen();
|
|
|
|
// The words "one" and "two" should appear above the viewport.
|
|
let joined = screen.contents();
|
|
assert!(
|
|
joined.contains("one"),
|
|
"expected committed 'one' to be visible\n{joined}"
|
|
);
|
|
assert!(
|
|
joined.contains("two"),
|
|
"expected committed 'two' to be visible\n{joined}"
|
|
);
|
|
// The last three (three,four,five) remain in the live ring, not committed here.
|
|
}
|