refactor: remove Ctrl+n/Ctrl+p tests per review feedback

Tests for this functionality are considered overkill.
Removed to keep CI times in check.

Author: pppp606

codex-rs/tui/src/bottom_pane/list_selection_view.rs

@@ -737,107 +737,4 @@ mod tests {
         );
     }
 
-    #[test]
-    fn ctrl_n_moves_selection_down() {
-        let mut view = make_selection_view(None);
-        // Initial selection is on first item (Read Only which is_current)
-        let initial = render_lines(&view);
-        assert!(
-            initial.contains("› 1. Read Only"),
-            "expected first item selected initially"
-        );
-
-        // Press Ctrl+n to move down
-        view.handle_key_event(KeyEvent::new(
-            KeyCode::Char('n'),
-            KeyModifiers::CONTROL,
-        ));
-        let after_ctrl_n = render_lines(&view);
-        assert!(
-            after_ctrl_n.contains("› 2. Full Access"),
-            "expected second item selected after Ctrl+n"
-        );
-    }
-
-    #[test]
-    fn ctrl_p_moves_selection_up() {
-        let mut view = make_selection_view(None);
-        // Move down first so we can move up
-        view.handle_key_event(KeyEvent::new(KeyCode::Down, KeyModifiers::NONE));
-        let after_down = render_lines(&view);
-        assert!(
-            after_down.contains("› 2. Full Access"),
-            "expected second item selected after Down"
-        );
-
-        // Press Ctrl+p to move up
-        view.handle_key_event(KeyEvent::new(
-            KeyCode::Char('p'),
-            KeyModifiers::CONTROL,
-        ));
-        let after_ctrl_p = render_lines(&view);
-        assert!(
-            after_ctrl_p.contains("› 1. Read Only"),
-            "expected first item selected after Ctrl+p"
-        );
-    }
-
-    #[test]
-    fn ctrl_n_and_ctrl_p_wrap_around() {
-        let mut view = make_selection_view(None);
-        // Move to second item
-        view.handle_key_event(KeyEvent::new(
-            KeyCode::Char('n'),
-            KeyModifiers::CONTROL,
-        ));
-        // Move past last item - should wrap to first
-        view.handle_key_event(KeyEvent::new(
-            KeyCode::Char('n'),
-            KeyModifiers::CONTROL,
-        ));
-        let wrapped_forward = render_lines(&view);
-        assert!(
-            wrapped_forward.contains("› 1. Read Only"),
-            "expected selection to wrap to first item"
-        );
-
-        // Move up from first item - should wrap to last
-        view.handle_key_event(KeyEvent::new(
-            KeyCode::Char('p'),
-            KeyModifiers::CONTROL,
-        ));
-        let wrapped_back = render_lines(&view);
-        assert!(
-            wrapped_back.contains("› 2. Full Access"),
-            "expected selection to wrap to last item"
-        );
-    }
-
-    #[test]
-    fn c0_control_chars_navigate_selection() {
-        let mut view = make_selection_view(None);
-        // Initial selection is on first item
-        let initial = render_lines(&view);
-        assert!(
-            initial.contains("› 1. Read Only"),
-            "expected first item selected initially"
-        );
-
-        // Simulate terminals that send C0 control chars without CONTROL modifier.
-        // ^N (U+000E) should move down
-        view.handle_key_event(KeyEvent::new(KeyCode::Char('\u{000e}'), KeyModifiers::NONE));
-        let after_c0_n = render_lines(&view);
-        assert!(
-            after_c0_n.contains("› 2. Full Access"),
-            "expected second item selected after ^N (C0)"
-        );
-
-        // ^P (U+0010) should move up
-        view.handle_key_event(KeyEvent::new(KeyCode::Char('\u{0010}'), KeyModifiers::NONE));
-        let after_c0_p = render_lines(&view);
-        assert!(
-            after_c0_p.contains("› 1. Read Only"),
-            "expected first item selected after ^P (C0)"
-        );
-    }
 }