From 5cf154c30a99e2b808a49debe0ea8bb6bc9846a5 Mon Sep 17 00:00:00 2001 From: "Michael J. Curry" Date: Wed, 2 Nov 2016 19:21:19 -0400 Subject: [PATCH] Added a few more tests --- src/indexer/log_merge_policy.rs | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/src/indexer/log_merge_policy.rs b/src/indexer/log_merge_policy.rs index 8ff450c9d..8ad4277dd 100644 --- a/src/indexer/log_merge_policy.rs +++ b/src/indexer/log_merge_policy.rs @@ -87,9 +87,14 @@ mod tests { } #[test] - fn test_log_merge_policy_small() { - // deal correctly with tiny segments? + fn test_log_merge_policy_within_levels() { + // multiple levels all get merged correctly + let test_input = vec![SegmentMeta::new(SegmentId::generate_random(), 10), + SegmentMeta::new(SegmentId::generate_random(), 11), + SegmentMeta::new(SegmentId::generate_random(), 12), + SegmentMeta::new(SegmentId::generate_random(), 1000), + SegmentMeta::new(SegmentId::generate_random(), 1000)]; + let result_list = LogMergePolicy::default().compute_merge_candidates(&test_input); + assert!(result_list.len() == 2); } - - // what else... deletions? threading/concurrency? }