Use deep cloning of reporters in auto_test() and auto_test_package() #456

krlmlr · 2016-04-16T19:57:38Z

Also uses withr::with_envvar() instead of devtools::with_envvar() to avoid warning.

Testing auto_test() and auto_test_package() is tricky, but not impossible. An external R script that runs for a certain time can be started using a pattern like this:

system("Rscript -e 'Sys.sleep(Inf)' & sleep 1")

The output of that script could be then analyzed by the test.

NEWS entry:

- `auto_test()` and `auto_test_package()` show only the results of the current test run and not of previously failed runs (#456, @krlmlr).

to avoid aliasing of contents of reporter members that are R6 objects

to work around r-lib/R6#85

hadley · 2016-04-18T16:21:42Z

Merged manually

Kirill Müller added 5 commits April 16, 2016 21:30

deep-clone reporter in auto_test() and auto_test_package()

549cf46

to avoid aliasing of contents of reporter members that are R6 objects

stack class must be portable

5c4bf52

to work around r-lib/R6#85

need to use private$ for member access

0620f78

use withr::with_envvar() instead of devtools::with_envvar()

c464a42

suggest withr

39281dc

hadley closed this Apr 18, 2016

krlmlr deleted the feature/441-deep-clone-reporter branch May 7, 2016 21:20

Use deep cloning of reporters in auto_test() and auto_test_package() #456

Use deep cloning of reporters in auto_test() and auto_test_package() #456

krlmlr commented Apr 16, 2016 •

edited

hadley commented Apr 18, 2016

Use deep cloning of reporters in auto_test() and auto_test_package() #456

Use deep cloning of reporters in auto_test() and auto_test_package() #456

Conversation

krlmlr commented Apr 16, 2016 • edited

hadley commented Apr 18, 2016

krlmlr commented Apr 16, 2016 •

edited