do not attempt to parse named parameters to call_command

Author: bckohan

django_typer/__init__.py

@@ -364,7 +364,6 @@ class _DjangoAdapterMixin(with_typehint(CoreTyperGroup)):  # type: ignore[misc]
     context_class: t.Type[click.Context] = Context
     django_command: "TyperCommand"
     callback_is_method: bool = True
-    param_map: t.Dict[str, click.Parameter] = {}
 
     class Converter:
         """
@@ -417,16 +416,14 @@ def call_with_self(*args, **kwargs):
             return callback(
                 *args,
                 **{
-                    # process supplied parameters incase they need type conversion
-                    param: (
-                        (
-                            self.param_map[param].process_value(ctx, val)
-                            if param in self.param_map
-                            else val
-                        )
-                        if param in ctx.supplied_params
-                        else val
-                    )
+                    # we could call param.process_value() here to allow named
+                    # parameters to be passed as their unparsed string values,
+                    # we don't because this forces some weird idempotency on custom
+                    # parsers that might make errors more frequent for users and also
+                    # this would be inconsistent with call_command behavior for BaseCommands
+                    # which expect the parsed values to be passed by name. Unparsed values can
+                    # always be passed as argument strings.
+                    param: val
                     for param, val in kwargs.items()
                     if param in expected
                 },
@@ -450,7 +447,6 @@ def call_with_self(*args, **kwargs):
             callback=call_with_self,
             **kwargs,
         )
-        self.param_map = {param.name: param for param in self.params}
 
 
 class TyperCommandWrapper(_DjangoAdapterMixin, CoreTyperCommand):

django_typer/tests/tests.py

@@ -1241,10 +1241,6 @@ def test_command_line(self, settings=None):
                 call_command("groups", "echo", "hey!", echoes=5).strip(),
                 ("hey! " * 5).strip(),
             )
-            self.assertEqual(
-                call_command("groups", "echo", "hey!", echoes="5").strip(),
-                ("hey! " * 5).strip(),
-            )
         else:
             self.assertIn("UsageError", result[1])
             with self.assertRaises(TypeError):
@@ -1281,7 +1277,7 @@ def test_command_line(self, settings=None):
 
         self.assertEqual(
             call_command(
-                "groups", "math", "multiply", "1.2", "3.5", " -12.3", precision="5"
+                "groups", "math", "--precision=5", "multiply", "1.2", "3.5", " -12.3"
             ),
             "-51.66000",
         )
@@ -1884,20 +1880,6 @@ def test_app_label_parser_completers(self):
             },
         )
 
-        self.assertEqual(
-            json.loads(
-                call_command(
-                    "completion",
-                    "django_typer_tests_polls",
-                    option="django_typer_tests_test_app",
-                )
-            ),
-            {
-                "django_apps": ["django_typer_tests_polls"],
-                "option": "django_typer_tests_test_app",
-            },
-        )
-
         self.assertEqual(
             json.loads(
                 call_command(