test(starknet_api): remove the version argument from l1 handler tx args

[ArniStarkware]

No description provided.

[reviewable-StarkWare]

This change is 

[ArniStarkware]

• chore(starknet_api): create a new method for l1 handler using version 0 #2702
• test(starknet_api): remove the version argument from l1 handler tx args #2699 👈 (View in Graphite)
• test(starknet_api): set l1 handler tx version to default zero #2698
• test(starknet_api): use const for l1 handler tx version #2763
• main

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

[github-actions]

Artifacts upload workflows:

• Started at 2024-12-16T13:50:20Z
• Started at 2024-12-16T14:00:45Z
• Started at 2024-12-16T15:51:43Z
• Started at 2024-12-17T09:08:42Z
• Started at 2024-12-17T09:32:08Z
• Started at 2024-12-18T13:45:35Z
• Started at 2024-12-19T12:22:30Z
• Started at 2024-12-22T08:40:29Z
• Started at 2024-12-22T09:02:39Z

[github-actions]

Benchmark movements:
tree_computation_flow performance improved 😺
tree_computation_flow time: [34.718 ms 34.758 ms 34.801 ms]
change: [-3.8599% -2.3599% -1.0338%] (p = 0.00 < 0.05)
Performance has improved.
Found 8 outliers among 100 measurements (8.00%)
3 (3.00%) low mild
4 (4.00%) high mild
1 (1.00%) high severe

[github-actions]

Benchmark movements:
tree_computation_flow performance regressed!
tree_computation_flow time: [35.530 ms 35.954 ms 36.477 ms]
change: [+1.5098% +2.6556% +4.0807%] (p = 0.00 < 0.05)
Performance has regressed.
Found 14 outliers among 100 measurements (14.00%)
7 (7.00%) high mild
7 (7.00%) high severe

[giladchase]

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

---

crates/starknet_api/src/test_utils/l1_handler.rs line 29 at r2 (raw file):

        }
    }
}

🔥

Code quote:

impl Default for L1HandlerTxArgs {
    fn default() -> Self {
        L1HandlerTxArgs {
            version: L1HandlerTransaction::VERSION,
            nonce: Nonce::default(),
            contract_address: ContractAddress::default(),
            entry_point_selector: EntryPointSelector::default(),
            calldata: Calldata::default(),
            tx_hash: TransactionHash::default(),
            paid_fee_on_l1: Fee::default(),
        }
    }
}

[ArniStarkware]

Merge activity

• Dec 22, 3:39 AM EST: A user started a stack merge that includes this pull request via Graphite.
• Dec 22, 4:03 AM EST: Graphite rebased this pull request as part of a merge.
• Dec 22, 4:23 AM EST: A user merged this pull request with Graphite.

[github-actions]

Benchmark movements:
tree_computation_flow performance regressed!
tree_computation_flow time: [34.759 ms 35.100 ms 35.479 ms]
change: [+1.3862% +2.3499% +3.4398%] (p = 0.00 < 0.05)
Performance has regressed.
Found 21 outliers among 100 measurements (21.00%)
6 (6.00%) high mild
15 (15.00%) high severe