Merge pull request #1387 from GDGSNF/master

Chore: Refactor Multi Expression Code ✨

graphene/pyutils/version.py

@@ -19,10 +19,7 @@ def get_version(version=None):
     sub = ""
     if version[3] == "alpha" and version[4] == 0:
         git_changeset = get_git_changeset()
-        if git_changeset:
+        sub = ".dev%s" % git_changeset if git_changeset else ".dev"
-            sub = ".dev%s" % git_changeset
-        else:
-            sub = ".dev"
     elif version[3] != "final":
         mapping = {"alpha": "a", "beta": "b", "rc": "rc"}
         sub = mapping[version[3]] + str(version[4])

graphene/relay/node.py

@@ -18,11 +18,7 @@ def is_node(objecttype):
     if not issubclass(objecttype, ObjectType):
         return False
 
-    for i in objecttype._meta.interfaces:
+    return any(issubclass(i, Node) for i in objecttype._meta.interfaces)
-        if issubclass(i, Node):
-            return True
-
-    return False
 
 
 class GlobalID(Field):
@@ -94,7 +90,7 @@ class Node(AbstractNode):
             raise Exception(
                 f'Unable to parse global ID "{global_id}". '
                 'Make sure it is a base64 encoded string in the format: "TypeName:id". '
-                f"Exception message: {str(e)}"
+                f"Exception message: {e}"
             )
 
         graphene_type = info.schema.get_type(_type)

graphene/types/mutation.py

@@ -101,10 +101,7 @@ class Mutation(ObjectType):
                         "Read more:"
                         " https://github.com/graphql-python/graphene/blob/v2.0.0/UPGRADE-v2.0.md#mutation-input"
                     )
-            if input_class:
+            arguments = props(input_class) if input_class else {}
-                arguments = props(input_class)
-            else:
-                arguments = {}
         if not resolver:
             mutate = getattr(cls, "mutate", None)
             assert mutate, "All mutations must define a mutate method in it"

graphene/types/resolver.py

@@ -7,9 +7,7 @@ def dict_resolver(attname, default_value, root, info, **args):
 
 
 def dict_or_attr_resolver(attname, default_value, root, info, **args):
-    resolver = attr_resolver
+    resolver = dict_resolver if isinstance(root, dict) else attr_resolver
-    if isinstance(root, dict):
-        resolver = dict_resolver
     return resolver(attname, default_value, root, info, **args)
 
 

graphene/types/tests/test_subscribe_async.py

@@ -14,9 +14,7 @@ class Subscription(ObjectType):
     count_to_ten = Field(Int)
 
     async def subscribe_count_to_ten(root, info):
-        count = 0
+        for count in range(1, 11):
-        while count < 10:
-            count += 1
             yield count
 
 

graphene/utils/module_loading.py

@@ -27,7 +27,6 @@ def import_string(dotted_path, dotted_attributes=None):
 
     if not dotted_attributes:
         return result
-    else:
     attributes = dotted_attributes.split(".")
     traveled_attributes = []
     try:

graphene/utils/tests/test_orderedtype.py

@@ -38,4 +38,4 @@ def test_orderedtype_non_orderabletypes():
 
     assert one.__lt__(1) == NotImplemented
     assert one.__gt__(1) == NotImplemented
-    assert not one == 1
+    assert one != 1

graphene/validation/tests/test_disable_introspection.py

@@ -18,14 +18,12 @@ schema = Schema(query=Query)
 def run_query(query: str):
     document = parse(query)
 
-    errors = validate(
+    return validate(
         schema=schema.graphql_schema,
         document_ast=document,
         rules=(DisableIntrospection,),
     )
 
-    return errors
-
 
 def test_disallows_introspection_queries():
     errors = run_query("{ __schema { queryType { name } } }")