fix tests

Author: rakita

crates/op-revm/src/handler.rs

@@ -439,6 +439,8 @@ where
 
             let old_balance = acc.info.balance;
 
+            // decrement transaction id as it was incremented when we discarded the tx.
+            acc.transaction_id -= acc.transaction_id;
             acc.info.nonce = acc.info.nonce.saturating_add(1);
             acc.info.balance = acc
                 .info

crates/state/src/lib.rs

@@ -121,7 +121,7 @@ impl Account {
         self.status.contains(AccountStatus::SelfDestructedLocal)
     }
 
-    /// Selfdestruct the account by clearing its storage and reseting its account info
+    /// Selfdestruct the account by clearing its storage and resetting its account info
     #[inline]
     pub fn selfdestruct(&mut self) {
         self.storage.clear();
@@ -261,13 +261,13 @@ bitflags! {
     /// Account status flags. Generated by bitflags crate.
     ///
     /// With multi transaction feature there is a need to have both global and local fields.
-    /// Global accross multiple transaction and local accross one transaction execution.
+    /// Global across multiple transaction and local across one transaction execution.
     ///
     /// Empty state without any flags set represent account that is loaded from db but not interacted with.
     ///
     /// `Touched` flag is used by database to check if account is potentially changed in some way.
-    /// Additionaly, after EIP-161 touch on empty-existing account would remove this account from state
-    /// after transaction execution ends. Touch can span accross multiple transactions as it is needed
+    /// Additionally, after EIP-161 touch on empty-existing account would remove this account from state
+    /// after transaction execution ends. Touch can span across multiple transactions as it is needed
     /// to be marked only once so it is safe to have only one global flag.
     /// Only first touch have different behaviour from others, and touch in first transaction will invalidate
     /// touch functionality in next transactions.
@@ -286,7 +286,7 @@ bitflags! {
     /// Additionally if account is loaded in second tx, storage and account that was destroyed in first tx needs to be cleared.
     ///
     /// `LoadedAsNotExisting` is used to mark account as loaded from database but with `balance == 0 && nonce == 0 && code = 0x`.
-    /// This flag is fine to span accross multiple transactions as it interucts with `Touched` flag this is used in global scope.
+    /// This flag is fine to span across multiple transactions as it interucts with `Touched` flag this is used in global scope.
     ///
     /// `CreatedLocal`, `SelfdestructedLocal` and `Cold` flags are reset on first account loading of local scope.
     #[derive(Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord, Hash)]
@@ -303,7 +303,7 @@ bitflags! {
         /// If account is marked for self destruction.
         const SelfDestructedLocal = 0b01000000;
         /// Only when account is marked as touched we will save it to database.
-        /// Additionaly first touch on empty existing account (After EIP-161) will mark it
+        /// Additionally first touch on empty existing account (After EIP-161) will mark it
         /// for removal from state after transaction execution.
         const Touched = 0b00000100;
         /// used only for pre spurious dragon hardforks where existing and empty were two separate states.