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

[Feature Request] Add support for longtests in package_coverage() #555

Open
xec-cm opened this issue Dec 23, 2023 · 0 comments
Open

[Feature Request] Add support for longtests in package_coverage() #555

xec-cm opened this issue Dec 23, 2023 · 0 comments

Comments

@xec-cm
Copy link

xec-cm commented Dec 23, 2023

Description

This feature would allow users to specify a new parameter, type = "longtests", in the package_coverage() function. This would be useful for Bioconductor users who want to divide their tests into two folders, one for normal tests and one for long tests that take longer to run.

Context

Currently, Bioconductor software packages run code in the tests subdirectory on a daily basis as part of the Bioconductor nightly builds. The maximum amount of time that R CMD check is allowed to spend on each package is 40 minutes. However, some package developers may wish to implement tests that will be too long to run in the context of the nightly builds. To address this issue, Bioconductor users can set up “long tests” in their package and add the package to the Bioconductor Long Tests builds. The code in the longtests subdirectory will be run once a week (every Saturday) as part of the Long Tests builds and will be allowed to run for a maximum of 6 hours before a TIMEOUT is raised.

Justification

Adding support for the type = "longtests" parameter in the package_coverage() function would allow Bioconductor users to generate coverage reports for their long tests separately from their normal tests. This would make it easier for users to identify which tests are taking longer to run and optimize their testing process accordingly.

Implementation details

The type = "longtests" parameter would be added to the package_coverage() function. The longtests folder would have the same structure as the tests folder, but with a different name.

Example usage

covr::package_coverage(type = "longtests")
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