Fix printing object type definitions

[swalkinshaw]

Directives come after the implemented interfaces.

See
https://github.com/graphql/graphql-js/blob/360fa06e43fed32b63fba99b353515e7b56eee20/src/language/printer.js#L117-L129
for the canonical printer.

This is pretty difficult to add a test for (I think?) and likely why there wasn't any to begin with.

@xuorig any suggestions for how to test this? Since right now we're building a document by parsing it but I don't think it supports directives like this?

[xuorig]

oh damn 😨

AST nodes can have direcitves I think, so you could build your own using Nodes and print it.

Nodes::ObjectTypeDefinition.new(
  directives: ...,
  interfaces: ...,
)

would that work? You'll need to find the right objects for directives and interfaces (TypeNameNode i think there?) but it should work

[swalkinshaw]

I was making it harder than it was. I just updated the existing kitchen sink schema to include this case.

[xuorig]

thanks 🙏

[rmosolgo]

👍