Skip to content
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

NEXT-38121 - Update readme again #38

Merged
merged 1 commit into from
Sep 10, 2024
Merged

Conversation

MalteJanz
Copy link
Contributor

Follow up on #37

@MalteJanz MalteJanz added the documentation Improvements or additions to documentation label Sep 9, 2024
@MalteJanz MalteJanz self-assigned this Sep 9, 2024
Copy link

github-actions bot commented Sep 9, 2024

Summary of the total line code coverage for the whole codebase

Total lines Covered Skipped % (pr) % (main)
2154 1361 793 63.18 63.28
Summary of each file (click to expand)
File Total lines Covered Skipped %
src/api/filter.rs 171 168 3 98.25
src/api/mod.rs 476 291 185 61.13
src/cli.rs 45 31 14 68.89
src/config_file.rs 76 56 20 73.68
src/data/export.rs 136 0 136 0.00
src/data/import.rs 185 0 185 0.00
src/data/transform/mod.rs 336 259 77 77.08
src/data/transform/script.rs 292 258 34 88.36
src/data/validate.rs 298 298 0 100.00
src/main.rs 139 0 139 0.00
More details (click to expand)

Download full HTML report

You can download the full HTML report here: click to download
Hint: You need to extract it locally and open the index.html, there you can see which lines are not covered in each file.

You can also generate these reports locally

For that, you need to install cargo-llvm-cov, then you can run:

cargo llvm-cov --all-features --no-fail-fast --open

Hint: There are also other ways to see code coverage in Rust. For example with RustRover, you can execute tests with coverage generation directly in the IDE.

Remember

Your tests should be meaningful and not just be written to raise the coverage.
Coverage is just a tool to detect forgotten code paths you may want to think about, not your instructor to write tests

@MalteJanz MalteJanz merged commit d10620c into main Sep 10, 2024
3 checks passed
@MalteJanz MalteJanz deleted the next-38121/update-readme-again branch September 10, 2024 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants