Skip to content

Commit 16c710a

Browse files
committed
Removed parser support for legacy syntax
Note that this was not included in GraphQL-core anyway, so we just do some minor clean-up. Replicates graphql/graphql-js@0d2bd54
1 parent 04a96a2 commit 16c710a

File tree

2 files changed

+5
-14
lines changed

2 files changed

+5
-14
lines changed

src/graphql/language/parser.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -658,10 +658,11 @@ def parse_object_type_definition(self) -> ObjectTypeDefinitionNode:
658658

659659
def parse_implements_interfaces(self) -> List[NamedTypeNode]:
660660
"""ImplementsInterfaces"""
661-
if not self.expect_optional_keyword("implements"):
662-
return []
663-
664-
return self.delimited_many(TokenKind.AMP, self.parse_named_type)
661+
return (
662+
self.delimited_many(TokenKind.AMP, self.parse_named_type)
663+
if self.expect_optional_keyword("implements")
664+
else []
665+
)
665666

666667
def parse_fields_definition(self) -> List[FieldDefinitionNode]:
667668
"""FieldsDefinition: {FieldDefinition+}"""

tests/language/test_schema_parser.py

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -792,13 +792,3 @@ def directive_with_incorrect_locations():
792792

793793
def parses_kitchen_sink_schema(kitchen_sink_sdl): # noqa: F811
794794
assert parse(kitchen_sink_sdl)
795-
796-
def disallow_legacy_sdl_empty_fields_supports_type_with_empty_fields():
797-
assert_syntax_error("type Hello { }", "Expected Name, found '}'.", (1, 14))
798-
799-
def disallow_legacy_sdl_implements_interfaces():
800-
assert_syntax_error(
801-
"type Hello implements Wo rld { field: String }",
802-
"Unexpected Name 'rld'.",
803-
(1, 26),
804-
)

0 commit comments

Comments
 (0)