fix: make it possible to call through to underlying stub in stub instance

[fatso83]

Purpose (TL;DR) - mandatory

A bug was introduced in #2477 that surfaced as #2501: it was impossible to call through to the underlying prototype's method when introducing no-ops. This fix moves the adding of no-ops to the restore phase, avoiding the issue.

How to verify - mandatory

Run the test suite 😄

Checklist for author

• npm run lint passes
• References to standard library functions are cached.

[codecov]

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.02 🎉

Comparison is base (6e19746) 95.96% compared to head (c8e5b37) 95.99%.

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

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2503      +/-   ##
==========================================
+ Coverage   95.96%   95.99%   +0.02%     
==========================================
  Files          38       40       +2     
  Lines        1883     1897      +14     
==========================================
+ Hits         1807     1821      +14     
  Misses         76       76              

Flag	Coverage Δ
unit	95.99% <100.00%> (+0.02%)	⬆️

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

Impacted Files	Coverage Δ
lib/sinon/create-stub-instance.js	100.00% <100.00%> (ø)
lib/sinon/sandbox.js	97.74% <100.00%> (+0.01%)	⬆️
lib/sinon/stub.js	95.19% <100.00%> (-0.54%)	⬇️
lib/sinon/util/core/sinon-type.js	100.00% <100.00%> (ø)
lib/sinon/util/core/wrap-method.js	82.17% <100.00%> (+0.73%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.