[Feature]: Include e2e tests in coverage

OpenZeppelin / rust-contracts-stylus

A library for secure smart contract development written in Rust

MIT License

54 stars 10 forks source link

[Feature]: Include e2e tests in coverage #92

Open alexfertel opened 3 months ago

alexfertel commented 3 months ago

What is the feature you would like to see?

We don't currently include e2e tests in coverage because some setup is required to run them. Specifically, you need solc and nitro-testnode to be available.

We should either create a new workflow that uploads this to codecov or update the current coverage workflow to include e2e tests.

Contribution Guidelines

[X] I agree to follow this project's Contribution Guidelines

alexfertel commented 3 months ago

Welp, it turns out this should be pretty easy https://docs.codecov.com/docs/merging-reports#multiple-languages

alexfertel commented 2 months ago

The coverage is not working, because the wasm binaries have to first be instrumented in order to collect the coverage. This means two things:

We need to compile them with the -C instrument-coverage flag.
For some reason, events get included in coverage.