quickcheck 1: do not unwrap Either

rust-itertools · Jun 13, 2023 · ac5c357 · ac5c357
1 parent dc359e0
commit ac5c357
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/tests/quick.rs b/tests/quick.rs
@@ -837,12 +837,12 @@ quickcheck! {
         let it_ord = merge_join_by(a.clone(), b.clone(), Ord::cmp).flat_map(|v| match v {
             EitherOrBoth::Both(l, r) => {
                 has_equal = true;
-                vec![l, r]
+                vec![Either::Left(l), Either::Right(r)]
             }
-            EitherOrBoth::Left(l) => vec![l],
-            EitherOrBoth::Right(r) => vec![r],
+            EitherOrBoth::Left(l) => vec![Either::Left(l)],
+            EitherOrBoth::Right(r) => vec![Either::Right(r)],
         });
-        let it_bool = merge_join_by(a, b, PartialOrd::le).map(Either::into_inner);
+        let it_bool = merge_join_by(a, b, PartialOrd::le);
         itertools::equal(it_ord, it_bool) || has_equal
     }
     fn merge_join_by_bool_unwrapped_is_merge_by(a: Vec<u8>, b: Vec<u8>) -> bool {