Initial fix to consider 'only' optimization for relay

[bellini666]

This is an initial step to solve #24

[codecov-io]

Codecov Report

Merging #31 into master will decrease coverage by 0.25%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master      #31      +/-   ##
==========================================
- Coverage   93.22%   92.97%   -0.26%     
==========================================
  Files           6        6              
  Lines         310      313       +3     
==========================================
+ Hits          289      291       +2     
- Misses         21       22       +1

Impacted Files	Coverage Δ
graphene_django_optimizer/query.py	91.4% <75%> (-0.3%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 48db2b2...3cd05a5. Read the comment docs.

[NyanKiyoshi]

Hi! Thank you for the PR! Could you add the proper test cases for it? Thanks!

[gghildyal]

@NyanKiyoshi @bellini666 There is indeed already a tests that attempts to test this very scenario.

See test_relay.py::test_should_optimize_query_by_only_requesting_id_field. Although the aim of the test is to test the very same scenario, it fails to reproduce this issue (and passes with the buggy code) because create_resolve_info() function doesn't set up the GlobalID field like it should be (i.e inside of a partial function).

[bellini666]

I'm closing this PR since #44 supersedes this.