search

makerdao / spells-mainnet

Staging repo for MakerDAO weekly executive spells
GNU Affero General Public License v3.0
107 stars 43 forks source link

fix: make `wallets` address persistent in tests #377

Closed wei3erHase closed 6 months ago

wei3erHase commented 6 months ago

Description

The objective of this PR is to fix a bug that appears when the test needs to rollFork, the error appearance is intermittent, in my experience, main branch used to revert because of calling a non-existent contract (wallets), while after running tests with this fix, same branch (perhaps because of being cached) works fine. In contracts, when testing #374, receive the same reverts as I initially had: image image

Contribution Checklist

SidestreamColdMelon commented 6 months ago

Thanks for fixing the tests for the maintenance PRs 🙂