Remove record command (#95)

rhel-lightspeed · Jan 6, 2025 · ce7b6ac · ce7b6ac
1 parent 0a4349d
commit ce7b6ac
Show file tree

Hide file tree

Showing 4 changed files with 1 addition and 195 deletions.
diff --git a/command_line_assistant/commands/record.py b/command_line_assistant/commands/record.py
diff --git a/command_line_assistant/initialize.py b/command_line_assistant/initialize.py
@@ -3,7 +3,7 @@
 import sys
 from argparse import ArgumentParser, Namespace
 
-from command_line_assistant.commands import history, query, record
+from command_line_assistant.commands import history, query
 from command_line_assistant.utils.cli import (
     add_default_command,
     create_argument_parser,
@@ -25,7 +25,6 @@ def register_subcommands() -> ArgumentParser:
     # and call `register_subcommand()` on each one.
     query.register_subcommand(commands_parser)  # type: ignore
     history.register_subcommand(commands_parser)  # type: ignore
-    record.register_subcommand(commands_parser)  # type: ignore
 
     return parser
 

diff --git a/tests/commands/test_record.py b/tests/commands/test_record.py
diff --git a/tests/test_initialize.py b/tests/test_initialize.py
@@ -43,7 +43,6 @@ def test_initialize_with_query_command(argv, stdin):
         patch("sys.argv", argv),
         patch("command_line_assistant.commands.query.register_subcommand"),
         patch("command_line_assistant.commands.history.register_subcommand"),
-        patch("command_line_assistant.commands.record.register_subcommand"),
         patch("command_line_assistant.initialize.read_stdin", lambda: stdin),
         patch("argparse.ArgumentParser.parse_args") as mock_parse,
     ):
@@ -62,26 +61,6 @@ def test_initialize_with_history_command():
         patch("sys.argv", ["c", "history", "--clear"]),
         patch("command_line_assistant.commands.query.register_subcommand"),
         patch("command_line_assistant.commands.history.register_subcommand"),
-        patch("command_line_assistant.commands.record.register_subcommand"),
-        patch("command_line_assistant.initialize.read_stdin", lambda: None),
-        patch("argparse.ArgumentParser.parse_args") as mock_parse,
-    ):
-        mock_parse.return_value.func = mock_command
-        result = initialize()
-
-        assert result == 0
-        mock_command.assert_called_once()
-
-
-def test_initialize_with_record_command():
-    """Test initialize with record command"""
-    mock_command = Mock(return_value=MockCommand())
-
-    with (
-        patch("sys.argv", ["c", "record"]),
-        patch("command_line_assistant.commands.query.register_subcommand"),
-        patch("command_line_assistant.commands.history.register_subcommand"),
-        patch("command_line_assistant.commands.record.register_subcommand"),
         patch("command_line_assistant.initialize.read_stdin", lambda: None),
         patch("argparse.ArgumentParser.parse_args") as mock_parse,
     ):
@@ -139,7 +118,6 @@ def test_initialize_bad_stdin(capsys):
         (["c"], "query"),  # Default to query
         (["c", "query"], "query"),
         (["c", "history"], "history"),
-        (["c", "record"], "record"),
     ],
 )
 def test_initialize_command_selection(argv, expected_command):
@@ -151,7 +129,6 @@ def test_initialize_command_selection(argv, expected_command):
         patch("command_line_assistant.initialize.read_stdin", lambda: None),
         patch("command_line_assistant.commands.query.register_subcommand"),
         patch("command_line_assistant.commands.history.register_subcommand"),
-        patch("command_line_assistant.commands.record.register_subcommand"),
         patch("argparse.ArgumentParser.parse_args") as mock_parse,
     ):
         mock_parse.return_value.func = mock_command