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

increase codecov to 100% #74

Open
erblast opened this issue Jan 20, 2025 · 5 comments
Open

increase codecov to 100% #74

erblast opened this issue Jan 20, 2025 · 5 comments

Comments

@erblast
Copy link
Collaborator

erblast commented Jan 20, 2025

identify untested source code lines and add unit tests

cov <- covr::package_coverage()
cov
covr::file_report(cov, "R/simaerep.R") # generate codecoverage report for source code file
covr::percent_coverage(cov) == 100 # this tests whether code coverage is at 100%

add github action.

example:
https://github.com/IMPALA-Consortium/gsm.simaerep/blob/main/.github/workflows/test-coverage.yaml

@erblast
Copy link
Collaborator Author

erblast commented Jan 20, 2025

open PR to branch 74_100perc_code_coverage

@erblast
Copy link
Collaborator Author

erblast commented Jan 20, 2025

we might identify unnecessary code that can be deleted as well

@erblast
Copy link
Collaborator Author

erblast commented Jan 22, 2025

code coverage for lint.R will be adressed here #75

@erblast
Copy link
Collaborator Author

erblast commented Feb 7, 2025

code coverage increased from 94.3% --> 96.4%

still potential to increase for these files:

R/simaerep_plot.R: 95.71%
R/simaerep.R: 95.82%

@erblast
Copy link
Collaborator Author

erblast commented Feb 7, 2025

code for parallelisation is not yet unit tested

a lot of it might be redundant when the next version of purrr is released

#80

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

No branches or pull requests

1 participant