Followup: simple load-library tests for R are in autodep8 as of version 0.5 (now in unstable).
On 11/04/16 11:21, Andreas Tille wrote: > On Sun, Apr 10, 2016 at 07:44:50PM +0200, Gordon Ball wrote: >> This is my understanding. From [4] "If debian/tests/control is absent, >> the autodep8 tool can generate an automatic control file.", so anything >> with existing test rules should not be affected. > > Sounds sensible. > >>> As an enhancement I would suggest to detect what test suite of R >>> packages might be used by the package (testthat, ...) and run this one. >>> This might enable more sophisticated testing. >> >> I agree that this would be valuable (much more scope to catch subtle >> regressions than this test), but actually doing it seems difficult >> today. I read #752609, and it seems to me that a universal way of >> running tests is difficult while DESCRIPTION::Suggests are not uniformly >> translated into debian dependencies. > > The packager could (should?) decide to add stronger dependency if a > package is needed for running a test. > >> You can try various test-system specific heuristics (eg, `grep testthat >> tests/*.R && grep-dctrl -F Suggests testthat DESCRIPTION`, if so run >> those files with additional dependency r-cran-testthat), but awkward >> corner cases seem likely. Some flag for preventing test generation would >> also be needed. > > This might be needed anyway. > >>> PS: BTW, I'd consider the Build-Time tests (see #752609) as similarly >>> important to CI tests. >> >> I think the requirement is the same. If a package can run build-time >> tests (and hence there is a mechanism for running them, and ensuring the >> dependencies are available), then this can be fairly easily translated >> into a CI target. > > I think adding dependencies for full testing as Build-Depends is pretty > cheap if there would be a chance to run the tests. > > Kind regards > > Andreas. > >>>> [1]: https://git.chronitis.net/autodep8.git/rrequire/ >>>> [2]: Packages are matched by source name /^r-/ and having a >>>> ./DESCRIPTION file >>> [3] https://git.chronitis.net/autodep8.git/blob/rrequire/test/r_test.sh >>> >> [4]: https://people.debian.org/~mpitt/autopkgtest/README.package-tests.html >> >> >
