test_ast_from_value: improve coverage

Also fix porting bug revealed in added test.

Replicates graphql/graphql-js@34c5580

Author: Cito

src/graphql/utilities/ast_from_value.py

@@ -14,7 +14,7 @@
     StringValueNode,
     ValueNode,
 )
-from ..pyutils import inspect, is_nullish, is_invalid
+from ..pyutils import FrozenList, inspect, is_nullish, is_invalid
 from ..type import (
     GraphQLID,
     GraphQLInputType,
@@ -77,7 +77,8 @@ def ast_from_value(value: Any, type_: GraphQLInputType) -> Optional[ValueNode]:
         item_type = type_.of_type
         if isinstance(value, Iterable) and not isinstance(value, str):
             value_nodes = [ast_from_value(item, item_type) for item in value]
-            return ListValueNode(values=value_nodes)
+            value_nodes = [item_node for item_node in value_nodes if item_node]
+            return ListValueNode(values=FrozenList(value_nodes))
         return ast_from_value(value, item_type)
 
     # Populate the fields of the input object by creating ASTs from each value in the
@@ -97,7 +98,7 @@ def ast_from_value(value: Any, type_: GraphQLInputType) -> Optional[ValueNode]:
                             name=NameNode(value=field_name), value=field_value
                         )
                     )
-        return ObjectValueNode(fields=field_nodes)
+        return ObjectValueNode(fields=FrozenList(field_nodes))
 
     if is_leaf_type(type_):
         # Since value is an internally represented value, it must be serialized to an

tests/utilities/test_ast_from_value.py

@@ -26,6 +26,7 @@
     GraphQLInt,
     GraphQLList,
     GraphQLNonNull,
+    GraphQLScalarType,
     GraphQLString,
 )
 from graphql.utilities import ast_from_value
@@ -124,6 +125,33 @@ def converts_id_values_to_int_or_string_asts():
 
         assert ast_from_value(Undefined, GraphQLString) is None
 
+    def converts_using_serialize_from_a_custom_scalar_type():
+        pass_through_scalar = GraphQLScalarType(
+            "PassThroughScalar", serialize=lambda value: value,
+        )
+
+        assert ast_from_value("value", pass_through_scalar) == StringValueNode(
+            value="value"
+        )
+
+        return_null_scalar = GraphQLScalarType(
+            "ReturnNullScalar", serialize=lambda value: None,
+        )
+
+        assert ast_from_value("value", return_null_scalar) is None
+
+        class SomeClass:
+            pass
+
+        return_custom_class_scalar = GraphQLScalarType(
+            "ReturnCustomClassScalar", serialize=lambda value: SomeClass(),
+        )
+
+        with raises(TypeError) as exc_info:
+            ast_from_value("value", return_custom_class_scalar)
+        msg = str(exc_info.value)
+        assert msg == "Cannot convert value to AST: <SomeClass instance>."
+
     def does_not_convert_non_null_values_to_null_value():
         non_null_boolean = GraphQLNonNull(GraphQLBoolean)
         assert ast_from_value(None, non_null_boolean) is None
@@ -170,12 +198,21 @@ def converts_list_singletons():
             value="FOO"
         )
 
-    def converts_input_objects():
-        input_obj = GraphQLInputObjectType(
-            "MyInputObj",
-            {"foo": GraphQLInputField(GraphQLFloat), "bar": GraphQLInputField(my_enum)},
+    def skips_invalid_list_items():
+        ast = ast_from_value(
+            ["FOO", None, "BAR"], GraphQLList(GraphQLNonNull(GraphQLString))
+        )
+
+        assert ast == ListValueNode(
+            values=[StringValueNode(value="FOO"), StringValueNode(value="BAR")]
         )
 
+    input_obj = GraphQLInputObjectType(
+        "MyInputObj",
+        {"foo": GraphQLInputField(GraphQLFloat), "bar": GraphQLInputField(my_enum)},
+    )
+
+    def converts_input_objects():
         assert ast_from_value({"foo": 3, "bar": "HELLO"}, input_obj) == ObjectValueNode(
             fields=[
                 ObjectFieldNode(
@@ -188,11 +225,9 @@ def converts_input_objects():
         )
 
     def converts_input_objects_with_explicit_nulls():
-        input_obj = GraphQLInputObjectType(
-            "MyInputObj",
-            {"foo": GraphQLInputField(GraphQLFloat), "bar": GraphQLInputField(my_enum)},
-        )
-
         assert ast_from_value({"foo": None}, input_obj) == ObjectValueNode(
             fields=[ObjectFieldNode(name=NameNode(value="foo"), value=NullValueNode())]
         )
+
+    def does_not_convert_non_object_values_as_input_objects():
+        assert ast_from_value(5, input_obj) is None