gsucarrat / gets

R Package for General-to-Specific (GETS) modelling and Indicator Saturation (ISAT) methods
8 stars 5 forks source link

Establish the new testing system on the repo #29

Closed moritzpschwarz closed 3 years ago

moritzpschwarz commented 3 years ago

This is based on extensive work by @jkurle and myself and of course based on the extensive existing test files of @gsucarrat.

Please see my email for additional details of this.

So far, the existing change to the master branch (to get going) is the introduction of a yaml workflow file (see here

All other changes are still in this branch - so no permanent changes have yet been made.

As an overview, the changes proposed in this PR are:

  1. we establish the automatic testthat system, which executes an R CMD check AND a unit test on every commit. The precise unit tests are specified in the testing files (see here)
  2. The data in the package has been compressed to reduce its size.
  3. The \thanks command in the Introduction vignette (one line of text in the JSS article) has had to be removed for now - I will work on this to get it back ASAP.
  4. The DESCRIPTION had to be adapted and now includes the testthat package as well as suggests two other packages, which are used in the tests. Note that this only suggests the packages and does not depend on them (which would be import).

A video explaining this test procedure will follow!