refactor(starknet_api): rename executable tx to account tx

[yair-starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[yair-starkware]

• feat(batcher): validate l1handler txs in validate_block #1921
• feat(batcher): fetch L1Handler txs in propose #1907
• refactor(batcher): create the tx provider in the batcher #1879
• feat(batcher): add l1-provider client #1870
• chore(l1-provider): use the correct tx type + fix crate package #1791
• refactor: wrap executable tx in account/l1handler enum #1788
• refactor(starknet_api): rename executable tx to account tx #1787 👈
• main

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

Join @yair-starkware and the rest of your teammates on Graphite

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 15 lines in your changes missing coverage. Please review.

Project coverage is 50.25%. Comparing base (e3165c4) to head (7ad1b50).
Report is 232 commits behind head on main.

Files with missing lines	Patch %	Lines
crates/starknet_api/src/executable_transaction.rs	0.00%	15 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #1787       +/-   ##
===========================================
+ Coverage   40.10%   50.25%   +10.14%     
===========================================
  Files          26       25        -1     
  Lines        1895     2553      +658     
  Branches     1895     2553      +658     
===========================================
+ Hits          760     1283      +523     
- Misses       1100     1205      +105     
- Partials       35       65       +30     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ArniStarkware]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @alonh5)

[alonh5]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @yair-starkware)

[alonh5]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @yair-starkware)

[ArniStarkware]

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @yair-starkware)

[ArniStarkware]

crates/starknet_api/src/executable_transaction.rs line 128 at r2 (raw file):

        sender_address: ContractAddress,
    ) -> AccountTransaction {
        AccountTransaction::Invoke(crate::executable_transaction::InvokeTransaction {

there should be a merge conflict here.

Code quote:

    // TODO(Arni): Update the function to support all transaction types.
    pub fn new_from_rpc_tx(
        rpc_tx: RpcTransaction,
        tx_hash: TransactionHash,
        sender_address: ContractAddress,
    ) -> AccountTransaction {
        AccountTransaction::Invoke(crate::executable_transaction::InvokeTransaction {

[ArniStarkware]

Reviewed 1 of 1 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @alonh5)

[yair-starkware]

Merge activity

• Nov 7, 4:58 AM EST: A user merged this pull request with Graphite.