Some adaptations to long lines after reformatting

Cito · Cito · commit ea0c52a799a9 · 2018-09-19T14:51:56.000+02:00
diff --git a/graphql/error/graphql_error.py b/graphql/error/graphql_error.py
@@ -4,9 +4,9 @@
 from .print_error import print_error
 
 if TYPE_CHECKING:  # pragma: no cover
-    from ..language.ast import Node  # noqa
-    from ..language.location import SourceLocation  # noqa
-    from ..language.source import Source  # noqa
+    from ..language.ast import Node  # noqa: F401
+    from ..language.location import SourceLocation  # noqa: F401
+    from ..language.source import Source  # noqa: F401
 
 __all__ = ["GraphQLError"]
 
diff --git a/graphql/error/located_error.py b/graphql/error/located_error.py
@@ -3,7 +3,7 @@
 from .graphql_error import GraphQLError
 
 if TYPE_CHECKING:  # pragma: no cover
-    from ..language.ast import Node  # noqa
+    from ..language.ast import Node  # noqa: F401
 
 __all__ = ["located_error"]
 
diff --git a/graphql/language/parser.py b/graphql/language/parser.py
@@ -107,7 +107,9 @@ def parse(
         source,
         no_location=no_location,
         experimental_fragment_variables=experimental_fragment_variables,
-        experimental_variable_definition_directives=experimental_variable_definition_directives,  # noqa
+        experimental_variable_definition_directives=(
+            experimental_variable_definition_directives
+        ),
     )
     return parse_document(lexer)
 
diff --git a/setup.py b/setup.py
@@ -46,4 +46,3 @@
     include_package_data=True,
     zip_safe=False,
 )
-
diff --git a/tests/execution/test_mutations.py b/tests/execution/test_mutations.py
@@ -80,8 +80,8 @@ async def promise_and_fail_to_change_the_number(self, newNumber: int):
             "promiseAndFailToChangeTheNumber": GraphQLField(
                 numberHolderType,
                 args={"newNumber": GraphQLArgument(GraphQLInt)},
-                resolve=lambda obj, _info, newNumber: obj.promise_and_fail_to_change_the_number(  # noqa
-                    newNumber
+                resolve=lambda obj, _info, newNumber: (
+                    obj.promise_and_fail_to_change_the_number(newNumber)
                 ),
             ),
         },
diff --git a/tests/execution/test_variables.py b/tests/execution/test_variables.py
@@ -90,7 +90,7 @@ def field_with_input_arg(input_arg: GraphQLArgument):
         "fieldWithDefaultArgumentValue": field_with_input_arg(
             GraphQLArgument(GraphQLString, default_value="Hello World")
         ),
-        "fieldWithNonNullableStringInputAndDefaultArgumentValue": field_with_input_arg(
+        "fieldWithNonNullableStringInputAndDefaultArgValue": field_with_input_arg(
             GraphQLArgument(GraphQLNonNull(GraphQLString), default_value="Hello World")
         ),
         "fieldWithNestedInputObject": field_with_input_arg(
@@ -162,7 +162,8 @@ def properly_parses_null_value_to_null():
 
                 assert result == (
                     {
-                        "fieldWithObjectInput": "{'a': None, 'b': None, 'c': 'C', 'd': None}"  # noqa
+                        "fieldWithObjectInput": "{'a': None, 'b': None,"
+                        " 'c': 'C', 'd': None}"
                     },
                     None,
                 )
@@ -907,14 +908,12 @@ def when_no_runtime_value_is_provided_to_a_non_null_argument():
             result = execute_query(
                 """
                 query optionalVariable($optional: String) {
-                  fieldWithNonNullableStringInputAndDefaultArgumentValue(input: $optional)
+                  fieldWithNonNullableStringInputAndDefaultArgValue(input: $optional)
                 }
-                """  # noqa
+                """
             )
 
             assert result == (
-                {
-                    "fieldWithNonNullableStringInputAndDefaultArgumentValue": "'Hello World'"  # noqa
-                },
+                {"fieldWithNonNullableStringInputAndDefaultArgValue": "'Hello World'"},
                 None,
             )
diff --git a/tests/language/test_printer.py b/tests/language/test_printer.py
@@ -195,4 +195,4 @@ def prints_kitchen_sink(kitchen_sink):  # noqa: F811
               query
             }
             '''
-        )  # noqa
+        )
diff --git a/tests/language/test_schema_printer.py b/tests/language/test_schema_printer.py
@@ -156,5 +156,5 @@ def prints_kitchen_sink(kitchen_sink):  # noqa: F811
             extend schema @onSchema {
               subscription: SubscriptionType
             }
-            '''  # noqa
+            '''  # noqa: E501
         )
diff --git a/tests/type/test_enum.py b/tests/type/test_enum.py
@@ -324,7 +324,8 @@ def may_be_internally_represented_with_complex_values():
             {"first": "ONE", "second": "TWO", "good": "TWO", "bad": None},
             [
                 {
-                    "message": "Expected a value of type 'Complex' but received: Complex2",  # noqa
+                    "message": "Expected a value of type 'Complex'"
+                    " but received: Complex2",
                     "locations": [(6, 15)],
                     "path": ["bad"],
                 }
diff --git a/tests/type/test_introspection.py b/tests/type/test_introspection.py
@@ -1214,25 +1214,28 @@ def exposes_descriptions_on_types_and_fields():
                     "fields": [
                         {
                             "name": "types",
-                            "description": "A list of all types supported by this server.",  # noqa
+                            "description": "A list of all types supported"
+                            " by this server.",
                         },
                         {
                             "name": "queryType",
-                            "description": "The type that query operations will be rooted at.",  # noqa
+                            "description": "The type that query operations"
+                            " will be rooted at.",
                         },
                         {
                             "name": "mutationType",
-                            "description": "If this server supports mutation, the type that"  # noqa
-                            " mutation operations will be rooted at.",  # noqa
+                            "description": "If this server supports mutation, the type"
+                            " that mutation operations will be rooted at.",
                         },
                         {
                             "name": "subscriptionType",
-                            "description": "If this server support subscription, the type"  # noqa
-                            " that subscription operations will be rooted at.",  # noqa
+                            "description": "If this server support subscription,"
+                            " the type that subscription operations will be rooted at.",
                         },
                         {
                             "name": "directives",
-                            "description": "A list of all directives supported by this server.",  # noqa
+                            "description": "A list of all directives supported"
+                            " by this server.",
                         },
                     ],
                 }
diff --git a/tests/type/test_validation.py b/tests/type/test_validation.py
@@ -133,7 +133,7 @@ def accepts_a_schema_whose_query_and_mutation_types_are_object_types():
 
         schema_with_def = build_schema(
             """
-              schema {
+            schema {
               query: QueryRoot
               mutation: MutationRoot
             }
@@ -165,7 +165,7 @@ def accepts_a_schema_whose_query_and_subscription_types_are_object_types():
 
         schema_with_def = build_schema(
             """
-              schema {
+            schema {
               query: QueryRoot
               subscription: SubscriptionRoot
             }
@@ -184,10 +184,10 @@ def accepts_a_schema_whose_query_and_subscription_types_are_object_types():
     def rejects_a_schema_without_a_query_type():
         schema = build_schema(
             """
-          type Mutation {
-            test: String
-          }
-          """
+            type Mutation {
+              test: String
+            }
+            """
         )
         assert validate_schema(schema) == [
             {"message": "Query root type must be provided.", "locations": None}
@@ -343,47 +343,47 @@ def rejects_a_schema_extended_with_invalid_root_types():
             schema,
             parse(
                 """
-            extend schema {
-              query: SomeInputObject
-            }
-            """
+                extend schema {
+                  query: SomeInputObject
+                }
+                """
             ),
         )
         schema = extend_schema(
             schema,
             parse(
                 """
-            extend schema {
-              mutation: SomeInputObject
-            }
-            """
+                extend schema {
+                  mutation: SomeInputObject
+                }
+                """
             ),
         )
         schema = extend_schema(
             schema,
             parse(
                 """
-            extend schema {
-              subscription: SomeInputObject
-            }
-            """
+                extend schema {
+                  subscription: SomeInputObject
+                }
+                """
             ),
         )
         assert validate_schema(schema) == [
             {
                 "message": "Query root type must be Object type,"
                 " it cannot be SomeInputObject.",
-                "locations": [(3, 22)],
+                "locations": [(3, 26)],
             },
             {
                 "message": "Mutation root type must be Object type"
                 " if provided, it cannot be SomeInputObject.",
-                "locations": [(3, 25)],
+                "locations": [(3, 29)],
             },
             {
                 "message": "Subscription root type must be Object type"
                 " if provided, it cannot be SomeInputObject.",
-                "locations": [(3, 29)],
+                "locations": [(3, 33)],
             },
         ]
 
@@ -521,17 +521,17 @@ def rejects_a_union_type_with_empty_types():
             schema,
             parse(
                 """
-            directive @test on UNION
+                directive @test on UNION
 
-            extend union BadUnion @test
-            """
+                extend union BadUnion @test
+                """
             ),
         )
         assert validate_schema(schema) == [
             {
                 "message": "Union type BadUnion must define one or more"
                 " member types.",
-                "locations": [(6, 13), (4, 13)],
+                "locations": [(6, 13), (4, 17)],
             }
         ]
 
diff --git a/tests/utilities/test_build_ast_schema.py b/tests/utilities/test_build_ast_schema.py
@@ -553,7 +553,7 @@ def simple_type_with_mutation():
               addHelloScalars(str: String, int: Int, bool: Boolean): HelloScalars
             }
             """
-        )  # noqa
+        )
         output = cycle_output(body)
         assert output == body
 
@@ -575,7 +575,7 @@ def simple_type_with_subscription():
               subscribeHelloScalars(str: String, int: Int, bool: Boolean): HelloScalars
             }
             """
-        )  # noqa
+        )
         output = cycle_output(body)
         assert output == body
 
diff --git a/tests/utilities/test_extend_schema.py b/tests/utilities/test_extend_schema.py
@@ -457,7 +457,7 @@ def builds_types_with_deprecated_fields_and_values():
               DEPRECATED @deprecated(reason: "do not use")
             }
             """
-        )  # noqa
+        )
         deprecated_field_def = extended_schema.get_type(
             "TypeWithDeprecatedField"
         ).fields["newDeprecatedField"]
diff --git a/tests/utilities/test_find_breaking_changes.py b/tests/utilities/test_find_breaking_changes.py
@@ -636,7 +636,7 @@ def should_detect_if_a_required_field_argument_was_added():
               field1: String
             }
             """
-        )  # noqa
+        )
 
         assert find_arg_changes(old_schema, new_schema).breaking_changes == [
             (
@@ -799,7 +799,7 @@ def should_detect_all_breaking_changes():
                 field1: String
             }
             """
-        )  # noqa
+        )
 
         new_schema = build_schema(
             """
@@ -844,7 +844,7 @@ def should_detect_all_breaking_changes():
               field1: String
             }
             """
-        )  # noqa
+        )
 
         expected_breaking_changes = [
             (BreakingChangeType.TYPE_REMOVED, "Int was removed."),
diff --git a/tests/utilities/test_lexicographic_sort_schema.py b/tests/utilities/test_lexicographic_sort_schema.py
diff --git a/tests/utilities/test_schema_printer.py b/tests/utilities/test_schema_printer.py
diff --git a/tests/validation/test_known_directives.py b/tests/validation/test_known_directives.py
diff --git a/tests/validation/test_possible_fragment_spreads.py b/tests/validation/test_possible_fragment_spreads.py
diff --git a/tests/validation/test_provided_required_arguments.py b/tests/validation/test_provided_required_arguments.py
diff --git a/tests/validation/test_values_of_correct_type.py b/tests/validation/test_values_of_correct_type.py

Original file line number	Diff line number	Diff line change
`@@ -46,4 +46,3 @@`
`46`	`46`	`include_package_data=True,`
`47`	`47`	`zip_safe=False,`
`48`	`48`	`)`
`49`		`-`
Original file line number	Diff line number	Diff line change
`@@ -195,4 +195,4 @@ def prints_kitchen_sink(kitchen_sink): # noqa: F811`
`195`	`195`	`query`
`196`	`196`	`}`
`197`	`197`	`'''`
`198`		`- ) # noqa`
	`198`	`+ )`
Original file line number	Diff line number	Diff line change
`@@ -156,5 +156,5 @@ def prints_kitchen_sink(kitchen_sink): # noqa: F811`
`156`	`156`	`extend schema @onSchema {`
`157`	`157`	`subscription: SubscriptionType`
`158`	`158`	`}`
`159`		`- ''' # noqa`
	`159`	`+ ''' # noqa: E501`
`160`	`160`	`)`
Original file line number	Diff line number	Diff line change
`@@ -324,7 +324,8 @@ def may_be_internally_represented_with_complex_values():`
`324`	`324`	`{"first": "ONE", "second": "TWO", "good": "TWO", "bad": None},`
`325`	`325`	`[`
`326`	`326`	`{`
`327`		`- "message": "Expected a value of type 'Complex' but received: Complex2", # noqa`
	`327`	`+ "message": "Expected a value of type 'Complex'"`
	`328`	`+ " but received: Complex2",`
`328`	`329`	`"locations": [(6, 15)],`
`329`	`330`	`"path": ["bad"],`
`330`	`331`	`}`