Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

auto_test should pay attention to reporter argument #81

Merged
merged 2 commits into from Dec 5, 2013
Merged
Changes from 1 commit
Commits
File filter...
Filter file types
Jump to…
Jump to file or symbol
Failed to load files and symbols.
+2 −2
Diff settings

Always

Just for now

Next

auto_test should pay attention to reporter argument

  • Loading branch information...
crowding committed Jul 2, 2013
commit ba9a044d9951a33ba37a5daa609d97be729120d0
Copy path View file
@@ -41,7 +41,7 @@ auto_test <- function(code_path, test_path, reporter = "summary", env = NULL) {
env <- new.env(parent = globalenv())
}
source_dir(code_path, env = env)
test_dir(test_path, env = env)
test_dir(test_path, env = env, reporter = reporter)

starts_with <- function(string, prefix) {
substr(string, 1, nchar(prefix)) == prefix
@@ -59,7 +59,7 @@ auto_test <- function(code_path, test_path, reporter = "summary", env = NULL) {
cat("Changed code: ", paste0(basename(code), collapse = ", "), "\n")
cat("Rerunning all tests\n")
source_dir(code_path, env = env)
test_dir(test_path, env = env)
test_dir(test_path, env = env, reporter = reporter)
} else if (length(tests) > 0) {
# If test changes, rerun just that test
cat("Rerunning tests: ", paste0(basename(tests), collapse = ", "), "\n")
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.