From 69eebbd848c2fedcae82fa71f419f11442fd1643 Mon Sep 17 00:00:00 2001
From: xxchan <xxchan22f@gmail.com>
Date: Sun, 11 Feb 2024 23:38:29 +0800
Subject: [PATCH] test: improve week1_day2::test_task2_merge_error (#43)

Previously a bad impl won't panic, because the heap only have one iter.
---
 mini-lsm/src/tests/week1_day2.rs | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/mini-lsm/src/tests/week1_day2.rs b/mini-lsm/src/tests/week1_day2.rs
index 6c1b002..92b5730 100644
--- a/mini-lsm/src/tests/week1_day2.rs
+++ b/mini-lsm/src/tests/week1_day2.rs
@@ -229,7 +229,11 @@ fn test_task2_merge_error() {
         ],
         1,
     );
-    let iter = MergeIterator::<MockIterator>::create(vec![Box::new(i1), Box::new(i2)]);
+    let iter = MergeIterator::<MockIterator>::create(vec![
+        Box::new(i1.clone()),
+        Box::new(i1),
+        Box::new(i2),
+    ]);
     // your implementation should correctly throw an error instead of panic
     expect_iter_error(iter);
 }