search

PcAux-Package / PcAux

This is the repository for the PcAux project.
GNU General Public License v3.0
4 stars 5 forks source link

Use "testthat" to run unit tests #33

Closed ppanko closed 1 year ago

ppanko commented 5 years ago

The testthat package is convenient for writing self-documenting unit tests that can be run during the package checking process (using either R CMD check or devtools::check()). I suggest we adjust our unit tests to incorporate testthat.

kylelang commented 5 years ago

In principle, I certainly agree. I would very much prefer to use testthat and run proper unit tests. Unfortunately, the tests we currently have are not true unit tests (they simultaneously test many computational units). Also, the majority of our tests are interactive and cannot be run in a "batch" style. So, moving to using testthat would require developing an entirely new set of tests. The move is still a good idea, but it will be a lot of work.

ppanko commented 5 years ago

I agree, I don't think PcAux -in its current form- can move entirely to a batch testing model.

kylelang commented 1 year ago

This project is no longer actively developed here, so I'm archiving this repository. In preparation thereof, I'm closing all open issues.