-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test: move nonce storage_key and compiled_class_hash macros to starknet api test utils #760
test: move nonce storage_key and compiled_class_hash macros to starknet api test utils #760
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @ArniStarkware and the rest of your teammates on Graphite |
Benchmark movements: |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #760 +/- ##
==========================================
+ Coverage 74.44% 74.51% +0.07%
==========================================
Files 364 366 +2
Lines 38395 38587 +192
Branches 38395 38587 +192
==========================================
+ Hits 28583 28754 +171
- Misses 7493 7506 +13
- Partials 2319 2327 +8 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewable status: 0 of 15 files reviewed, all discussions resolved (waiting on @MohammadNassar1)
crates/blockifier/src/test_utils.rs
line 114 at r1 (raw file):
pub const CURRENT_BLOCK_TIMESTAMP_FOR_VALIDATE: u64 = 1069200; // TODO(Yoni, 1/1/2025): move to SN API.
Removing old todos.
Code quote:
// TODO(Yoni, 1/1/2025): move to SN API.
baf9c6d
to
f22eca1
Compare
a2e82e5
to
ae4ae2f
Compare
Benchmark movements: |
c55cc6a
to
21143c4
Compare
ae4ae2f
to
93a2b3d
Compare
Benchmark movements: |
21143c4
to
dd33fdb
Compare
93a2b3d
to
5276c8d
Compare
dd33fdb
to
4437f42
Compare
5276c8d
to
a98cbd6
Compare
4437f42
to
2466922
Compare
a98cbd6
to
5e10607
Compare
Benchmark movements: |
2466922
to
49b511a
Compare
5e10607
to
65970ba
Compare
Benchmark movements: |
…et api test utils
65970ba
to
f1c4d52
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 8 of 15 files at r1, 1 of 3 files at r2, 6 of 6 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @MohammadNassar1)
This change is