search

NCEAS / metajam

Bringing data and metadata togetheR
https://nceas.github.io/metajam/
Apache License 2.0
16 stars 7 forks source link

Remove `context` from `testthat` Tests #148

Open njlyon0 opened 4 months ago

njlyon0 commented 4 months ago

According to the chapter on testing in R Packages 2e, the context function is superseded in testthat version 3.0 and can be removed (context is provided by test file name and location in package). Not sure we'll get a flag from CRAN if we don't remove it but seems worthwhile to change while we're prepping for a new version release to CRAN.

brunj7 commented 2 months ago

I removed it from the milestone has CRAN did not mention this. But in the long run it is probably an update we want to do.