Remove runtime checks that duplicate type hints

The code contained many runtime checks that threw TypeErrors
if wrong types were passed, even though it is considered
to be more "pythonic" to not check types at runtime at all.

In general, these checks add runtime cost and we can't
realistically check all arguments to all functions.
Instead we should focus on adding more asserts on stuff
that can't be checked using type hints.

If runtime checks are desired, they can still be added
by using runtime type-checkers like Beartype.

Replicates graphql/graphql-js@a4b085b

Author: Cito

README.md

@@ -10,7 +10,7 @@ a query language for APIs created by Facebook.
 ![Lint Status](https://github.com/graphql-python/graphql-core/actions/workflows/lint.yml/badge.svg)
 [![Code Style](https://img.shields.io/badge/code%20style-black-000000.svg)](https://github.com/ambv/black)
 
-An extensive test suite with over 2400 unit tests and 100% coverage comprises a
+An extensive test suite with over 2300 unit tests and 100% coverage comprises a
 replication of the complete test suite of GraphQL.js, making sure this port is
 reliable and compatible with GraphQL.js.
 

src/graphql/execution/execute.py

@@ -251,7 +251,7 @@ def build(
         For internal use only.
         """
         # If arguments are missing or incorrect, throw an error.
-        assert_valid_execution_arguments(schema, document, raw_variable_values)
+        assert_valid_execution_arguments(schema)
 
         operation: Optional[OperationDefinitionNode] = None
         fragments: Dict[str, FragmentDefinitionNode] = {}
@@ -1104,8 +1104,6 @@ def execute_sync(
 
 def assert_valid_execution_arguments(
     schema: GraphQLSchema,
-    document: DocumentNode,
-    raw_variable_values: Optional[Dict[str, Any]] = None,
 ) -> None:
     """Check that the arguments are acceptable.
 
@@ -1114,20 +1112,9 @@ def assert_valid_execution_arguments(
 
     For internal use only.
     """
-    if not document:
-        raise TypeError("Must provide document.")
-
     # If the schema used for execution is invalid, throw an error.
     assert_valid_schema(schema)
 
-    # Variables, if provided, must be a dictionary.
-    if not (raw_variable_values is None or isinstance(raw_variable_values, dict)):
-        raise TypeError(
-            "Variable values must be provided as a dictionary"
-            " with variable names as keys. Perhaps look to see"
-            " if an unparsed JSON string was provided."
-        )
-
 
 def invalid_return_type_error(
     return_type: GraphQLObjectType, result: Any, field_nodes: List[FieldNode]
@@ -1345,7 +1332,7 @@ def create_source_event_stream(
     """
     # If arguments are missing or incorrectly typed, this is an internal developer
     # mistake which should throw an early error.
-    assert_valid_execution_arguments(schema, document, variable_values)
+    assert_valid_execution_arguments(schema)
 
     if not execution_context_class:
         execution_context_class = ExecutionContext