Support Recording Classmethods That are Overriden by Instance Methods

[peterallenwebb]

Recording code was crashing when attempting to wrap a method override, specifically when the method on the parent class was a classmethod, but the override is an instance method. Such an override is a poor programming practice, and we should look at removing them, but we also shouldn't crash during recording.

This change makes recording work as well as possible in such cases, and avoids a crash.

Checklist

• I have read the contributing guide and understand what's expected of me
• I have signed the CLA
• I have run this code in development and it appears to resolve the stated issue
• I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
• I have run changie new to create a changelog entry

[github-actions]

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.86%. Comparing base (4cdc152) to head (4c2909e).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #287   +/-   ##
=======================================
  Coverage   71.86%   71.86%           
=======================================
  Files          52       52           
  Lines        3632     3633    +1     
=======================================
+ Hits         2610     2611    +1     
  Misses       1022     1022           

Flag	Coverage Δ
unit	71.86% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.