refactor: AssembledTransaction.simulate and AssembledTransactionAsync.simulate will no longer restore the state for read-only transactions. (#1026)

Author: overcat

CHANGELOG.md

@@ -5,6 +5,7 @@ Release History
 
 #### Update
 - feat: add `TransactionBuilder.append_payment_to_contract_op` and `TransactionBuilder.append_restore_asset_balance_entry_op` to send assets to contract accounts without relying on Stellar RPC. ([#1023](https://github.com/StellarCN/py-stellar-base/pull/1023))
+- refactor: `AssembledTransaction.simulate` and `AssembledTransactionAsync.simulate` will no longer restore the state for read-only transactions. ([#1026](https://github.com/StellarCN/py-stellar-base/pull/1026))
 
 ### Version 12.1.0
 

stellar_sdk/contract/assembled_transaction.py

@@ -88,7 +88,7 @@ def simulate(self, restore: bool = True) -> "AssembledTransaction":
         built_tx = self.transaction_builder.build()
         self.simulation = self.server.simulate_transaction(built_tx)
 
-        if restore and self.simulation.restore_preamble:
+        if restore and self.simulation.restore_preamble and not self.is_read_call():
             try:
                 self.restore_footprint(self.simulation.restore_preamble)
             except (

stellar_sdk/contract/assembled_transaction_async.py

@@ -88,7 +88,7 @@ async def simulate(self, restore: bool = True) -> "AssembledTransactionAsync":
         built_tx = self.transaction_builder.build()
         self.simulation = await self.server.simulate_transaction(built_tx)
 
-        if restore and self.simulation.restore_preamble:
+        if restore and self.simulation.restore_preamble and not self.is_read_call():
             try:
                 await self.restore_footprint(self.simulation.restore_preamble)
             except (