Copy over acessor properties to target object #2387

[sauravazad]

Fix for issue #2387 where acessor properties were not copied over to target object

The existing implementation assumed that all the properties of an object are Data property
Documentation link : https://developer.mozilla.org/en-US/docs/Web/JavaScript/Data_structures#properties

Checklist for author

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

[codecov]

Codecov Report

Merging #2391 (258caee) into master (57c6c63) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2391      +/-   ##
==========================================
+ Coverage   96.07%   96.08%   +0.01%     
==========================================
  Files          41       41              
  Lines        1883     1891       +8     
==========================================
+ Hits         1809     1817       +8     
  Misses         74       74              

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

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

Impacted Files	Coverage Δ
lib/sinon/util/core/extend.js	97.87% <100.00%> (+0.43%)	⬆️

---

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 57c6c63...258caee. Read the comment docs.

[mroderick]

Thank you for the pull request and the thorough investigation of this issue 💯

[mroderick]

This has been published to the npm registry as sinon@11.1.2