remove redundant TYPE_MAP properties

Author: adamchainz

src/django_mysql/models/fields/dynamic.py

@@ -179,13 +179,18 @@ def _check_spec_recursively(
                 subpath = f"{path}.{key}"
                 errors.extend(self._check_spec_recursively(value, subpath))
             elif value not in KeyTransform.SPEC_MAP:
+                valid_names = ", ".join(
+                    sorted(x.__name__ for x in KeyTransform.SPEC_MAP.keys())
+                )
                 errors.append(
                     checks.Error(
                         "The value for '{}' in 'spec{}' is not an allowed type".format(
                             key, path
                         ),
-                        hint="'spec' values must be one of the following "
-                        "types: {}".format(KeyTransform.SPEC_MAP_NAMES),
+                        hint=(
+                            "'spec' values must be one of the following types: "
+                            + valid_names
+                        ),
                         obj=self,
                         id="django_mysql.E011",
                     )
@@ -306,8 +311,6 @@ class KeyTransform(Transform):
         dict: "BINARY",
     }
 
-    SPEC_MAP_NAMES = ", ".join(sorted(x.__name__ for x in SPEC_MAP.keys()))
-
     TYPE_MAP: dict[str, Field[Any, Any]] = {
         "CHAR": TextField(),
         "DATE": DateField(),

src/django_mysql/models/functions.py

@@ -441,20 +441,16 @@ class AsType(Func):
     template = "%(expressions)s AS %(data_type)s"
 
     def __init__(self, expression: ExpressionArgument, data_type: str) -> None:
+        from django_mysql.models.fields.dynamic import KeyTransform
+
         if not hasattr(expression, "resolve_expression"):
             expression = Value(expression)
 
-        if data_type not in self.TYPE_MAP:
+        if data_type not in KeyTransform.TYPE_MAP and data_type != "BINARY":
             raise ValueError(f"Invalid data_type '{data_type}'")
 
         super().__init__(expression, data_type=data_type)
 
-    @property
-    def TYPE_MAP(self) -> dict[str, type[DjangoField] | DjangoField]:
-        from django_mysql.models.fields.dynamic import KeyTransform
-
-        return KeyTransform.TYPE_MAP
-
 
 class ColumnAdd(Func):
     function = "COLUMN_ADD"
@@ -508,25 +504,22 @@ def __init__(
         self,
         expression: ExpressionArgument,
         column_name: ExpressionArgument,
-        data_type: ExpressionArgument,
+        data_type: str,
     ):
+        from django_mysql.models.fields.dynamic import DynamicField, KeyTransform
+
         if not hasattr(column_name, "resolve_expression"):
             column_name = Value(column_name)
 
-        try:
-            output_field = self.TYPE_MAP[data_type]
-        except KeyError:
-            raise ValueError(f"Invalid data_type '{data_type}'")
-
+        output_field: DjangoField[Any, Any]
         if data_type == "BINARY":
-            output_field = output_field()
+            output_field = DynamicField()
+        else:
+            try:
+                output_field = KeyTransform.TYPE_MAP[data_type]
+            except KeyError:
+                raise ValueError(f"Invalid data_type {data_type!r}")
 
         super().__init__(
             expression, column_name, output_field=output_field, data_type=data_type
         )
-
-    @property
-    def TYPE_MAP(self) -> dict[str, DjangoField | type[DjangoField]]:
-        from django_mysql.models.fields.dynamic import KeyTransform
-
-        return KeyTransform.TYPE_MAP