From 48186cd680a302d464629da27cdf1f60975c1ad5 Mon Sep 17 00:00:00 2001 From: Rushmore75 Date: Tue, 11 Nov 2025 14:34:19 -0700 Subject: [PATCH] fix tests --- src/app/screen.rs | 31 ++++++++++++++++++++++++++++--- 1 file changed, 28 insertions(+), 3 deletions(-) diff --git a/src/app/screen.rs b/src/app/screen.rs index 046bf2f..54623e6 100644 --- a/src/app/screen.rs +++ b/src/app/screen.rs @@ -127,17 +127,19 @@ fn scroll() { assert_eq!(app.screen.scroll.1, 0); // at the edge of visible, but shouldn't scroll yet - app.screen.scroll_based_on_cursor_location((18,0), &app.vars); + app.screen.scroll_based_on_cursor_location((16,0), &app.vars); assert_eq!(app.screen.scroll.0, 0); assert_eq!(app.screen.scroll.1, 0); // scroll 1 right - app.screen.scroll_based_on_cursor_location((19,0), &app.vars); + // Yes, you would think this shouldn't scroll yet `floor(181/10) = 18` + // but for some reason we are missing 2 cells in each direction. + app.screen.scroll_based_on_cursor_location((17,0), &app.vars); assert_eq!(app.screen.scroll.0, 1); assert_eq!(app.screen.scroll.1, 0); // cursor comes back, scroll should stay - app.screen.scroll_based_on_cursor_location((18,0), &app.vars); + app.screen.scroll_based_on_cursor_location((16,0), &app.vars); assert_eq!(app.screen.scroll.0, 1); assert_eq!(app.screen.scroll.1, 0); @@ -146,4 +148,27 @@ fn scroll() { assert_eq!(app.screen.scroll.0, 0); assert_eq!(app.screen.scroll.1, 0); + // now for y + + // we aren't scrolled at all yet + app.screen.scroll_based_on_cursor_location((0,0), &app.vars); + assert_eq!(app.screen.scroll.0, 0); + assert_eq!(app.screen.scroll.1, 0); + + app.screen.scroll_based_on_cursor_location((0,12), &app.vars); + assert_eq!(app.screen.scroll.0, 0); + assert_eq!(app.screen.scroll.1, 0); + + app.screen.scroll_based_on_cursor_location((0,13), &app.vars); + assert_eq!(app.screen.scroll.0, 0); + assert_eq!(app.screen.scroll.1, 1); + + app.screen.scroll_based_on_cursor_location((0,12), &app.vars); + assert_eq!(app.screen.scroll.0, 0); + assert_eq!(app.screen.scroll.1, 1); + + app.screen.scroll_based_on_cursor_location((0,0), &app.vars); + assert_eq!(app.screen.scroll.0, 0); + assert_eq!(app.screen.scroll.1, 0); + }