Raise TypeError instead of ValueError in format_error

We currently don't discern between assertions with
invariant, devAssert and explicitly thrown errors,
but always raise errors explicitly. Maybe revisit this:
graphql/graphql-js@ebcc754

Author: Cito

src/graphql/error/format_error.py

@@ -14,7 +14,7 @@ def format_error(error: "GraphQLError") -> Dict[str, Any]:
     Format, Errors" section of the GraphQL Specification.
     """
     if not error:
-        raise ValueError("Received null or undefined error.")
+        raise TypeError("Received no error object.")
     formatted: Dict[str, Any] = dict(  # noqa: E701 (pycqa/flake8#394)
         message=error.message or "An unknown error occurred.",
         locations=error.locations,

src/graphql/execution/execute.py

@@ -794,7 +794,7 @@ def complete_list_value(
         Complete a list value by completing each item in the list with the inner type.
         """
         if not isinstance(result, Iterable) or isinstance(result, str):
-            raise TypeError(
+            raise GraphQLError(
                 "Expected Iterable, but did not find one for field"
                 f" {info.parent_type.name}.{info.field_name}."
             )

src/graphql/utilities/build_client_schema.py

@@ -95,21 +95,21 @@ def get_named_type(type_name: str) -> GraphQLNamedType:
 
     def get_input_type(type_ref: Dict) -> GraphQLInputType:
         input_type = get_type(type_ref)
-        if not is_input_type(input_type):
-            raise TypeError(
-                "Introspection must provide input type for arguments,"
-                f" but received: {inspect(input_type)}."
-            )
-        return cast(GraphQLInputType, input_type)
+        if is_input_type(input_type):
+            return cast(GraphQLInputType, input_type)
+        raise TypeError(
+            "Introspection must provide input type for arguments,"
+            f" but received: {inspect(input_type)}."
+        )
 
     def get_output_type(type_ref: Dict) -> GraphQLOutputType:
         output_type = get_type(type_ref)
-        if not is_output_type(output_type):
-            raise TypeError(
-                "Introspection must provide output type for fields,"
-                f" but received: {inspect(output_type)}."
-            )
-        return cast(GraphQLOutputType, output_type)
+        if is_output_type(output_type):
+            return cast(GraphQLOutputType, output_type)
+        raise TypeError(
+            "Introspection must provide output type for fields,"
+            f" but received: {inspect(output_type)}."
+        )
 
     def get_object_type(type_ref: Dict) -> GraphQLObjectType:
         object_type = get_type(type_ref)

tests/error/test_format_error.py

@@ -6,7 +6,7 @@
 
 def describe_format_error():
     def throw_if_not_an_error():
-        with raises(ValueError):
+        with raises(TypeError):
             # noinspection PyTypeChecker
             format_error(None)