starknet_os: os resources test - remove all fee transfer syscalls#14137
Conversation
PR SummaryLow Risk Overview A pinned Reviewed by Cursor Bugbot for commit ba9b395. Bugbot is set up for automated code reviews on this repo. Configure here. |
0d44efb to
cd7ac82
Compare
00a30cb to
6b8d3d4
Compare
cd7ac82 to
72d6d62
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 72d6d62. Configure here.
72d6d62 to
ba9b395
Compare
6b8d3d4 to
755576b
Compare
Yoni-Starkware
left a comment
There was a problem hiding this comment.
@Yoni-Starkware made 1 comment.
Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on dorimedini-starkware).
crates/starknet_os_flow_tests/src/os_resources_test.rs line 80 at r2 (raw file):
const SYSCALLS_WITH_LINEAR_FACTOR: [Selector; 2] = [Selector::Deploy, Selector::MetaTxV0]; /// Expected syscalls in the fee transfer call. Should be removed from the list of syscalls during
You can run this tx with trivial fee bounds and make this test a bit cleaner


No description provided.