Skip to content

starknet_os_flow_tests: long fuzz test#12970

Merged
dorimedini-starkware merged 1 commit intomain-v0.14.2from
03-02-starknet_os_flow_tests_long_fuzz_test
Mar 9, 2026
Merged

starknet_os_flow_tests: long fuzz test#12970
dorimedini-starkware merged 1 commit intomain-v0.14.2from
03-02-starknet_os_flow_tests_long_fuzz_test

Conversation

@dorimedini-starkware
Copy link
Collaborator

@dorimedini-starkware dorimedini-starkware commented Mar 2, 2026

Note

Low Risk
Test-only change behind an opt-in feature flag; main risk is longer runtimes/resource usage if enabled in CI or local runs.

Overview
Adds a new long_fuzz_test Cargo feature that gates a heavy fuzz test suite in starknet_os_flow_tests.

Replaces the previously commented/shorter fuzz test with a #[cfg(feature = "long_fuzz_test")] module that expands the seed space via 4-digit base-10 combinatorics (10k cases) while keeping the existing daily-seed fuzz test unchanged.

Written by Cursor Bugbot for commit e83105d. This will update automatically on new commits. Configure here.

@reviewable-StarkWare
Copy link

This change is Reviewable

Copy link
Collaborator Author

dorimedini-starkware commented Mar 2, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@dorimedini-starkware dorimedini-starkware force-pushed the 03-02-starknet_os_flow_tests_long_fuzz_test branch from a60eeec to be4d820 Compare March 2, 2026 18:36
@dorimedini-starkware dorimedini-starkware force-pushed the 03-02-starknet_os_flow_tests_daily_fuzz_test branch from 166334b to eee7604 Compare March 2, 2026 19:19
@dorimedini-starkware dorimedini-starkware force-pushed the 03-02-starknet_os_flow_tests_long_fuzz_test branch from be4d820 to 2dec630 Compare March 2, 2026 19:19
@dorimedini-starkware dorimedini-starkware force-pushed the 03-02-starknet_os_flow_tests_daily_fuzz_test branch from eee7604 to 7917b7c Compare March 3, 2026 11:10
@dorimedini-starkware dorimedini-starkware force-pushed the 03-02-starknet_os_flow_tests_long_fuzz_test branch from 2dec630 to 7f8357c Compare March 3, 2026 11:10
@dorimedini-starkware dorimedini-starkware changed the base branch from 03-02-starknet_os_flow_tests_daily_fuzz_test to graphite-base/12970 March 4, 2026 08:07
@dorimedini-starkware dorimedini-starkware force-pushed the 03-02-starknet_os_flow_tests_long_fuzz_test branch from 7f8357c to 9521370 Compare March 4, 2026 17:33
@dorimedini-starkware dorimedini-starkware changed the base branch from graphite-base/12970 to 03-02-starknet_os_flow_tests_daily_fuzz_test March 4, 2026 17:33
Copy link
Collaborator

@Yoni-Starkware Yoni-Starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

@Yoni-Starkware reviewed 1 file and all commit messages, and made 2 comments.
Reviewable status: 1 of 2 files reviewed, 1 unresolved discussion (waiting on dorimedini-starkware).


crates/starknet_os_flow_tests/src/fuzz_tests.rs line 967 at r2 (raw file):

    #[rstest]
    #[tokio::test]
    async fn test_cairo1_revert_fuzz(

Suggestion:

test_fuzz

Copy link
Collaborator Author

@dorimedini-starkware dorimedini-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dorimedini-starkware resolved 1 discussion.
Reviewable status: 1 of 2 files reviewed, all discussions resolved (waiting on dorimedini-starkware).

Copy link
Collaborator

@Yoni-Starkware Yoni-Starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Yoni-Starkware reviewed 1 file.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on dorimedini-starkware).

@dorimedini-starkware dorimedini-starkware force-pushed the 03-02-starknet_os_flow_tests_daily_fuzz_test branch from dba2e1f to 448795c Compare March 9, 2026 14:49
@dorimedini-starkware dorimedini-starkware force-pushed the 03-02-starknet_os_flow_tests_long_fuzz_test branch from a867150 to 0d852a4 Compare March 9, 2026 14:49
Copy link
Collaborator Author

@dorimedini-starkware dorimedini-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dorimedini-starkware reviewed 1 file and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on dorimedini-starkware).

@graphite-app graphite-app bot changed the base branch from 03-02-starknet_os_flow_tests_daily_fuzz_test to graphite-base/12970 March 9, 2026 16:28
@dorimedini-starkware dorimedini-starkware force-pushed the 03-02-starknet_os_flow_tests_long_fuzz_test branch from 0d852a4 to e83105d Compare March 9, 2026 18:02
@graphite-app graphite-app bot changed the base branch from graphite-base/12970 to main-v0.14.2 March 9, 2026 18:03
@graphite-app
Copy link

graphite-app bot commented Mar 9, 2026

Merge activity

  • Mar 9, 6:03 PM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.

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.

3 participants