Renamed functions in cmd2.ansi.Cursor to be pep8 compliant

Author: tleonhardt

cmd2/ansi.py

@@ -207,33 +207,33 @@ class BgColor(AnsiSequence):
 
 
 ####################################################################################
-# Implementations intended for direct use
+# Implementations intended for direct use (do NOT use outside of cmd2)
 ####################################################################################
 class Cursor:
     """Create ANSI sequences to alter the cursor position."""
 
     @staticmethod
-    def UP(count: int = 1) -> str:
+    def up(count: int = 1) -> str:
         """Move the cursor up a specified amount of lines (Defaults to 1)."""
         return f"{CSI}{count}A"
 
     @staticmethod
-    def DOWN(count: int = 1) -> str:
+    def down(count: int = 1) -> str:
         """Move the cursor down a specified amount of lines (Defaults to 1)."""
         return f"{CSI}{count}B"
 
     @staticmethod
-    def FORWARD(count: int = 1) -> str:
+    def forward(count: int = 1) -> str:
         """Move the cursor forward a specified amount of lines (Defaults to 1)."""
         return f"{CSI}{count}C"
 
     @staticmethod
-    def BACK(count: int = 1) -> str:
+    def back(count: int = 1) -> str:
         """Move the cursor back a specified amount of lines (Defaults to 1)."""
         return f"{CSI}{count}D"
 
     @staticmethod
-    def SET_POS(x: int, y: int) -> str:
+    def set_pos(x: int, y: int) -> str:
         """Set the cursor position to coordinates which are 1-based."""
         return f"{CSI}{y};{x}H"
 
@@ -1059,11 +1059,11 @@ def async_alert_str(*, terminal_columns: int, prompt: str, line: str, cursor_off
 
     # Move the cursor down to the last input line
     if cursor_input_line != num_input_terminal_lines:
-        terminal_str += Cursor.DOWN(num_input_terminal_lines - cursor_input_line)
+        terminal_str += Cursor.down(num_input_terminal_lines - cursor_input_line)
 
     # Clear each line from the bottom up so that the cursor ends up on the first prompt line
     total_lines = num_prompt_terminal_lines + num_input_terminal_lines
-    terminal_str += (clear_line() + Cursor.UP(1)) * (total_lines - 1)
+    terminal_str += (clear_line() + Cursor.up(1)) * (total_lines - 1)
 
     # Clear the first prompt line
     terminal_str += clear_line()

pyproject.toml

@@ -260,10 +260,6 @@ per-file-ignores."cmd2/argparse_custom.py" = [
     "UP031", # Use format specifiers instead of percent format (auto fix is unsafe)
 ]
 
-per-file-ignores."cmd2/ansi.py" = [
-    "N802", # Function names should be lowercase for things like Curor.UP
-]
-
 per-file-ignores."examples/*.py" = [
     "ANN",     # Ignore all type annotation rules in examples folder
     "D",       # Ignore all pydocstyle rules in examples folder

tests/test_ansi.py

@@ -202,14 +202,14 @@ def test_clear_line() -> None:
 
 def test_cursor() -> None:
     count = 1
-    assert ansi.Cursor.UP(count) == f"{ansi.CSI}{count}A"
-    assert ansi.Cursor.DOWN(count) == f"{ansi.CSI}{count}B"
-    assert ansi.Cursor.FORWARD(count) == f"{ansi.CSI}{count}C"
-    assert ansi.Cursor.BACK(count) == f"{ansi.CSI}{count}D"
+    assert ansi.Cursor.up(count) == f"{ansi.CSI}{count}A"
+    assert ansi.Cursor.down(count) == f"{ansi.CSI}{count}B"
+    assert ansi.Cursor.forward(count) == f"{ansi.CSI}{count}C"
+    assert ansi.Cursor.back(count) == f"{ansi.CSI}{count}D"
 
     x = 4
     y = 5
-    assert ansi.Cursor.SET_POS(x, y) == f"{ansi.CSI}{y};{x}H"
+    assert ansi.Cursor.set_pos(x, y) == f"{ansi.CSI}{y};{x}H"
 
 
 @pytest.mark.parametrize(