better support for assoc proxy lists (rather than one-to-one)

Author: dpep

graphene_sqlalchemy/converter.py

@@ -49,17 +49,23 @@ def dynamic_type():
         if isinstance(attr, AssociationProxy):
             return convert_sqlalchemy_association_proxy(
                 attr,
+                obj_type,
                 registry,
                 connection_field_factory,
                 batching,
                 resolver,
                 **field_kwargs
             )
         elif isinstance(attr, ColumnProperty):
-            return convert_sqlalchemy_column(
-                attr,
-                registry,
-                resolver,
+            # similar to convert_sqlalchemy_column, but supports lists
+            column = attr.columns[0]
+            type_ = field_kwargs.pop('type', convert_sqlalchemy_type(getattr(column, "type", None), column, registry))
+            field_kwargs.setdefault('required', not is_column_nullable(column))
+            field_kwargs.setdefault('description', get_column_doc(column))
+
+            return Field(
+                type_ if association_prop.scalar else List(type_),
+                resolver=resolver,
                 **field_kwargs
             )
         elif isinstance(attr, CompositeProperty):

graphene_sqlalchemy/tests/test_converter.py

@@ -302,7 +302,8 @@ class Meta:
     assert isinstance(dynamic_field, graphene.Dynamic)
     graphene_type = dynamic_field.get_type()
     assert isinstance(graphene_type, graphene.Field)
-    assert graphene_type.type == graphene.String
+    assert isinstance(graphene_type.type, graphene.List)
+    assert graphene_type.type.of_type == graphene.String
 
     dynamic_field = convert_sqlalchemy_association_proxy(
         Article.reporter_pets,
@@ -312,7 +313,6 @@ class Meta:
         True,
         mock_resolver,
     )
-    assert isinstance(dynamic_field.get_type().type, graphene.List)
     assert dynamic_field.get_type().type.of_type == P