Check strings for matching enum variants which can be passed in from variables

[dhendrie91]

trafficstars

…variables

[jamesbirtles]

Ran into this, is there anything holding this back?

[dhendrie91]

@jamesbirtles so this fix works (we've been running it in production since I submitted this PR) however it means Graphgate isn't fully spec compliant as it will try and process any string to a matching enum variant, not just ones passed as a variable, in practice this isn't really an issue. Been meaning to take a look at it again to get it, but from what I remember it's tricky as we lose the context for values at the point the enum check is carried out, so we don't know if an argument is just a string or a string passed in as a variable, so this would need a bigger fix. Will hopefully get another look at this very soon

[dhendrie91]

@sunli829 do you have any opinions on the above comments ?

[Jonatthu]

@dhendrie91 @jamesbirtles any updates on this