Tests could warn, when vm.warp() was used to reduce/decrease the timestamp

[valle-xyz]

Component

Forge

Describe the feature you would like

This Situation, ok I was also stupid.

I am debugging some tests, and don't know, why they fail. In the end, it was because somewhere deep in my test setups I set vm.warp(1 hours);, and in the test I wrote I set vm.warp(1);, and this lead to a underflow error.

Proposed solution:

• When tests detect a warp that decreases the blocktime AND when tests fails, a hint is given that the blocktime was reduced.

If this is a bad idea, just close :-)

Additional context

No response

[mds1]

An alternative solution that should be simpler to implement is to have forge revert if you warp/roll to an earlier timestamp/block. I can't think of a valid use case for that, and if there is one we can add a config flag to disable this behavior

[valle-xyz]

Agree, that's better.