Fix for skip outside test (#2039)

Fixes #2038
r-lib · Dec 12, 2024 · 24ff6c0 · 24ff6c0
1 parent dbe8883
commit 24ff6c0
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/NEWS.md b/NEWS.md
@@ -1,5 +1,8 @@
 # testthat (development version)
 
+* Fixed an issue where calling `skip()` outside of an active test could
+  cause an unexpected error (@kevinushey, #2039).
+
 # testthat 3.2.2
 
 ## New expectations

diff --git a/R/snapshot-reporter.R b/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