From 7d9329c95275263e1073064277c35360494ba6b3 Mon Sep 17 00:00:00 2001 From: Piotr Kaznowski Date: Fri, 29 Jan 2021 16:50:38 +0100 Subject: [PATCH] #5 renames pa schedule delete 'nuke' to less epic but more informative 'all' command. by: Piotr --- cli/schedule.py | 2 +- tests/test_cli_schedule.py | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/cli/schedule.py b/cli/schedule.py index 76fec0c..173d4e2 100644 --- a/cli/schedule.py +++ b/cli/schedule.py @@ -70,7 +70,7 @@ def set( ) -@delete_app.command("nuke", help="Delete all scheduled tasks.") +@delete_app.command("all", help="Delete all scheduled tasks.") def delete_all_tasks( force: bool = typer.Option( False, "-f", "--force", help="Turns off user confirmation before deleting tasks" diff --git a/tests/test_cli_schedule.py b/tests/test_cli_schedule.py index 80df92f..944446b 100644 --- a/tests/test_cli_schedule.py +++ b/tests/test_cli_schedule.py @@ -63,7 +63,7 @@ class TestDeleteAllTasks: def test_deletes_all_tasks_with_user_permission(self, task_list, mock_confirm): mock_confirm.return_value = True - runner.invoke(delete_app, ["nuke"]) + runner.invoke(delete_app, ["all"]) assert mock_confirm.call_args == call( "This will irrevocably delete all your tasks, proceed?" @@ -75,12 +75,12 @@ def test_deletes_all_tasks_with_user_permission(self, task_list, mock_confirm): def test_exits_when_user_changes_mind(self, task_list, mock_confirm): mock_confirm.return_value = False - runner.invoke(delete_app, ["nuke"]) + runner.invoke(delete_app, ["all"]) assert task_list.call_count == 0 def test_deletes_all_tasks_when_forced(self, task_list, mock_confirm): - runner.invoke(delete_app, ["nuke", "--force"]) + runner.invoke(delete_app, ["all", "--force"]) assert mock_confirm.call_count == 0 assert task_list.call_count == 1 @@ -90,7 +90,7 @@ def test_deletes_all_tasks_when_forced(self, task_list, mock_confirm): def test_sets_logging_to_info(self, mocker): mock_logger = mocker.patch("cli.schedule.get_logger") - runner.invoke(delete_app, ["nuke"]) + runner.invoke(delete_app, ["all"]) assert mock_logger.call_args == call(set_info=True)