Skip to content

Commit

Permalink
fix: correct testing naming convention
Browse files Browse the repository at this point in the history
  • Loading branch information
han0110 committed Jul 10, 2022
1 parent 6bf26cc commit 64f358f
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
8 changes: 4 additions & 4 deletions src/protocol/halo2/test/kzg/evm.rs
Original file line number Diff line number Diff line change
Expand Up @@ -119,14 +119,14 @@ test!(
);
test!(
#[ignore = "cause it requires 64GB memory to run"],
zk_two_snark,
zk_accumulation_two_snark,
21,
halo2_kzg_config!(true, 1, (0..4 * LIMBS).map(|idx| (0, idx)).collect()),
Accumulation::two_snark(true)
);
test!(
#[ignore = "cause it requires 128GB memory to run"],
zk_two_snark_with_accumulator,
zk_accumulation_two_snark_with_accumulator,
22,
halo2_kzg_config!(true, 1, (0..4 * LIMBS).map(|idx| (0, idx)).collect()),
Accumulation::two_snark_with_accumulator(true)
Expand All @@ -145,14 +145,14 @@ test!(
);
test!(
#[ignore = "cause it requires 64GB memory to run"],
two_snark,
accumulation_two_snark,
21,
halo2_kzg_config!(false, 1, (0..4 * LIMBS).map(|idx| (0, idx)).collect()),
Accumulation::two_snark(false)
);
test!(
#[ignore = "cause it requires 128GB memory to run"],
two_snark_with_accumulator,
accumulation_two_snark_with_accumulator,
22,
halo2_kzg_config!(false, 1, (0..4 * LIMBS).map(|idx| (0, idx)).collect()),
Accumulation::two_snark_with_accumulator(false)
Expand Down
8 changes: 4 additions & 4 deletions src/protocol/halo2/test/kzg/halo2.rs
Original file line number Diff line number Diff line change
Expand Up @@ -353,28 +353,28 @@ macro_rules! test {

test!(
#[ignore = "cause it requires 64GB memory to run"],
zk_two_snark,
zk_accumulation_two_snark,
21,
halo2_kzg_config!(true, 1, Accumulation::accumulator_indices()),
Accumulation::two_snark(true)
);
test!(
#[ignore = "cause it requires 128GB memory to run"],
zk_two_snark_with_accumulator,
zk_accumulation_two_snark_with_accumulator,
22,
halo2_kzg_config!(true, 1, Accumulation::accumulator_indices()),
Accumulation::two_snark_with_accumulator(true)
);
test!(
#[ignore = "cause it requires 64GB memory to run"],
two_snark,
accumulation_two_snark,
21,
halo2_kzg_config!(false, 1, Accumulation::accumulator_indices()),
Accumulation::two_snark(false)
);
test!(
#[ignore = "cause it requires 128GB memory to run"],
two_snark_with_accumulator,
accumulation_two_snark_with_accumulator,
22,
halo2_kzg_config!(false, 1, Accumulation::accumulator_indices()),
Accumulation::two_snark_with_accumulator(false)
Expand Down

0 comments on commit 64f358f

Please sign in to comment.