starknet_os_flow_tests: fuzz test orchestrator address in contract state#12956
Conversation
Yoni-Starkware
left a comment
There was a problem hiding this comment.
@Yoni-Starkware reviewed 9 files and all commit messages, and made 1 comment.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on dorimedini-starkware).
Yoni-Starkware
left a comment
There was a problem hiding this comment.
@Yoni-Starkware made 1 comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on dorimedini-starkware).
crates/blockifier_test_utils/resources/feature_contracts/cairo0/fuzz_revert.cairo line 26 at r1 (raw file):
} return (); }
Suggestion:
@constructor
func constructor{syscall_ptr: felt*, pedersen_ptr: HashBuiltin*, range_check_ptr}(
maybe_orchestrator_address: felt,
) {
if (maybe_orchestrator_address != 0) {
initialize(maybe_orchestrator_address);
return ()
}
return ();
}
Yoni-Starkware
left a comment
There was a problem hiding this comment.
@Yoni-Starkware resolved 1 discussion.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on dorimedini-starkware).
9a27943 to
fee963a
Compare
efedd19 to
9cd6151
Compare
fee963a to
6b0d614
Compare
9cd6151 to
0804e95
Compare
dorimedini-starkware
left a comment
There was a problem hiding this comment.
@dorimedini-starkware reviewed 1 file and all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on dorimedini-starkware).
0804e95 to
39d07ec
Compare
Merge activity
|

Note
Medium Risk
Updates the fuzz-test feature contracts’ ABI/constructor behavior and regenerates compiled artifacts, which changes compiled class hashes and can break downstream tests or assumptions that depend on exact hashes/entrypoints.
Overview
Adds an
orchestrator_addressfield to thefuzz_revertandfuzz_revert_2feature contracts (Cairo0 and Cairo1) and introduces aninitializeentrypoint that writes this value to storage.The constructor now accepts an optional orchestrator address and, when non-zero, calls
initializeto persist it; all compiled artifacts (compiled/*.json, Sierra/CASM) are regenerated accordingly. Expected compiled class hashes forFUZZ_TEST*incontracts.rsare updated to match the new contract classes.Written by Cursor Bugbot for commit efedd19. This will update automatically on new commits. Configure here.