refactor(katana): encapsulate all the tx execution logic into a single function

[kariy]

Description

put all the logic that is required for every tx execution (ie building the receipt, exec info, etc) into a single unit of execution (ie transact).

this allow us to simplify testing/benchmarking tx execution, bcs all the necessary steps are now included in a single function that we can easily test, without having to create the entire BlockExecutor trait.

Related issue

Tests

• Yes
• No, because they aren't needed
• No, because I need help

Added to documentation?

• README.md
• Dojo Book
• No documentation needed

Checklist

• I've formatted my code (scripts/prettier.sh, scripts/rust_fmt.sh, scripts/cairo_fmt.sh)
• I've linted my code (scripts/clippy.sh, scripts/docs.sh)
• I've commented my code
• I've requested a review after addressing the comments

[kariy]

• Add simple benchmark for the transact function #1969
• refactor(katana): encapsulate all the tx execution logic into a single function #1968 👈
• test(katana): provider test utils  #1967
• main

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

Join @kariy and the rest of your teammates on Graphite

[codecov]

Codecov Report

Attention: Patch coverage is 56.58915% with 56 lines in your changes are missing coverage. Please review.

Project coverage is 69.74%. Comparing base (49f0a4e) to head (87d6943).
Report is 1 commits behind head on main.

❗ Current head 87d6943 differs from pull request most recent head e202651

Please upload reports for the commit e202651 to get more accurate results.

Files	Patch %	Lines
crates/katana/storage/provider/src/test_utils.rs	0.00%	49 Missing ⚠️
...tana/executor/src/implementation/blockifier/mod.rs	79.16%	5 Missing ⚠️
...na/executor/src/implementation/blockifier/utils.rs	98.03%	1 Missing ⚠️
crates/katana/executor/src/utils.rs	80.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1968      +/-   ##
==========================================
- Coverage   69.81%   69.74%   -0.07%     
==========================================
  Files         313      314       +1     
  Lines       35562    35628      +66     
==========================================
+ Hits        24828    24850      +22     
- Misses      10734    10778      +44     

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

[kariy]

Merge activity

• May 16, 3:54 PM EDT: @kariy started a stack merge that includes this pull request via Graphite.
• May 16, 3:56 PM EDT: Graphite rebased this pull request as part of a merge.
• May 16, 3:58 PM EDT: @kariy merged this pull request with Graphite.