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

NTR - prepare release 0.9.0 #42

Merged
merged 1 commit into from
Sep 18, 2024
Merged

NTR - prepare release 0.9.0 #42

merged 1 commit into from
Sep 18, 2024

Conversation

MalteJanz
Copy link
Contributor

No description provided.

Copy link
Member

@LarsKemper LarsKemper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

Copy link

Summary of the total line code coverage for the whole codebase

Total lines Covered Skipped % (pr) % (main)
2252 1455 797 64.61 64.61
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 77 57 20 74.03
src/data/export.rs 136 0 136 0.00
src/data/import.rs 185 0 185 0.00
src/data/transform/mod.rs 422 341 81 80.81
src/data/transform/script.rs 294 260 34 88.44
src/data/validate.rs 307 307 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 f71962f into main Sep 18, 2024
3 checks passed
@MalteJanz MalteJanz deleted the ntr/prepare-release-0-9-0 branch September 18, 2024 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants