americanexpress / nodes

A GraphQL JVM Client - Java, Kotlin, Scala, etc.
Apache License 2.0
307 stars 70 forks source link

Testing Input Object . #38

Closed bpillai closed 5 years ago

bpillai commented 5 years ago

Currently in the client since we have protected access over the methods in InputObject it becomes impossible to test an argument that has an input object as the value .

chemdrew commented 5 years ago

I am good with there being a public getter method on InputObject

bpillai commented 5 years ago

PR - https://github.com/americanexpress/nodes/pull/39/

chemdrew commented 5 years ago

👍 will be releasing a new version shortly