Makes all subcommands to print help when no arg is provided.

Co-authored-by: Piotr Kaznowski <[email protected]>

Author: filiplajszczak

cli/django.py

@@ -6,7 +6,7 @@
 from pythonanywhere.snakesay import snakesay
 from pythonanywhere.utils import ensure_domain
 
-app = typer.Typer()
+app = typer.Typer(no_args_is_help=True)
 
 
 @app.command()

cli/path.py

@@ -9,7 +9,7 @@
 from pythonanywhere.files import PAPath
 from pythonanywhere.scripts_commons import get_logger
 
-app = typer.Typer()
+app = typer.Typer(no_args_is_help=True)
 
 
 def setup(path: str, quiet: bool) -> PAPath:

cli/schedule.py

@@ -10,7 +10,7 @@
 from pythonanywhere.snakesay import snakesay
 from pythonanywhere.task import Task, TaskList
 
-app = typer.Typer()
+app = typer.Typer(no_args_is_help=True)
 
 
 @app.command()

cli/students.py

@@ -5,7 +5,7 @@
 from pythonanywhere.scripts_commons import get_logger
 from pythonanywhere.students import Students
 
-app = typer.Typer()
+app = typer.Typer(no_args_is_help=True)
 
 
 def setup(quiet: bool) -> Students:

cli/webapp.py

@@ -10,7 +10,7 @@
 from pythonanywhere.snakesay import snakesay
 from pythonanywhere.utils import ensure_domain
 
-app = typer.Typer()
+app = typer.Typer(no_args_is_help=True)
 
 
 @app.command()

tests/test_cli_django.py

@@ -33,6 +33,15 @@ def running_python_version():
     return ".".join(python_version().split(".")[:2])
 
 
+def test_main_subcommand_without_args_prints_help():
+    result = runner.invoke(
+        app,
+        [],
+    )
+    assert result.exit_code == 0
+    assert "Show this message and exit." in result.stdout
+
+
 def test_autoconfigure_calls_all_stuff_in_right_order(mock_django_project):
     result = runner.invoke(
         app,

tests/test_cli_pa.py

@@ -0,0 +1,21 @@
+from typer.testing import CliRunner
+
+from cli.pa import app
+
+runner = CliRunner()
+
+
+def test_main_command_without_args_prints_help():
+    result = runner.invoke(
+        app,
+        [],
+    )
+    assert result.exit_code == 0
+    assert "This is a new experimental PythonAnywhere cli client." in result.stdout
+    assert "Makes Django Girls tutorial projects deployment easy" in result.stdout
+    assert "Perform some operations on files" in result.stdout
+    assert "Manage scheduled tasks" in result.stdout
+    assert "Perform some operations on students" in result.stdout
+    assert "Everything for web apps" in result.stdout
+
+

tests/test_cli_path.py

@@ -41,6 +41,15 @@ def mock_file_path(mock_path):
     return mock_path
 
 
+def test_main_subcommand_without_args_prints_help():
+    result = runner.invoke(
+        app,
+        [],
+    )
+    assert result.exit_code == 0
+    assert "Show this message and exit." in result.stdout
+
+
 class TestGet:
     def test_exits_early_when_no_contents_for_given_path(self, mock_path):
         mock_path.return_value.contents = None

tests/test_cli_schedule.py

@@ -40,6 +40,15 @@ def mock_confirm(mocker):
     return mocker.patch("cli.schedule.typer.confirm")
 
 
+def test_main_subcommand_without_args_prints_help():
+    result = runner.invoke(
+        app,
+        [],
+    )
+    assert result.exit_code == 0
+    assert "Show this message and exit." in result.stdout
+
+
 class TestSet:
     def test_calls_all_stuff_in_right_order(self, mocker):
         mock_logger = mocker.patch("cli.schedule.get_logger")

tests/test_cli_students.py

@@ -22,6 +22,15 @@ def mock_students_delete(mock_students):
     return mock_students.return_value.delete
 
 
+def test_main_subcommand_without_args_prints_help():
+    result = runner.invoke(
+        app,
+        [],
+    )
+    assert result.exit_code == 0
+    assert "Show this message and exit." in result.stdout
+
+
 @pytest.mark.students
 class TestGet:
     def test_exits_early_with_error_when_api_does_not_return_expected_list(