logos-co / staking

SNT Staking contracts
Creative Commons Zero v1.0 Universal
0 stars 4 forks source link

fix(Certora specs): ensure prover runs rules on `currentContract` #75

Closed 0x-r4bbit closed 8 months ago

0x-r4bbit commented 8 months ago

Since we're implementing rules for StakeManager migrations, we need multiple instances inside the certora specs.

This results in the prover trying to run rules on the other StakeManager instance as well, which isn't always desired, as it causes some rules to fail, even though they'd pass if they'd be executed only on the currentContract.

This commit makes the filter condition for relevant rules stronger, such that the prover will not run them on the newStakeManager contract instance.

Checklist

Ensure you completed all of the steps below before submitting your pull request: