fix tests for coverage

Author: bckohan

django_typer/tests/apps/test_app/management/commands/completion.py

@@ -81,6 +81,18 @@ def handle(
                 ),
             ),
         ] = [],
+        command_first: Annotated[
+            t.List[str],
+            typer.Option(
+                "--cmd-first",
+                help=_("A list of commands by import path or name."),
+                shell_complete=completers.chain(
+                    completers.complete_import_path,
+                    completers.commands(allow_duplicates=True),
+                    first_match=True,
+                ),
+            ),
+        ] = [],
     ):
         assert self.__class__ is Command
         for app in django_apps:

django_typer/tests/test_parser_completers.py

@@ -107,6 +107,21 @@ def test_app_label_parser_idempotency(self):
     def test_app_label_parser_completers(self):
         from django.apps import apps
 
+        result = StringIO()
+        with contextlib.redirect_stdout(result):
+            call_command("shellcompletion", "complete", "completion ")
+        result = result.getvalue()
+        self.assertTrue("test_app" in result)
+        self.assertTrue("django_typer_tests_apps_util" in result)
+        self.assertTrue("django_typer" in result)
+        self.assertTrue("django_typer_tests_apps_examples_polls" in result)
+        self.assertTrue("admin" in result)
+        self.assertTrue("auth" in result)
+        self.assertTrue("contenttypes" in result)
+        self.assertTrue("sessions" in result)
+        self.assertTrue("messages" in result)
+        self.assertTrue("staticfiles" in result)
+
         result = StringIO()
         with contextlib.redirect_stdout(result):
             call_command(
@@ -1353,3 +1368,15 @@ def test_chain_and_commands_completer(self):
         self.assertTrue("dj_params2" in result)
         self.assertTrue("dj_params3" in result)
         self.assertTrue("dj_params4" in result)
+
+        result = run_command(
+            "shellcompletion", "complete", "completion --cmd-first dj"
+        )[0].strip()
+
+        self.assertTrue("django" in result)
+        self.assertTrue("django_typer" in result)
+
+        self.assertFalse("dj_params1" in result)
+        self.assertFalse("dj_params2" in result)
+        self.assertFalse("dj_params3" in result)
+        self.assertFalse("dj_params4" in result)

doc/source/architecture.rst

@@ -27,8 +27,8 @@ following:
   commands they plugin to directly.
 * The group/command tree on instantiated commands must be walkable using attributes from the
   command instance itself to support subgroup name overloads.
-* Common django options should appear on a common initializer for compound commands with multiple
-  groups or commands and should appear directly on the command for non-compound commands.
+* Common django options should appear on the initializer for compound commands and should be
+  directly on the command for non-compound commands.
 
 During all of this, the correct self must be passed if the function accepts it, but all of the
 registered functions are not registered as methods because they enter the Typer_ app tree as