makerdao / spells-mainnet

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

Update Foundry `vm.snapshot()` / `vm.revertTo()` usage in tests #434

Open amusingaxl opened 1 month ago

amusingaxl commented 1 month ago

With the latest version of Foundry (forge 0.2.0 (cc8e430 2024-10-16T00:22:08.572482504Z)), we currently get the following warning:

Warning: the following cheatcode(s) are deprecated and will be removed in future versions:
snapshot(): replaced by `snapshotState`
revertTo(uint256): replaced by `revertToState`

We should probably start using the new terminology sooner than later. It seems like a pretty straightforward change.