googleapis / googleapis/google-cloud-rust

Reconsider code coverage approach

Open
#3,886 1 comment 0 reactions 0 assignees View on GitHub
type: cleanup
Dominant language
Rust
Stars
955
Forks
144
Avg merge
1d 5h
Merged PRs (30d)
279

Description

We can close this issue when we have documented why we should not do these things **or** we have implemented them.

With the code coverage build moving to GCB (Google Cloud Build, see #3880) it may be time to reconsider how we run this build. We use `cargo tarpaulin` because it was easy to set up, and was not too slow when compared to `cargo llvm-cov`. That may have changed, as the code size has grown.

We also exclude all the integration tests because we could not run them in GHA (GitHub Actions). That is no longer true.

In summary:

- [x] Use `cargo llvm-cov` to run coverage builds.
- [ ] Enable as many integration tests as possible as part of the coverage builds.
- [ ] Trim the list of directories excluded from code coverage analysis.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.