apollo_gateway: test account tx in mempool influence on validation

[itamar-starkware]

TL;DR

Added a test to verify that validation is skipped for account transactions already in the mempool.

What changed?

Added a new test case test_account_tx_in_mempool_influence_on_validation that verifies the behavior of transaction validation based on whether an account transaction is already in the mempool. The test covers three scenarios:

1. When the transaction is in the mempool (validation should be skipped)
2. When the transaction is not in the mempool (validation should run)
3. When there's a mempool error (validation should run)

How to test?

Run the new test with:

cargo test -p apollo_gateway test_account_tx_in_mempool_influence_on_validation

Why make this change?

This test ensures that the stateful transaction validator correctly optimizes validation by skipping redundant validation for transactions already present in the mempool, while still properly validating transactions that aren't in the mempool or when mempool errors occur.

[reviewable-StarkWare]

This change is 

[itamar-starkware]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• apollo_gateway: test account tx in mempool influence on validation #9950 👈 (View in Graphite)
• apollo_gateway: extract async mempool query from blocking task #9941
• apollo_gateway: use get_nonce outside extract_state_nonce_and_run_validations #10009
• apollo_gateway: add get_nonce function to StatefulValidator trait #10008
• apollo_gateway: insert stateful_tx_validator creating to ProcessTxBlockingTask constructor #10007
• apollo_gateway: split instantiate_validator #10123
• apollo_gateway: move instantiate_validator to be async #10122
• apollo_gateway: move get_latest_block_info to be async #10121
• apollo_gateway: move get_block_info to be async #10120
• apollo_gateway: move get_state_reader_from_latest_block to be async #10119
• main-v0.14.1

This stack of pull requests is managed by Graphite. Learn more about stacking.

[ArniStarkware]

crates/apollo_gateway/src/stateful_transaction_validator_test.rs line 444 at r1 (raw file):

#[rstest]
#[case(Ok(true), false)] // account tx in mempool → skip validation

Is that arrow symbol standard ASCII? I assume not. Please refrain from using it if this is the case.

Suggestion:

account tx in mempool -> skip validation