Merge branch 'main' into feature/add-recursive-dirs

Author: radbasa

NEWS.md

@@ -1,6 +1,8 @@
 # testthat (development version)
 
 * `test_dir()` gains a `recursive` argument which allows test files in nested directories (#1605).
+* Fixed an issue where calling `skip()` outside of an active test could
+  cause an unexpected error (@kevinushey, #2039).
 
 # testthat 3.2.2
 

R/snapshot-reporter.R

@@ -35,7 +35,9 @@ SnapshotReporter <- R6::R6Class("SnapshotReporter",
     },
 
     start_test = function(context, test) {
-      self$test <- gsub("\n", "", test)
+      if (is.character(test)) {
+        self$test <- gsub("\n", "", test)
+      }
     },
 
     # Called by expectation