Skip to content

Commit

Permalink
fixed R CMD check freeze
Browse files Browse the repository at this point in the history
  • Loading branch information
markvanderloo committed Sep 25, 2019
1 parent 20fd42b commit 0ab0659
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/DESCRIPTION
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ Description: Provides a lightweight (zero-dependency) and easy to use
configurable output printing. Compare computed output with output stored
with the package. Run tests in parallel. Extensible by other packages.
Report side effects.
Version: 1.0.0.8
Version: 1.0.0.9
URL: https://github.com/markvanderloo/tinytest
BugReports: https://github.com/markvanderloo/tinytest/issues
Imports: parallel, utils
Expand Down
2 changes: 2 additions & 0 deletions pkg/NEWS
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,8 @@ version 1.1.0
Bart-Jan van Rossum).
- Fixed crash of 'build_install_test' when pkg was developed under directory
with spaces (thanks to Bart-Jan van Rossum).
- Fixed issue where R CMD check would hang when packages use paralellization (GH #36)
or certain Rcpp features (GH #37). Thanks to George G Vega Yon and Dirk Eddelbuettel.


version 1.0.0
Expand Down
6 changes: 6 additions & 0 deletions pkg/R/tinytest.R
Original file line number Diff line number Diff line change
Expand Up @@ -524,6 +524,12 @@ run_test_file <- function( file
## Reduce user side effects by capturing options that will be reset
## on exit
e$options <- capture_options(options, oldop)

## Set useFancyQuotes, which is usually done by startup.Rs, the location
## of which is defined by envvar R_TESTS, which we set to empty now.
## See GH issues 36,37
options(useFancyQuotes=FALSE)
Sys.setenv(R_TESTS="")

## Make sure that we catch side-effects if the user asks for it.
# an environment to store side-effects, and wheter we report them.
Expand Down

0 comments on commit 0ab0659

Please sign in to comment.