tangle-network / cggmp-threshold-ecdsa

MPC protocols for threshold ECDSA
GNU General Public License v3.0
47 stars 10 forks source link

[BUG] paillier_decryption_modulo_q test is flaky #27

Closed tmpfs closed 1 year ago

tmpfs commented 1 year ago

Describe the bug

Just saw this failure, subsequent test run passes ok:

running 21 tests
test utilities::aff_g::tests::test_affine_g_proof ... ok
test utilities::dec_q::tests::test_paillier_decryption_modulo_q ... FAILED
test utilities::enc::tests::test_paillier_encryption_in_range_proof ... ok
test utilities::log_star::tests::test_log_star_proof ... ok
test utilities::mta::range_proofs::tests::alice_zkp ... ok
test presign::state_machine::test::presign_all_parties_works ... ok
test sign::state_machine::test::sign_all_parties_works ... ok
test utilities::mta::test::test_mta ... ok
test utilities::mul::tests::test_paillier_mul ... ok
test utilities::mul_star::tests::test_mul_star_proof ... ok
test utilities::zk_pdl_with_slack::test::test_zk_pdl_with_slack ... ok
test utilities::zk_pdl::test::test_zk_pdl ... ok
test utilities::zk_pdl_with_slack::test::test_zk_pdl_with_slack_soundness - should panic ... ok
test presign::state_machine::test::presign_threshold_works ... ok
test sign::state_machine::test::sign_threshold_works ... ok
test utilities::mta::range_proofs::tests::bob_zkp ... ok
test refresh::state_machine::test::test_dkr_with_remove_parties ... ok
test refresh::state_machine::test::test_dkr_with_no_new_parties ... ok
test refresh::state_machine::test::test_dkr_with_new_threshold ... ok
test refresh::state_machine::test::test_dkr_with_replace_parties ... ok
test refresh::state_machine::test::test_dkr_with_new_parties ... ok

failures:

---- utilities::dec_q::tests::test_paillier_decryption_modulo_q stdout ----
thread 'utilities::dec_q::tests::test_paillier_decryption_modulo_q' panicked at src/utilities/dec_q/mod.rs:168:79:
called `Result::unwrap()` on an `Err` value: [5, 125, 189, 169, 34, 120, 151, 194, 233, 222, 20, 200, 108, 156, 213, 51, 130, 65, 232, 80, 37, 41, 158, 85, 105, 164, 15, 195, 144, 245, 96]
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
davidsemakula commented 1 year ago

@tmpfs @drewstone Cause and fixes described here apply to this as well.

tmpfs commented 1 year ago

Closing as replaced by #28.