diff --git a/src/adaptors/mod.rs b/src/adaptors/mod.rs index 22f7aa363..52e36c48b 100644 --- a/src/adaptors/mod.rs +++ b/src/adaptors/mod.rs @@ -612,7 +612,7 @@ where /// See [`.tuple_combinations()`](crate::Itertools::tuple_combinations) for more /// information. #[derive(Clone, Debug)] -#[must_use = "iterator adaptors are lazy and do nothing unless consumed"] +#[must_use = "this iterator adaptor is not lazy but does nearly nothing unless consumed"] pub struct TupleCombinations where I: Iterator, diff --git a/src/kmerge_impl.rs b/src/kmerge_impl.rs index 4660caf07..0be3840a1 100644 --- a/src/kmerge_impl.rs +++ b/src/kmerge_impl.rs @@ -153,7 +153,7 @@ where /// /// See [`.kmerge_by()`](crate::Itertools::kmerge_by) for more /// information. -#[must_use = "iterator adaptors are lazy and do nothing unless consumed"] +#[must_use = "this iterator adaptor is not lazy but does nearly nothing unless consumed"] pub struct KMergeBy where I: Iterator, diff --git a/tests/laziness.rs b/tests/laziness.rs index f936decde..c559d33ad 100644 --- a/tests/laziness.rs +++ b/tests/laziness.rs @@ -143,11 +143,11 @@ must_use_tests! { let _ = Panicking.merge_join_by(Panicking, |_, _| true); let _ = Panicking.merge_join_by(Panicking, Ord::cmp); } - #[ignore] + #[should_panic] kmerge { let _ = Panicking.map(|_| Panicking).kmerge(); } - #[ignore] + #[should_panic] kmerge_by { let _ = Panicking.map(|_| Panicking).kmerge_by(|_, _| true); } @@ -196,10 +196,15 @@ must_use_tests! { while_some { let _ = Panicking.map(Some).while_some(); } - #[ignore] - tuple_combinations { + tuple_combinations1 { let _ = Panicking.tuple_combinations::<(_,)>(); + } + #[should_panic] + tuple_combinations2 { let _ = Panicking.tuple_combinations::<(_, _)>(); + } + #[should_panic] + tuple_combinations3 { let _ = Panicking.tuple_combinations::<(_, _, _)>(); } combinations {