Skip to content

Commit

Permalink
amalgamate test verify_share_with_multiplicity into another test
Browse files Browse the repository at this point in the history
  • Loading branch information
ggutoski committed Sep 6, 2024
1 parent 6353895 commit ada3dda
Showing 1 changed file with 6 additions and 20 deletions.
26 changes: 6 additions & 20 deletions vid/src/advz/test.rs
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,12 @@ fn sad_path_verify_share_corrupt_share() {
let (shares, common, commit) = (disperse.shares, disperse.common, disperse.commit);

for (i, share) in shares.iter().enumerate() {
// happy path: uncorrupted share
assert!(common.multiplicity > 1, "multiplicity should be nontrivial");
advz.verify_share(&share, &common, &commit)
.unwrap()
.unwrap();

// missing share eval
{
let mut share_missing_eval = share.clone();
Expand Down Expand Up @@ -273,26 +279,6 @@ fn sad_path_recover_payload_corrupt_shares() {
}
}

#[test]
fn verify_share_with_multiplicity() {
let advz_params = AdvzParams {
recovery_threshold: 16,
num_storage_nodes: 20,
max_multiplicity: 4,
payload_len: 4000,
};
let (mut advz, payload) = advz_init_with::<Bn254>(advz_params);

let disperse = advz.disperse(payload).unwrap();
let (shares, common, commit) = (disperse.shares, disperse.common, disperse.commit);

for share in shares {
advz.verify_share(&share, &common, &commit)
.unwrap()
.unwrap()
}
}

#[test]
fn verify_share_with_different_multiplicity() {
// leader_multiplicity < everyone else's multiplicity
Expand Down

0 comments on commit ada3dda

Please sign in to comment.