Skip to content

blockifier_test_utils: add option to deploy from zero in test accounts#14258

Open
dorimedini-starkware wants to merge 1 commit into
05-29-starknet_os_flow_tests_helper_function_to_declare_explicit_classesfrom
05-29-blockifier_test_utils_add_option_to_deploy_from_zero_in_test_accounts
Open

blockifier_test_utils: add option to deploy from zero in test accounts#14258
dorimedini-starkware wants to merge 1 commit into
05-29-starknet_os_flow_tests_helper_function_to_declare_explicit_classesfrom
05-29-blockifier_test_utils_add_option_to_deploy_from_zero_in_test_accounts

Conversation

@dorimedini-starkware
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Collaborator Author

dorimedini-starkware commented May 29, 2026

@reviewable-StarkWare
Copy link
Copy Markdown

This change is Reviewable

@cursor
Copy link
Copy Markdown

cursor Bot commented May 29, 2026

PR Summary

Low Risk
Changes are limited to test account ABIs and test expectations; production paths are untouched and default deploy behavior remains deploy-from-sender (false).

Overview
Test account contracts in blockifier_test_utils now expose deploy_from_zero on deploy_contract, forwarding it to the deploy syscall instead of hard-coding false. Cairo 0 and Cairo 1 dummy accounts were updated and the Cairo 0 artifact was recompiled; callers append false so behavior stays “deploy from sender.”

Invoke/deploy calldata in integration, blockifier, concurrency, stack-trace, and OS flow tests was extended with that flag. Expected hashes and metrics (proof-flow genesis root, STRK fee token address, fuzz contract addresses, bouncer migration gas snapshots, stack-trace PCs, builtin counts) were refreshed for the new bytecode/init ordering. .gitignore adds blockifier_test_venv/.

Reviewed by Cursor Bugbot for commit f886597. Bugbot is set up for automated code reviews on this repo. Configure here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants