Skip to content

chore: turn forge coverage and forge snapshot into forge test flags (--coverage, --snapshot)Β #9528

Open
@zerosnacks

Description

@zerosnacks

Component

Forge

Describe the feature you would like

Proposal: merge forge coverage and forge snapshot into forge test, expose them with feature flags (--coverage, --snapshot). This is in line with other reports like --gas-report.

This would simplify forge test compatibility issues described in #9525 and make it easier to flag incompatible flags (current limitation because of TestArgs inheritance)

We should also refactor run_tests method in forge test into helpers

This is a significant breaking change for scripted workflows i.e. https://github.com/atarpara/foundry-snapshot-diff and CI's of users.

Metadata

Metadata

Assignees

No one assigned

    Labels

    A-internalsArea: internalsT-choreType: choreT-likely-breakingType: requires changes that can be breakingT-to-discussType: requires discussion

    Type

    No type

    Projects

    Status

    Todo

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions