pytest migration of unit.common.resolve.test_Resolve

[bjlittle]

🚀 Pull Request

Description

---

Consult Iris pull request check list

---

Add any of the below labels to trigger actions on this PR:

• benchmark_this Request that this pull request be benchmarked to check if it introduces performance shifts

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.69%. Comparing base (9c4a9cc) to head (af9b1ef).
Report is 12 commits behind head on FEATURE_pytest_conversion.

❗ Current head af9b1ef differs from pull request most recent head 66c375a. Consider uploading reports for the commit 66c375a to get more accurate results

Additional details and impacted files

@@                    Coverage Diff                     @@
##           FEATURE_pytest_conversion    #5882   +/-   ##
==========================================================
  Coverage                      89.69%   89.69%           
==========================================================
  Files                             92       92           
  Lines                          22940    22940           
  Branches                        5462     5462           
==========================================================
  Hits                           20575    20575           
  Misses                          1638     1638           
  Partials                         727      727           

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

[pp-mo]

OK I've scanned this at last !
I identified just a few tweaks

[bjlittle]

@pp-mo Thanks for the review and wading through this one 💯

Back to you 👍

[pp-mo]

Thanks @bjlittle -- done.
That feels good !

[bjlittle]

@pp-mo Thanks, you're a legend 💯