fix: fix optimized nested connections failing to access totalCount

[bellini666]

When resolving the connection from cache, we were missing setting nodes in it 🤦‍♂️

We do have a test that was passing with this, but MPTT triggered the issue. The issue was probably happening to corner cases as well

Used #551 as a base for the reproduction, but adjusted it to avoid depending on python 3.9

Fix #552

[sourcery-ai]

Reviewer's Guide by Sourcery

This pull request addresses an issue with nested MPTT (Modified Preorder Tree Traversal) connections when trying to access the totalCount in a GraphQL query. The changes include a fix in the resolve_connection method in strawberry_django/relay.py and the addition of several test files to ensure the correctness of the fix.

File-Level Changes

Files	Changes
tests/relay/mptt/test_nested_children.py tests/relay/mptt/test_lazy_annotations.py	Added tests to verify the correctness of nested MPTT connections and lazy type annotations in the schema.
tests/relay/mptt/a.py tests/relay/mptt/b.py tests/relay/mptt/models.py	Defined models and types for MPTTAuthor and MPTTBook with necessary connections and filters.

---

Tips

• Trigger a new Sourcery review by commenting @sourcery-ai review on the pull request.
• You can change your review settings at any time by accessing your dashboard:
  • Enable or disable the Sourcery-generated pull request summary or reviewer's guide;
  • Change the review language;
• You can always contact us if you have any questions or feedback.

[sourcery-ai]

Hey @bellini666 - I've reviewed your changes and they look great!

Here's what I looked at during the review

• 🟡 General issues: 3 issues found
• 🟢 Security: all looks good
• 🟡 Testing: 2 issues found
• 🟢 Complexity: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.}

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.71%. Comparing base (015621c) to head (98d9c9c).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #553      +/-   ##
==========================================
+ Coverage   88.67%   88.71%   +0.03%     
==========================================
  Files          41       41              
  Lines        3524     3527       +3     
==========================================
+ Hits         3125     3129       +4     
+ Misses        399      398       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.