Skip to content

refactor(fuzz:ledger): Simplify and improve ledger fuzzer #925

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 1 commit into
base: ink/more-fuzzer-cli-subcmds
Choose a base branch
from
Draft

refactor(fuzz:ledger): Simplify and improve ledger fuzzer #925

wants to merge 1 commit into from

Conversation

InKryption
Copy link
Contributor

  • Removes global state, simplifying executed_actions in the process.
  • Remove the possibility of non-exhaustiveness.
  • Now accepts a logger instead of using debug.print.
  • Replace managed types with unmanaged ones.
  • Removes dependency on the c_allocator and fix leaks.
  • Parameterize the data folder instead of hardcoding it.

@github-project-automation github-project-automation bot moved this to 🏗 In progress in Sig Sep 2, 2025
This was referenced Sep 2, 2025
Merged
Open
@InKryption Graphite App
Copy link
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

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

@InKryption InKryption mentioned this pull request Sep 2, 2025
Draft
@codecov Codecov
Copy link

codecov bot commented Sep 2, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 92.56198% with 9 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/ledger/fuzz.zig 92.56% 9 Missing ⚠️
Files with missing lines Coverage Δ
src/ledger/fuzz.zig 93.37% <92.56%> (-1.77%) ⬇️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@InKryption InKryption force-pushed the ink/more-fuzzer-cli-subcmds branch from 1ffd932 to cd1aab2 Compare September 2, 2025 16:26
@InKryption InKryption force-pushed the ink/rework-ledger-fuzzer branch from 4feb7a3 to 3c5fe66 Compare September 2, 2025 16:26
@InKryption InKryption force-pushed the ink/more-fuzzer-cli-subcmds branch from cd1aab2 to ffa33fb Compare September 2, 2025 17:09
@InKryption InKryption force-pushed the ink/rework-ledger-fuzzer branch from 3c5fe66 to fef90ba Compare September 2, 2025 17:09
@InKryption InKryption force-pushed the ink/more-fuzzer-cli-subcmds branch from ffa33fb to 94dcfbc Compare September 3, 2025 00:32
@InKryption InKryption force-pushed the ink/rework-ledger-fuzzer branch 2 times, most recently from 57d8ce0 to 9352766 Compare September 3, 2025 00:42
@InKryption InKryption force-pushed the ink/more-fuzzer-cli-subcmds branch from 94dcfbc to 03543cc Compare September 3, 2025 00:42
@InKryption InKryption self-assigned this Sep 3, 2025
@InKryption InKryption marked this pull request as draft September 3, 2025 13:28
@InKryption InKryption force-pushed the ink/rework-ledger-fuzzer branch from 9352766 to dd62e50 Compare September 3, 2025 16:09
@InKryption InKryption force-pushed the ink/more-fuzzer-cli-subcmds branch 2 times, most recently from b19c9d0 to 1268b5c Compare September 5, 2025 15:03
@InKryption InKryption force-pushed the ink/rework-ledger-fuzzer branch from dd62e50 to 572b430 Compare September 5, 2025 15:03
@InKryption InKryption force-pushed the ink/more-fuzzer-cli-subcmds branch from 1268b5c to b00cd3e Compare September 8, 2025 13:29
@InKryption InKryption force-pushed the ink/rework-ledger-fuzzer branch from 572b430 to 336a193 Compare September 8, 2025 13:29
@InKryption InKryption force-pushed the ink/more-fuzzer-cli-subcmds branch from b00cd3e to 9c6eb51 Compare September 23, 2025 11:54
@InKryption InKryption force-pushed the ink/rework-ledger-fuzzer branch from 336a193 to 86694e5 Compare September 23, 2025 11:54
@InKryption InKryption force-pushed the ink/more-fuzzer-cli-subcmds branch from 9c6eb51 to de736d8 Compare September 29, 2025 17:58
@InKryption InKryption force-pushed the ink/rework-ledger-fuzzer branch from 86694e5 to 124d08a Compare September 29, 2025 17:58
@InKryption InKryption force-pushed the ink/more-fuzzer-cli-subcmds branch from de736d8 to 46d5718 Compare September 30, 2025 15:42
@InKryption InKryption force-pushed the ink/rework-ledger-fuzzer branch 2 times, most recently from 34b6f77 to 8b2f1dc Compare October 1, 2025 21:16
@InKryption InKryption force-pushed the ink/more-fuzzer-cli-subcmds branch from 46d5718 to 634a1e1 Compare October 1, 2025 21:16
@InKryption
Significantly rework the ledger fuzzer
9df2420 
* Removes global state, simplifying `executed_actions` in the process.
* Remove the possibility of non-exhaustiveness.
* Now accepts a logger instead of using `debug.print`.
* Replace managed types with unmanaged ones.
* Removes dependency on the c_allocator and fix leaks.
* Parameterize the data folder instead of hardcoding it.
@InKryption InKryption force-pushed the ink/more-fuzzer-cli-subcmds branch from 634a1e1 to 3c5582c Compare October 1, 2025 22:30
@InKryption InKryption force-pushed the ink/rework-ledger-fuzzer branch from 8b2f1dc to 9df2420 Compare October 1, 2025 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ultd ultd Awaiting requested review from ultd ultd is a code owner

@dnut dnut Awaiting requested review from dnut dnut is a code owner

@yewman yewman Awaiting requested review from yewman yewman is a code owner

Assignees

@InKryption InKryption

Labels
None yet
Projects
Sig
Status: 🏗 In progress
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@InKryption