From a36f474554f6f1eb2197fe34a4a0a7de311cfb38 Mon Sep 17 00:00:00 2001 From: Fredrik Fornwall Date: Sun, 24 Nov 2019 19:07:34 +0100 Subject: [PATCH] Remove outdated tests Remove tests that asserted that Cursor Down (CUD) and Cursor Up (CUU) escape sequences were affected by the scrolling region set by DECSTBM. This was incorrect and recently fixed: https://github.com/termux/termux-app/issues/1340 --- .../test/java/com/termux/terminal/CursorAndScreenTest.java | 4 ---- 1 file changed, 4 deletions(-) diff --git a/terminal-emulator/src/test/java/com/termux/terminal/CursorAndScreenTest.java b/terminal-emulator/src/test/java/com/termux/terminal/CursorAndScreenTest.java index 12635425ad..121cb22318 100644 --- a/terminal-emulator/src/test/java/com/termux/terminal/CursorAndScreenTest.java +++ b/terminal-emulator/src/test/java/com/termux/terminal/CursorAndScreenTest.java @@ -133,8 +133,6 @@ public void testCursorUp() { withTerminalSized(3, 3).enterString("ABCDEFG\033[2AH").assertLinesAre("AHC", "DEF", "G "); // If an attempt is made to move the cursor above the top margin, the cursor stops at the top margin: withTerminalSized(3, 3).enterString("ABCDEFG\033[44AH").assertLinesAre("AHC", "DEF", "G "); - // Set top margin and validate that cursor does not go above it: - withTerminalSized(3, 3).enterString("\033[2rABCDEFG\033[44AH").assertLinesAre("ABC", "DHF", "G "); } public void testCursorDown() { @@ -143,8 +141,6 @@ public void testCursorDown() { withTerminalSized(3, 3).enterString("AB\033[2BC").assertLinesAre("AB ", " ", " C"); // If an attempt is made to move the cursor below the bottom margin, the cursor stops at the bottom margin: withTerminalSized(3, 3).enterString("AB\033[44BC").assertLinesAre("AB ", " ", " C"); - // Set bottom margin and validate that cursor does not go above it: - withTerminalSized(3, 3).enterString("\033[1;2rAB\033[44BC").assertLinesAre("AB ", " C", " "); } public void testReportCursorPosition() {