morelinq / MoreLINQ

Extensions to LINQ to Objects
https://morelinq.github.io/
Apache License 2.0
3.67k stars 412 forks source link

Address to-do about consolidating testing sequence implementations #923

Closed atifaziz closed 1 year ago

atifaziz commented 1 year ago

This PR addresses a to-do comment by removing DisposalTrackingSequence in MemoizeTest and using TestingSequence instead. This required enhancing TestingSequence with an option to allow re-iteration of a sequence as well as exposing a property to assert if the enumerator was disposed.

codecov[bot] commented 1 year ago

Codecov Report

Merging #923 (096e00f) into master (63d4efe) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #923   +/-   ##
=======================================
  Coverage   92.44%   92.44%           
=======================================
  Files         112      112           
  Lines        3440     3440           
  Branches     1019     1019           
=======================================
  Hits         3180     3180           
  Misses        199      199           
  Partials       61       61           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more