Default to not running helpers when loading (#1841)

hadley · web-flow · commit 4c6bf287f59d · 2023-08-01T17:19:09.000-05:00
diff --git a/NEWS.md b/NEWS.md
@@ -1,5 +1,7 @@
 # testthat (development version)
 
+* Helpers should no longer be run twice.
+
 * `test_file()` gains a `desc` argument which allows you to run a single 
   test from a file (#1776).
 
diff --git a/R/test-files.R b/R/test-files.R
@@ -251,7 +251,7 @@ test_files_setup_env <- function(test_package,
 }
 
 find_load_all_args <- function(path) {
-  default <- list(export_all = TRUE, helpers = TRUE)
+  default <- list(export_all = TRUE, helpers = FALSE)
 
   desc <- find_description(path)
   if (is.null(desc)) {

Original file line number	Diff line number	Diff line change
`@@ -251,7 +251,7 @@ test_files_setup_env <- function(test_package,`
`251`	`251`	`}`
`252`	`252`
`253`	`253`	`find_load_all_args <- function(path) {`
`254`		`- default <- list(export_all = TRUE, helpers = TRUE)`
	`254`	`+ default <- list(export_all = TRUE, helpers = FALSE)`
`255`	`255`
`256`	`256`	`desc <- find_description(path)`
`257`	`257`	`if (is.null(desc)) {`