search

Open-Systems-Pharmacology / OSPSuite-R

R package for the OSPSuite
https://www.open-systems-pharmacology.org/OSPSuite-R/
Other
28 stars 12 forks source link

Tests are missing in the Linux package #1478

Closed Yuri05 closed 1 month ago

Yuri05 commented 1 month ago

Testthat tests are missing in the Linux package

Felixmil commented 1 month ago

What do you mean ? Tests are not supposed to be present in packaged packages:

PavelBal commented 1 month ago

I think we had this discussion already :) Actually for validation purposes, supplying the tests with the package would be a good idea. Is this doable?

Felixmil commented 1 month ago

This is possible with an installation option:

devtools::install_github("tidyverse/ggplot2", INSTALL_opts = "--install-tests")
testthat::test_package("ggplot2")

But I can also change the building workflow to include tests with:

devtools::build(...,  args=c("--preclean", "--install-tests"))

This way, user can test package installed from binary

Yuri05 commented 1 month ago

But I can also change the building workflow to include tests with:

devtools::build(...,  args=c("--preclean", "--install-tests"))

Please do. Having testthat-tests included in (at least) the Linux version is mandatory on our side due to the internal package validation procedure.

Felixmil commented 1 month ago

Proposed in this PR