Enigmatis / graphql-java-annotations

GraphQL Annotations for Java
Other
387 stars 97 forks source link

fix @GraphQLConnection for [ObjectType!]! #250

Closed asinbow closed 4 years ago

asinbow commented 4 years ago

For easing the frontend development in strict typed system like typescript or flowjs, we usually use double @GraphQLNonNull like [ObjectType!]!. But there is a small bug in ConnectionUtil.java that it doesn't unwrap the internal NonNull!. This PR is for it , thanks for reviewing.

asinbow commented 4 years ago

~~Totally new about this repository. I want to improve unit test for it, any suggestion?~~

Added my tests for it. I think it's ready for reviewing.