Skip to content

fix: enable edge coverage only when needed #11041

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.

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

Merged
merged 3 commits into from
Jul 20, 2025

Conversation

grandizzy
Copy link
Collaborator

@grandizzy grandizzy commented Jul 18, 2025

Motivation

  • enable edge coverage inspector only for invariant runners and per invariant test config (can be changed with inline config)
  • enabled on coverage guided fuzzing (corpus dir enabled) or explicitly setting show_edge_coverage setting - useful for benchmarking

Solution

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

DaniPopes
DaniPopes previously approved these changes Jul 18, 2025
@DaniPopes DaniPopes marked this pull request as ready for review July 18, 2025 20:10
@DaniPopes
Copy link
Member

Anything else to do here?

@grandizzy
Copy link
Collaborator Author

Anything else to do here?

Per @0xalpharush would be good to also add a config to enable the edge coverage and metrics, but not use corpus/mutations, useful for benchmarking.

mattsse
mattsse previously approved these changes Jul 19, 2025
zerosnacks
zerosnacks previously approved these changes Jul 19, 2025
@grandizzy grandizzy marked this pull request as draft July 20, 2025 06:36
@grandizzy
Copy link
Collaborator Author

going to make it adhere to show_metrics flag and move logic to enable edge inspector only for invariant executor

@grandizzy grandizzy dismissed stale reviews from zerosnacks, mattsse, and DaniPopes via 3eb0845 July 20, 2025 12:27
@grandizzy grandizzy force-pushed the optional-edge-inspector branch from f4bd627 to 3eb0845 Compare July 20, 2025 12:27
@grandizzy grandizzy force-pushed the optional-edge-inspector branch from 3eb0845 to 0d9b0f5 Compare July 20, 2025 12:44
@grandizzy
Copy link
Collaborator Author

going to make it adhere to show_metrics flag and move logic to enable edge inspector only for invariant executor

@0xalpharush 0d9b0f5

@grandizzy grandizzy changed the title fix: enable edge coverage inspector only if corpus set fix: enable edge coverage inspector only when needed Jul 20, 2025
@grandizzy grandizzy changed the title fix: enable edge coverage inspector only when needed fix: enable edge coverage only when needed Jul 20, 2025
@grandizzy grandizzy marked this pull request as ready for review July 20, 2025 13:08
@grandizzy grandizzy merged commit 679c2ec into foundry-rs:master Jul 20, 2025
22 checks passed
@github-project-automation github-project-automation bot moved this to Done in Foundry Jul 20, 2025
@grandizzy grandizzy deleted the optional-edge-inspector branch July 20, 2025 13:30
@grandizzy grandizzy self-assigned this Jul 22, 2025
@grandizzy grandizzy moved this from Done to Completed in Foundry Jul 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

4 participants