Fixed flake issues

Author: syrusakbary

graphene_sqlalchemy/converter.py

@@ -47,10 +47,12 @@ def dynamic_type():
             return Field(List(_type))
 
     return Dynamic(dynamic_type)
-    
+
+
 def convert_sqlalchemy_hybrid_method(hybrid_item):
-     return String(description=getattr(hybrid_item, '__doc__', None),
-                   required=False)
+    return String(description=getattr(hybrid_item, '__doc__', None),
+                  required=False)
+
 
 def convert_sqlalchemy_composite(composite, registry):
     converter = registry.get_converter_for_composite(composite.composite_class)

graphene_sqlalchemy/types.py

@@ -50,16 +50,17 @@ def construct_fields(model, registry, only_fields, exclude_fields):
             name = hybrid_item.__name__
 
             is_not_in_only = only_fields and name not in only_fields
-            is_already_created = name in options.fields
-            is_excluded = name in exclude_fields or is_already_created
+            # is_already_created = name in options.fields
+            is_excluded = name in exclude_fields  # or is_already_created
 
             if is_not_in_only or is_excluded:
-            # We skip this field if we specify only_fields and is not
-            # in there. Or when we excldue this field in exclude_fields
-
+                # We skip this field if we specify only_fields and is not
+                # in there. Or when we excldue this field in exclude_fields
                 continue
+
             converted_hybrid_property = convert_sqlalchemy_hybrid_method(
-            hybrid_item)
+                hybrid_item
+            )
             fields[name] = converted_hybrid_property
 
     # Get all the columns for the relationships on the model