add test for overriding a django param default

django_typer/tests/test_app/management/commands/dj_params4.py

@@ -1,5 +1,6 @@
 from django.core.management import CommandError
-
+from typing import Annotated
+from typer import Option
 from django_typer import TyperCommand, callback, command, types
 from django_typer.tests.utils import log_django_parameters
 
@@ -8,8 +9,19 @@ class Command(TyperCommand):
     help = "Test that django parameters work as expected"
 
     @command(epilog="epilog")
-    def handle(self, throw: bool = False, verbosity: types.Verbosity = 1):
+    def handle(
+        self,
+        throw: bool = False,
+        verbosity: types.Verbosity = 1,
+        traceback: Annotated[
+            bool,
+            Option(
+                help=("Raise on CommandError exceptions"),
+                rich_help_panel=types.COMMON_PANEL,
+            ),
+        ] = True  # this should change the default!
+    ):
         assert self.__class__ == Command
-        log_django_parameters(self, verbosity=verbosity)
+        log_django_parameters(self, verbosity=verbosity, traceback=traceback)
         if throw:
             raise CommandError("Test Exception")

django_typer/tests/tests.py

@@ -530,9 +530,16 @@ def test_skip_checks(self):
     def test_traceback(self):
         for cmd, args in self.commands:
             result = run_command(cmd, *args, "--throw")
-            self.assertFalse("Traceback" in result)
-            result_tb = run_command(cmd, "--traceback", *args, "--throw")
-            self.assertTrue("Traceback" in result_tb)
+            if cmd != 'dj_params4':
+                self.assertFalse("Traceback" in result)
+            else:
+                self.assertTrue("Traceback" in result)
+            if cmd != 'dj_params4':
+                result_tb = run_command(cmd, "--traceback", *args, "--throw")
+                self.assertTrue("Traceback" in result_tb)
+            else:
+                result_tb = run_command(cmd, "--no-traceback", *args, "--throw")
+                self.assertFalse("Traceback" in result_tb)
 
     def test_verbosity(self):
         run_command("dj_params3", "cmd1")

django_typer/types.py

@@ -8,7 +8,7 @@
 
 from typer import Option
 
-_COMMON_PANEL = "Django"
+COMMON_PANEL = "Django"
 
 
 def print_version(context, _, value):
@@ -28,7 +28,7 @@ def print_version(context, _, value):
         help="Show program's version number and exit.",
         callback=print_version,
         is_eager=True,
-        rich_help_panel=_COMMON_PANEL,
+        rich_help_panel=COMMON_PANEL,
     ),
 ]
 
@@ -42,7 +42,7 @@ def print_version(context, _, value):
         show_choices=True,
         min=0,
         max=3,
-        rich_help_panel=_COMMON_PANEL,
+        rich_help_panel=COMMON_PANEL,
     ),
 ]
 
@@ -54,7 +54,7 @@ def print_version(context, _, value):
             '"myproject.settings.main". If this isn\'t provided, the '
             "DJANGO_SETTINGS_MODULE environment variable will be used."
         ),
-        rich_help_panel=_COMMON_PANEL,
+        rich_help_panel=COMMON_PANEL,
     ),
 ]
 
@@ -65,7 +65,7 @@ def print_version(context, _, value):
             "A directory to add to the Python path, e.g. "
             '"/home/djangoprojects/myproject".'
         ),
-        rich_help_panel=_COMMON_PANEL,
+        rich_help_panel=COMMON_PANEL,
     ),
 ]
 
@@ -74,7 +74,7 @@ def print_version(context, _, value):
     Option(
         "--traceback",
         help=("Raise on CommandError exceptions"),
-        rich_help_panel=_COMMON_PANEL,
+        rich_help_panel=COMMON_PANEL,
     ),
 ]
 
@@ -83,7 +83,7 @@ def print_version(context, _, value):
     Option(
         "--no-color",
         help=("Don't colorize the command output."),
-        rich_help_panel=_COMMON_PANEL,
+        rich_help_panel=COMMON_PANEL,
     ),
 ]
 
@@ -92,13 +92,13 @@ def print_version(context, _, value):
     Option(
         "--force-color",
         help=("Force colorization of the command output."),
-        rich_help_panel=_COMMON_PANEL,
+        rich_help_panel=COMMON_PANEL,
     ),
 ]
 
 SkipChecks = Annotated[
     bool,
     Option(
-        "--skip-checks", help=("Skip system checks."), rich_help_panel=_COMMON_PANEL
+        "--skip-checks", help=("Skip system checks."), rich_help_panel=COMMON_PANEL
     ),
 ]