Fixed variable tests errors

Author: syrusakbary

graphql/execution/tests/test_variables.py

@@ -1,4 +1,5 @@
 import json
+from collections import OrderedDict
 
 from pytest import raises
 
@@ -17,12 +18,12 @@
     parse_literal=lambda v: 'DeserializedValue' if v.value == 'SerializedValue' else None
 )
 
-TestInputObject = GraphQLInputObjectType('TestInputObject', {
-    'a': GraphQLInputObjectField(GraphQLString),
-    'b': GraphQLInputObjectField(GraphQLList(GraphQLString)),
-    'c': GraphQLInputObjectField(GraphQLNonNull(GraphQLString)),
-    'd': GraphQLInputObjectField(TestComplexScalar)
-})
+TestInputObject = GraphQLInputObjectType('TestInputObject', OrderedDict([
+    ('a', GraphQLInputObjectField(GraphQLString)),
+    ('b', GraphQLInputObjectField(GraphQLList(GraphQLString))),
+    ('c', GraphQLInputObjectField(GraphQLNonNull(GraphQLString))),
+    ('d', GraphQLInputObjectField(TestComplexScalar))
+]))
 
 stringify = lambda obj: json.dumps(obj, sort_keys=True)
 

graphql/type/definition.py

@@ -522,6 +522,9 @@ def _define_field_map(self):
         if callable(fields):
             fields = fields()
 
+        if not isinstance(fields, OrderedDict):
+            fields = OrderedDict(sorted(list(fields.items())))
+
         assert isinstance(fields, collections.Mapping) and len(fields) > 0, (
             '{} fields must be a mapping (dict / OrderedDict) with field names as keys or a '
             'function which returns such a mapping.'