Removed default object value in graphql query. Add tests and refine default resolve function.

Related GraphQL-js commit: graphql/graphql-js@d506c23

Author: syrusakbary

graphql/__init__.py

@@ -35,7 +35,7 @@ def graphql(schema, request='', root=None, args=None, operation_name=None):
         return execute(
             schema,
             ast,
-            root or object(),
+            root,
             operation_name=operation_name,
             variable_values=args or {},
         )

graphql/execution/tests/test_resolve.py

@@ -0,0 +1,70 @@
+from collections import OrderedDict
+import json
+
+from graphql import graphql
+from graphql.type import GraphQLSchema, GraphQLString, GraphQLObjectType, GraphQLInt, GraphQLField, GraphQLArgument
+
+
+def _test_schema(test_field):
+    return GraphQLSchema(
+        query=GraphQLObjectType(
+            name='Query',
+            fields={
+                'test': test_field
+            }
+        )
+    )
+
+
+def test_default_function_accesses_properties():
+    schema = _test_schema(GraphQLField(GraphQLString))
+
+    class source:
+        test = 'testValue'
+
+    result = graphql(schema, '{ test }', source)
+    assert not result.errors
+    assert result.data == {'test': 'testValue'}
+
+
+def test_default_function_calls_methods():
+    schema = _test_schema(GraphQLField(GraphQLString))
+
+    class source:
+        _secret = 'testValue'
+
+        def test(self):
+            return self._secret
+
+    result = graphql(schema, '{ test }', source())
+    assert not result.errors
+    assert result.data == {'test': 'testValue'}
+
+
+def test_uses_provided_resolve_function():
+    def resolver(source, args, *_):
+        return json.dumps([source, args], separators=(',', ':'))
+
+    schema = _test_schema(GraphQLField(
+        GraphQLString,
+        args=OrderedDict([
+            ('aStr', GraphQLArgument(GraphQLString)),
+            ('aInt', GraphQLArgument(GraphQLInt)),
+        ]),
+        resolver=resolver
+    ))
+
+    result = graphql(schema, '{ test }', None)
+    assert not result.errors
+    assert result.data == {'test': '[null,{}]'}
+
+    result = graphql(schema, '{ test(aStr: "String!") }', 'Source!')
+    assert not result.errors
+    assert result.data == {'test': '["Source!",{"aStr":"String!"}]'}
+
+    result = graphql(schema, '{ test(aInt: -123, aStr: "String!",) }', 'Source!')
+    assert not result.errors
+    assert result.data in [
+        {'test': '["Source!",{"aStr":"String!","aInt":-123}]'},
+        {'test': '["Source!",{"aInt":-123,"aStr":"String!"}]'}
+    ]