add test for repeated fields, fix __name__ attr for py<3.10

Author: onursatici

marshmallow_dataclass/__init__.py

@@ -374,7 +374,10 @@ def _internal_class_schema(
     clazz_frame: types.FrameType = None,
     generic_params_to_args: Optional[Tuple[Tuple[type, type], ...]] = None,
 ) -> Type[marshmallow.Schema]:
-    _RECURSION_GUARD.seen_classes[clazz] = clazz.__name__
+    # generic aliases do not have a __name__ prior python 3.10
+    _name = getattr(clazz, "__name__", repr(clazz))
+
+    _RECURSION_GUARD.seen_classes[clazz] = _name
     try:
         fields = _dataclass_fields(clazz)
     except TypeError:  # Not a dataclass
@@ -427,7 +430,7 @@ def _internal_class_schema(
         if field.init
     )
 
-    schema_class = type(clazz.__name__, (_base_schema(clazz, base_schema),), attributes)
+    schema_class = type(_name, (_base_schema(clazz, base_schema),), attributes)
     return cast(Type[marshmallow.Schema], schema_class)
 
 
@@ -446,6 +449,7 @@ def _field_by_supertype(
     metadata: dict,
     base_schema: Optional[Type[marshmallow.Schema]],
     typ_frame: Optional[types.FrameType],
+    generic_params_to_args: Optional[Tuple[Tuple[type, type], ...]] = None,
 ) -> marshmallow.fields.Field:
     """
     Return a new field for fields based on a super field. (Usually spawned from NewType)
@@ -477,6 +481,7 @@ def _field_by_supertype(
             default=default,
             base_schema=base_schema,
             typ_frame=typ_frame,
+            generic_params_to_args=generic_params_to_args,
         )
 
 
@@ -501,6 +506,7 @@ def _field_for_generic_type(
     typ: type,
     base_schema: Optional[Type[marshmallow.Schema]],
     typ_frame: Optional[types.FrameType],
+    generic_params_to_args: Optional[Tuple[Tuple[type, type], ...]] = None,
     **metadata: Any,
 ) -> Optional[marshmallow.fields.Field]:
     """
@@ -514,7 +520,10 @@ def _field_for_generic_type(
 
         if origin in (list, List):
             child_type = field_for_schema(
-                arguments[0], base_schema=base_schema, typ_frame=typ_frame
+                arguments[0],
+                base_schema=base_schema,
+                typ_frame=typ_frame,
+                generic_params_to_args=generic_params_to_args,
             )
             list_type = cast(
                 Type[marshmallow.fields.List],
@@ -529,14 +538,20 @@ def _field_for_generic_type(
             from . import collection_field
 
             child_type = field_for_schema(
-                arguments[0], base_schema=base_schema, typ_frame=typ_frame
+                arguments[0],
+                base_schema=base_schema,
+                typ_frame=typ_frame,
+                generic_params_to_args=generic_params_to_args,
             )
             return collection_field.Sequence(cls_or_instance=child_type, **metadata)
         if origin in (set, Set):
             from . import collection_field
 
             child_type = field_for_schema(
-                arguments[0], base_schema=base_schema, typ_frame=typ_frame
+                arguments[0],
+                base_schema=base_schema,
+                typ_frame=typ_frame,
+                generic_params_to_args=generic_params_to_args,
             )
             return collection_field.Set(
                 cls_or_instance=child_type, frozen=False, **metadata
@@ -545,14 +560,22 @@ def _field_for_generic_type(
             from . import collection_field
 
             child_type = field_for_schema(
-                arguments[0], base_schema=base_schema, typ_frame=typ_frame
+                arguments[0],
+                base_schema=base_schema,
+                typ_frame=typ_frame,
+                generic_params_to_args=generic_params_to_args,
             )
             return collection_field.Set(
                 cls_or_instance=child_type, frozen=True, **metadata
             )
         if origin in (tuple, Tuple):
             children = tuple(
-                field_for_schema(arg, base_schema=base_schema, typ_frame=typ_frame)
+                field_for_schema(
+                    arg,
+                    base_schema=base_schema,
+                    typ_frame=typ_frame,
+                    generic_params_to_args=generic_params_to_args,
+                )
                 for arg in arguments
             )
             tuple_type = cast(
@@ -566,10 +589,16 @@ def _field_for_generic_type(
             dict_type = type_mapping.get(Dict, marshmallow.fields.Dict)
             return dict_type(
                 keys=field_for_schema(
-                    arguments[0], base_schema=base_schema, typ_frame=typ_frame
+                    arguments[0],
+                    base_schema=base_schema,
+                    typ_frame=typ_frame,
+                    generic_params_to_args=generic_params_to_args,
                 ),
                 values=field_for_schema(
-                    arguments[1], base_schema=base_schema, typ_frame=typ_frame
+                    arguments[1],
+                    base_schema=base_schema,
+                    typ_frame=typ_frame,
+                    generic_params_to_args=generic_params_to_args,
                 ),
                 **metadata,
             )
@@ -587,6 +616,7 @@ def _field_for_generic_type(
                 metadata=metadata,
                 base_schema=base_schema,
                 typ_frame=typ_frame,
+                generic_params_to_args=generic_params_to_args,
             )
         from . import union_field
 
@@ -599,6 +629,7 @@ def _field_for_generic_type(
                         metadata={"required": True},
                         base_schema=base_schema,
                         typ_frame=typ_frame,
+                        generic_params_to_args=generic_params_to_args,
                     ),
                 )
                 for subtyp in subtypes
@@ -707,7 +738,9 @@ def field_for_schema(
                 )
         else:
             subtyp = Any
-        return field_for_schema(subtyp, default, metadata, base_schema, typ_frame)
+        return field_for_schema(
+            subtyp, default, metadata, base_schema, typ_frame, generic_params_to_args
+        )
 
     # Generic types
     generic_field = _field_for_generic_type(typ, base_schema, typ_frame, **metadata)
@@ -725,6 +758,7 @@ def field_for_schema(
             metadata=metadata,
             base_schema=base_schema,
             typ_frame=typ_frame,
+            generic_params_to_args=generic_params_to_args,
         )
 
     # enumerations

tests/test_class_schema.py

@@ -449,6 +449,31 @@ class NestedGeneric(typing.Generic[T]):
         with self.assertRaises(ValidationError):
             schema_nested_generic.load({"data": {"data": "str"}})
 
+    def test_generic_dataclass_repeated_fields(self):
+        T = typing.TypeVar("T")
+
+        @dataclasses.dataclass
+        class AA:
+            a: int
+
+        @dataclasses.dataclass
+        class BB(typing.Generic[T]):
+            b: T
+
+        @dataclasses.dataclass
+        class Nested:
+            x: BB[float]
+            z: BB[float]
+            # if y is the first field in this class, deserialisation will fail.
+            # see https://github.com/lovasoa/marshmallow_dataclass/pull/172#issuecomment-1334024027
+            y: BB[AA]
+
+        schema_nested = class_schema(Nested)()
+        self.assertEqual(
+            Nested(x=BB(b=1), z=BB(b=1), y=BB(b=AA(1))),
+            schema_nested.load({"x": {"b": 1}, "z": {"b": 1}, "y": {"b": {"a": 1}}}),
+        )
+
     def test_recursive_reference(self):
         @dataclasses.dataclass
         class Tree: