From 677d75e173e44c38e4d4e02d160d15afd5b81928 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 12635425..121cb223 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 class CursorAndScreenTest extends TerminalTestCase { 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 class CursorAndScreenTest extends TerminalTestCase { 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() {