test(op-revm): Adds caller nonce assertion to op-revm intergation tests (#2815)

* Adds handler test for verifying unchanged nonce on balance too low revert

* Fix system call test

* fixup! Fix merge conflicts

Author: emhane

crates/ee-tests/src/op_revm_tests.rs

@@ -1134,11 +1134,9 @@ fn test_system_call() {
 
     let mut evm = ctx.build_op();
 
-    evm.system_call_one(SYSTEM_ADDRESS, BENCH_TARGET, bytes!("0x0001"))
-        .unwrap();
-
-    // Run evm.
-    let output = evm.replay().unwrap();
+    let _ = evm.system_call_one(SYSTEM_ADDRESS, BENCH_TARGET, bytes!("0x0001"));
+    let state = evm.finalize();
 
-    compare_or_save_op_testdata("test_system_call.json", &output);
+    assert!(state.get(&SYSTEM_ADDRESS).is_none());
+    assert!(state.get(&BENCH_TARGET).unwrap().is_touched());
 }

crates/op-revm/src/handler.rs

@@ -1109,4 +1109,37 @@ mod tests {
         let account = evm.ctx().journal_mut().load_account(SENDER).unwrap();
         assert_eq!(account.info.balance, expected_refund);
     }
+
+    #[test]
+    fn test_tx_low_balance_nonce_unchanged() {
+        let ctx = Context::op().with_tx(
+            OpTransaction::builder()
+                .base(TxEnv::builder().value(U256::from(1000)))
+                .build_fill(),
+        );
+
+        let mut evm = ctx.build_op();
+
+        let handler =
+            OpHandler::<_, EVMError<_, OpTransactionError>, EthFrame<EthInterpreter>>::new();
+
+        let result = handler.validate_against_state_and_deduct_caller(&mut evm);
+
+        assert!(matches!(
+            result.err().unwrap(),
+            EVMError::Transaction(OpTransactionError::Base(
+                InvalidTransaction::LackOfFundForMaxFee { .. }
+            ))
+        ));
+        assert_eq!(
+            evm.0
+                .ctx
+                .journal_mut()
+                .load_account(Address::ZERO)
+                .unwrap()
+                .info
+                .nonce,
+            0
+        );
+    }
 }