oksa icon indicating copy to clipboard operation
oksa copied to clipboard

Fix parser bug with sequential selection sets

Open ilmoraunio opened this issue 1 year ago • 0 comments

It was possible to write:

(oksa.core/gql [[:foo {} [:qux :baz]] [:basho]])
; or
(oksa.core/gql
  (oksa.alpha.api/select (oksa.alpha.api/field :foo
                                (oksa.alpha.api/select :qux :baz))
                         (oksa.alpha.api/select :basho)))

resulting in incorrect GraphQL:

{foo{qux baz}{basho}}

This PR fixes the parser to consider sequential selection sets as invalid. We also remove an API test case that was invalid to begin with.

ilmoraunio avatar Apr 26 '24 20:04 ilmoraunio