Skip to content

Commit 1a6103a

Browse files
committed
PR review adjustment: better naming for a test helper
1 parent b629b31 commit 1a6103a

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

mithril-common/src/crypto_helper/merkle_map.rs

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -491,7 +491,7 @@ mod tests {
491491
MKTree::new(&leaves).unwrap()
492492
}
493493

494-
fn join_merkle_tree(block_ranges: &[BlockRange]) -> Vec<(BlockRange, MKTree)> {
494+
fn generate_merkle_trees_for_ranges(block_ranges: &[BlockRange]) -> Vec<(BlockRange, MKTree)> {
495495
block_ranges
496496
.iter()
497497
.map(|block_range| (block_range.to_owned(), generate_merkle_tree(block_range)))
@@ -530,7 +530,7 @@ mod tests {
530530

531531
#[test]
532532
fn test_mk_map_should_accept_replacement_with_same_root_value() {
533-
let entries = join_merkle_tree(&[
533+
let entries = generate_merkle_trees_for_ranges(&[
534534
BlockRange::new(0, 3),
535535
BlockRange::new(4, 6),
536536
BlockRange::new(7, 9),
@@ -555,7 +555,7 @@ mod tests {
555555

556556
#[test]
557557
fn test_mk_map_should_reject_replacement_with_different_root_value() {
558-
let entries = join_merkle_tree(&[
558+
let entries = generate_merkle_trees_for_ranges(&[
559559
BlockRange::new(0, 3),
560560
BlockRange::new(4, 6),
561561
BlockRange::new(7, 9),
@@ -572,7 +572,7 @@ mod tests {
572572

573573
#[test]
574574
fn test_mk_map_replace_should_accept_replacement_with_same_root_value() {
575-
let entries = join_merkle_tree(&[
575+
let entries = generate_merkle_trees_for_ranges(&[
576576
BlockRange::new(0, 3),
577577
BlockRange::new(4, 6),
578578
BlockRange::new(7, 9),
@@ -606,7 +606,7 @@ mod tests {
606606

607607
#[test]
608608
fn test_mk_map_replace_should_reject_replacement_if_key_doesnt_exist() {
609-
let entries = join_merkle_tree(&[
609+
let entries = generate_merkle_trees_for_ranges(&[
610610
BlockRange::new(0, 3),
611611
BlockRange::new(4, 6),
612612
BlockRange::new(7, 9),
@@ -630,7 +630,7 @@ mod tests {
630630

631631
#[test]
632632
fn test_mk_map_replace_should_reject_replacement_with_different_root_value() {
633-
let entries = join_merkle_tree(&[
633+
let entries = generate_merkle_trees_for_ranges(&[
634634
BlockRange::new(0, 3),
635635
BlockRange::new(4, 6),
636636
BlockRange::new(7, 9),
@@ -654,7 +654,7 @@ mod tests {
654654

655655
#[test]
656656
fn test_mk_map_should_compress_correctly() {
657-
let entries = join_merkle_tree(&[
657+
let entries = generate_merkle_trees_for_ranges(&[
658658
BlockRange::new(0, 3),
659659
BlockRange::new(4, 6),
660660
BlockRange::new(7, 9),
@@ -675,7 +675,7 @@ mod tests {
675675

676676
#[test]
677677
fn test_mk_map_should_reject_out_of_order_insertion() {
678-
let entries = join_merkle_tree(&[
678+
let entries = generate_merkle_trees_for_ranges(&[
679679
BlockRange::new(0, 3),
680680
BlockRange::new(4, 6),
681681
BlockRange::new(7, 9),
@@ -693,7 +693,7 @@ mod tests {
693693

694694
#[test]
695695
fn test_mk_map_should_list_keys_correctly() {
696-
let entries = join_merkle_tree(&[
696+
let entries = generate_merkle_trees_for_ranges(&[
697697
BlockRange::new(0, 3),
698698
BlockRange::new(4, 6),
699699
BlockRange::new(7, 9),
@@ -716,7 +716,7 @@ mod tests {
716716

717717
#[test]
718718
fn test_mk_map_should_list_values_correctly() {
719-
let entries = join_merkle_tree(&[
719+
let entries = generate_merkle_trees_for_ranges(&[
720720
BlockRange::new(0, 3),
721721
BlockRange::new(4, 6),
722722
BlockRange::new(7, 9),
@@ -742,7 +742,7 @@ mod tests {
742742

743743
#[test]
744744
fn test_mk_map_should_find_value_correctly() {
745-
let entries = join_merkle_tree(&[
745+
let entries = generate_merkle_trees_for_ranges(&[
746746
BlockRange::new(0, 3),
747747
BlockRange::new(4, 6),
748748
BlockRange::new(7, 9),
@@ -755,7 +755,7 @@ mod tests {
755755

756756
#[test]
757757
fn test_mk_map_should_clone_and_compute_same_root() {
758-
let entries = join_merkle_tree(&[
758+
let entries = generate_merkle_trees_for_ranges(&[
759759
BlockRange::new(0, 3),
760760
BlockRange::new(4, 6),
761761
BlockRange::new(7, 9),

0 commit comments

Comments
 (0)