Skip to content

Commit

Permalink
Add TODO
Browse files Browse the repository at this point in the history
process_results is as of now tested separately because the corresponding
iterator is not exposed nicely, and because it is not `Clone`.

However, the respective tests already diverged. Instead of fixing
process_results, we should strive to uniformly use test_specializations
for ProcessResults, too.
  • Loading branch information
phimuemue committed Oct 15, 2023
1 parent ad5e25f commit f00e3ae
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions tests/specializations.rs
Original file line number Diff line number Diff line change
Expand Up @@ -129,6 +129,7 @@ quickcheck! {
}

quickcheck! {
// TODO Replace this function by a normal call to test_specializations
fn process_results(v: Vec<Result<u8, u8>>) -> () {
helper(v.iter().copied());
helper(v.iter().copied().filter(Result::is_ok));
Expand Down

0 comments on commit f00e3ae

Please sign in to comment.