matter-labs / foundry-zksync

Fork of Foundry tailored for zkSync environment
Apache License 2.0
302 stars 130 forks source link

chore: perform startup zkEVM storage migration just before test contract execution #732

Closed nbaztec closed 1 week ago

nbaztec commented 1 week ago

What :computer:

Why :hand:

Evidence :camera:

Tests pass

nbaztec commented 1 week ago

We can substitute it with functional and_then but it is actually the idiomatic way. I reckoned adding a 4th invariant (SkipInEvm, EvmModeNoop, etc.) but it didn't feel right in the enum category at all, and was requiring quite a bit of docs in explanation, which is usually a direct sign that it doesn't belong there.