mimblewimble / rust-secp256k1-zkp

ZKP fork for rust-secp256k1, adds wrappers for range proofs, pedersen commitments, etc
Creative Commons Zero v1.0 Universal
56 stars 51 forks source link

test: expose the failure of batch verification on 4 proofs and so on #22

Closed garyyu closed 6 years ago

garyyu commented 6 years ago

Enhance the test to expose the failure of batch verification on range proofs.

So far, very confusing problem:

Investigation still on going.

And this PR is just to enhance the test to expose the problem, and surely it will cause travis-ci fail on Linux test, but it's the purpose.

garyyu commented 6 years ago

CI test fail as expected:)

---- pedersen::tests::test_bullet_proof_verify_multi stdout ----
1
2
3
4
5
6
7
4 proofs batch verify failed
17 proofs batch verify failed
18 proofs batch verify failed
19 proofs batch verify failed
20 proofs batch verify failed
99thread 'pedersen::tests::test_bullet_proof_verify_multi' panicked at 'assertion failed: `(left == right)`
  left: `5`,
 right: `0`', src/pedersen.rs:1223:3
note: Run with `RUST_BACKTRACE=1` for a backtrace.
failures:
    pedersen::tests::test_bullet_proof_verify_multi
test result: FAILED. 46 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out