KMergeBy is not lazy but must be used nonetheless

Philippe-Cholet · jswrenn · commit 074c7fcc07c2 · 2024-05-16T13:07:33.000Z
diff --git 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<I, F>
 where
     I: Iterator,
diff --git 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);
     }

Original file line number	Diff line number	Diff line change
`@@ -143,11 +143,11 @@ must_use_tests! {`
`143`	`143`	`let _ = Panicking.merge_join_by(Panicking, \|_, _\| true);`
`144`	`144`	`let _ = Panicking.merge_join_by(Panicking, Ord::cmp);`
`145`	`145`	`}`
`146`		`- #[ignore]`
	`146`	`+ #[should_panic]`
`147`	`147`	`kmerge {`
`148`	`148`	`let _ = Panicking.map(\|_\| Panicking).kmerge();`
`149`	`149`	`}`
`150`		`- #[ignore]`
	`150`	`+ #[should_panic]`
`151`	`151`	`kmerge_by {`
`152`	`152`	`let _ = Panicking.map(\|_\| Panicking).kmerge_by(\|_, _\| true);`
`153`	`153`	`}`