Bug on related_models_with_context with single resource

[thibaudgg]

Hi @valscion,

While testing the v1.0.0.alpha1 release on our app, I got a failing test on the AuthorizingProcessor#related_models_with_context on line 280.

Basically if assoc_value is not an Hash or an Array then assoc_value[:id] should not be used, but rather assoc_value instead. Which would give (and make my test pass):

resource_class.find_by_key(assoc_value, context: context)._model

What do you think? Could you fix that one directly one master or should I submit a PR with a spec?

[valscion]

Oh yeah, this change was part of the diff in #60

I'll cherry-pick b4233b8 to a new PR and merge + release a new version. Thanks!

[valscion]

On another note, it'd be awesome to have a test that would've failed because of this — I assume some specs in #60 might've failed without this fix..?

[gnfisher]

Yes, I discovered it while working on specs in #60. I will include specs that test it in that PR.

[valscion]

Released in v1.0.0.alpha2 :)