refactor(rooch-framework-tests): simplify execute_as_result

Remove unnecessary mutable variable and tx_hash caching in `execute_as_result`. This improves code clarity and reduces redundancy.

crates/rooch-framework-tests/src/binding_test.rs

@@ -259,11 +259,7 @@ impl RustBindingTest {
         self.execute_verified_tx(verified_tx)
     }
 
-    pub fn execute_as_result(
-        &mut self,
-        mut tx: RoochTransaction,
-    ) -> Result<ExecuteTransactionResult> {
-        let _ = tx.tx_hash(); // cache tx_hash
+    pub fn execute_as_result(&mut self, tx: RoochTransaction) -> Result<ExecuteTransactionResult> {
         let verified_tx = self.executor.validate_l2_tx(tx)?;
         self.execute_verified_tx_as_result(verified_tx)
     }