Skip to content

coverage-worker flag path construction fix#172

Merged
louismerlin merged 3 commits into
mainfrom
fix-coverage-worker
Jun 15, 2026
Merged

coverage-worker flag path construction fix#172
louismerlin merged 3 commits into
mainfrom
fix-coverage-worker

Conversation

@acsrlabs

Copy link
Copy Markdown
Collaborator

fixed the --coverage-worker of ziggy fuzz by changing the path construction so the correct path to the profraw files can be found.

fixes #171

@louismerlin

Copy link
Copy Markdown
Collaborator

Nice! Could you please add an e2e test? (which should fail on the previous version of the code)

@acsrlabs

Copy link
Copy Markdown
Collaborator Author

will do!

Comment thread src/bin/cargo-ziggy/fuzz.rs
Comment thread src/bin/cargo-ziggy/fuzz.rs
@chris-srlabs

Copy link
Copy Markdown
Collaborator

Nice! Could you please add an e2e test? (which should fail on the previous version of the code)

Something like cargo ziggy fuzz --coverage-worker --coverage-interval=0 --corpus-sync-interval=0 for the url example seems to work as an e2e test.

@chris-srlabs chris-srlabs requested a review from louismerlin June 15, 2026 07:11
@louismerlin louismerlin merged commit a3c0130 into main Jun 15, 2026
4 checks passed
@louismerlin louismerlin deleted the fix-coverage-worker branch June 15, 2026 07:42
@louismerlin

Copy link
Copy Markdown
Collaborator

Cool thank you @acsrlabs, and thank you @chris-srlabs for getting this through the finish line!

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.

coverage-worker not working during fuzzing

3 participants