search

entropyxyz / entropy-core

Protocol and cryptography development.
https://docs.entropy.xyz/
GNU Affero General Public License v3.0
11 stars 2 forks source link

Fix `change_signers_info` benchmark #1035

Closed HCastano closed 2 months ago

HCastano commented 2 months ago

In https://github.com/entropyxyz/entropy-core/pull/978/ a couple of extra checks were added to ensure that parameters related to signing could only change so quickly. This ended up breaking one of the benchmarks since the benchmark setup used a difference that was larger than the allowed amount.

I've fixed that by reading the current values from storage (set at Genesis) and updating those by the allowed amount.

Maybe worth revisiting https://github.com/entropyxyz/entropy-core/pull/916...