closes #7303 - apply invariant preserve_state setting even when fail_…

…on_revert is false (#7304)

crates/config/src/invariant.rs

@@ -30,8 +30,7 @@ pub struct InvariantConfig {
     pub shrink_run_limit: usize,
     /// If set to true then VM state is committed and available for next call
     /// Useful for handlers that use cheatcodes as roll or warp
-    /// Applies only when `fail_on_revert` set to true. Use it with caution, introduces performance
-    /// penalty.
+    /// Use it with caution, introduces performance penalty.
     pub preserve_state: bool,
 }
 

crates/evm/evm/src/executors/invariant/mod.rs

@@ -162,7 +162,7 @@ impl<'a> InvariantExecutor<'a> {
                 let (sender, (address, calldata)) = inputs.last().expect("no input generated");
 
                 // Executes the call from the randomly generated sequence.
-                let call_result = if self.config.fail_on_revert && self.config.preserve_state {
+                let call_result = if self.config.preserve_state {
                     executor
                         .call_raw_committing(*sender, *address, calldata.clone(), U256::ZERO)
                         .expect("could not make raw evm call")