fix(`coverage`): separate dir + caching for coverage artifacts - Githubissues

foundry-rs / foundry

Foundry is a blazing fast, portable and modular toolkit for Ethereum application development written in Rust.

https://getfoundry.sh

Apache License 2.0

8.33k stars 1.76k forks source link

fix(`coverage`): separate dir + caching for coverage artifacts #9366

Open yash-atreya opened 1 day ago

yash-atreya commented 1 day ago

Motivation

Closes #8840. Addresses https://github.com/foundry-rs/foundry/issues/8840#issuecomment-2390792012. Somewhat addresses #8904 + https://github.com/foundry-rs/foundry/issues/8889 - by enabling cache and avoiding recompilation

Solution

Add a warning that optimizer settings have been disabled.
Write coverage and build-info artifacts to out/coverage and out/coverage/build-info.
Write compiler cache to cache/coverage/
Enabling caching for coverage artifacts to avoid recompilation

klkvr commented 11 hours ago

did you test that coverage is collected correctly even after multiple different builds? it doesn't look like we account for different source_ids here