Skip to content
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

Merged

Conversation

ArniStarkware
Copy link
Contributor

@ArniStarkware ArniStarkware commented Sep 9, 2024

This change is Reviewable

Copy link
Contributor Author

ArniStarkware commented Sep 9, 2024

@ArniStarkware ArniStarkware marked this pull request as ready for review September 9, 2024 14:18
Copy link

github-actions bot commented Sep 9, 2024

Benchmark movements:
tree_computation_flow performance improved 😺
tree_computation_flow time: [65.956 ms 66.025 ms 66.097 ms]
change: [-7.3185% -4.0643% -1.3902%] (p = 0.00 < 0.05)
Performance has improved.
Found 3 outliers among 100 measurements (3.00%)
3 (3.00%) high mild

Copy link

codecov bot commented Sep 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.51%. Comparing base (8aa893b) to head (f1c4d52).
Report is 18 commits behind head on main.

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.
📢 Have feedback on the report? Share it here.

Copy link
Contributor Author

@ArniStarkware ArniStarkware left a 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.

Copy link

Benchmark movements:
tree_computation_flow performance improved 😺
tree_computation_flow time: [66.397 ms 66.728 ms 67.293 ms]
change: [-8.4913% -5.0156% -1.7583%] (p = 0.00 < 0.05)
Performance has improved.
Found 10 outliers among 100 measurements (10.00%)
4 (4.00%) high mild
6 (6.00%) high severe

@ArniStarkware ArniStarkware force-pushed the arni/test_utils/account_tx/wrap_executable_tx branch from c55cc6a to 21143c4 Compare September 15, 2024 10:54
@ArniStarkware ArniStarkware force-pushed the arni/test_utils/move_test_object_creation_macros_to_snapi branch from ae4ae2f to 93a2b3d Compare September 15, 2024 10:55
Copy link

Benchmark movements:
tree_computation_flow performance improved 😺
tree_computation_flow time: [66.219 ms 66.375 ms 66.569 ms]
change: [-7.7345% -4.4004% -1.6073%] (p = 0.00 < 0.05)
Performance has improved.
Found 6 outliers among 100 measurements (6.00%)
2 (2.00%) high mild
4 (4.00%) high severe

@ArniStarkware ArniStarkware force-pushed the arni/test_utils/account_tx/wrap_executable_tx branch from 21143c4 to dd33fdb Compare September 15, 2024 13:07
@ArniStarkware ArniStarkware force-pushed the arni/test_utils/move_test_object_creation_macros_to_snapi branch from 93a2b3d to 5276c8d Compare September 15, 2024 13:07
@ArniStarkware ArniStarkware force-pushed the arni/test_utils/account_tx/wrap_executable_tx branch from dd33fdb to 4437f42 Compare September 15, 2024 14:48
@ArniStarkware ArniStarkware force-pushed the arni/test_utils/move_test_object_creation_macros_to_snapi branch from 5276c8d to a98cbd6 Compare September 15, 2024 14:48
@ArniStarkware ArniStarkware force-pushed the arni/test_utils/account_tx/wrap_executable_tx branch from 4437f42 to 2466922 Compare September 15, 2024 15:01
@ArniStarkware ArniStarkware force-pushed the arni/test_utils/move_test_object_creation_macros_to_snapi branch from a98cbd6 to 5e10607 Compare September 15, 2024 15:01
Copy link

Benchmark movements:
tree_computation_flow performance improved 😺
tree_computation_flow time: [65.797 ms 65.900 ms 66.040 ms]
change: [-8.1273% -4.9037% -2.1824%] (p = 0.00 < 0.05)
Performance has improved.
Found 6 outliers among 100 measurements (6.00%)
2 (2.00%) high mild
4 (4.00%) high severe

@ArniStarkware ArniStarkware force-pushed the arni/test_utils/account_tx/wrap_executable_tx branch from 2466922 to 49b511a Compare September 18, 2024 05:14
@ArniStarkware ArniStarkware force-pushed the arni/test_utils/move_test_object_creation_macros_to_snapi branch from 5e10607 to 65970ba Compare September 18, 2024 05:14
Copy link

Benchmark movements:
tree_computation_flow performance improved 😺
tree_computation_flow time: [66.773 ms 67.269 ms 68.026 ms]
change: [-9.3334% -5.3442% -1.8538%] (p = 0.00 < 0.05)
Performance has improved.
Found 6 outliers among 100 measurements (6.00%)
1 (1.00%) high mild
5 (5.00%) high severe

@ArniStarkware ArniStarkware force-pushed the arni/test_utils/move_test_object_creation_macros_to_snapi branch from 65970ba to f1c4d52 Compare September 18, 2024 13:35
@ArniStarkware ArniStarkware changed the base branch from arni/test_utils/account_tx/wrap_executable_tx to main September 18, 2024 13:35
Copy link
Contributor

@Yael-Starkware Yael-Starkware left a 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: :shipit: complete! all files reviewed, all discussions resolved (waiting on @MohammadNassar1)

@ArniStarkware ArniStarkware merged commit 3848c9f into main Sep 18, 2024
29 of 45 checks passed
@ArniStarkware ArniStarkware deleted the arni/test_utils/move_test_object_creation_macros_to_snapi branch September 18, 2024 13:57
@github-actions github-actions bot locked and limited conversation to collaborators Sep 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants