Parser fails on empty parenthesis without arguments #15

New Issue

belka · 2024-09-15T22:10:56+02:00

belka commented

2024-09-15 22:10:56 +02:00

Example:

{ workers() { id firstName lastName } }

It fails with an error:

unexpected ')'

expecting Argument

Example: ```graphql { workers() { id firstName lastName } } ``` It fails with an error: > unexpected ')' > > expecting Argument

belka added the

Kind

bug

label 2024-09-15 22:10:56 +02:00

belka added the

Status

fixed

label 2024-09-17 18:42:32 +02:00

belka commented

2024-09-17 18:42:36 +02:00

Works as expected as it seems. ArgumentsDefinition is defined as a list of InputValueDefinition in parens. And a list is defined as:

A subscript suffix “Symbol list” is shorthand for a list of one or more of that symbol, represented as an additional recursive production.

So if the parens are there, it least one argument should be given. I added a test to check this.

Works as expected as it seems. `ArgumentsDefinition` is defined as a list of `InputValueDefinition` in parens. And a list is defined as: > A subscript suffix “Symbol list” is shorthand for a list of one or more of that symbol, represented as an additional recursive production. So if the parens are there, it least one argument should be given. I added a test to check this.

belka closed this issue

2024-09-17 18:42:36 +02:00

belka added

and removed

labels 2024-09-20 17:53:15 +02:00

Sign in to join this conversation.