build(fee): implement methods on resource bounds

[nimrod-starkware]

This change is 

[nimrod-starkware]

• build(fee): add events to l2 cost #462
• build(fee): add code to l2 cost #461
• build(fee): add calldata and signature l2 cost #460
• build(fee): tx_resources depend on resource bounds signature #455
• build(fee): sn_resources depend on resource bounds signature #454
• build(fee): implement methods on resource bounds #600 👈
• refactor(fee): remove more occurnces of resource mapping #547 : 1 other dependent PR (#557 )
• build(fee): tx_context and create_tx_info cannot_fail #538
• refactor(fee): use valid resource bounds in blockifier #537
• refactor(fee): tx_args holds valid resource bounds #508 : 1 other dependent PR (#509 )
• refactor(fee): block context creator may fail #507
• refactor(fee): transaction info creator may fail #506
• build(fee): current tx_info holds validResourceBounds #505
• refactor(fee): remove resource mapping usage in rpc and adjust to use all resources #580
• main

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

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

[codecov]

Codecov Report

Attention: Patch coverage is 61.05263% with 37 lines in your changes missing coverage. Please review.

Project coverage is 75.72%. Comparing base (c65f0b9) to head (f357376).

Files with missing lines	Patch %	Lines
crates/blockifier/src/transaction/objects.rs	44.64%	31 Missing ⚠️
crates/starknet_api/src/transaction.rs	45.45%	6 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #600      +/-   ##
==========================================
- Coverage   75.83%   75.72%   -0.12%     
==========================================
  Files         352       85     -267     
  Lines       37423    10984   -26439     
  Branches    37423    10984   -26439     
==========================================
- Hits        28381     8318   -20063     
+ Misses       6726     2242    -4484     
+ Partials     2316      424    -1892     

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

[nimrod-starkware]

Reviewable status: 0 of 7 files reviewed, 2 unresolved discussions

---

crates/blockifier/src/transaction/objects.rs line 585 at r1 (raw file):

type FeeCheckResult<T> = Result<T, FeeCheckError>;
// Better name for the trait?

?

Code quote:

// Better name for the trait?

---

crates/blockifier/src/transaction/objects.rs line 621 at r1 (raw file):

                l1_data_gas,
            }) => {
                // Does the order matter?

If more than one resource has been exceeded, the user won't be able to know that.
Maybe instead we should add only one new error variant which will hold the gas vector consumed by the user and the resources he signed on. If more than one resource has exceeded, he could see that

Code quote:

/ Does the order matter?