Fix broken shinytest2 infrastructure (#2294)

Fixes #2293
Fixes #2288

---------

Co-authored-by: Hadley Wickham <[email protected]>

Author: schloerke

NEWS.md

@@ -1,5 +1,7 @@
 # testthat (development version)
 
+* Fixed support for `shinytest2::AppDriver$expect_values()` screenshot snapshot failing on CI (#2293, #2288).
+
 # testthat 3.3.0
 
 ## Lifecycle changes

R/expectation.R

@@ -104,7 +104,10 @@ exp_signal <- function(exp) {
     } else {
       signalCondition(exp)
     },
-    muffle_expectation = function(e) NULL
+    muffle_expectation = function(e) NULL,
+    # Legacy support for shinytest2
+    # https://github.com/r-lib/testthat/pull/2271#discussion_r2528722708
+    continue_test = function(e) NULL
   )
 
   invisible(exp)

R/snapshot-file.R

@@ -261,9 +261,11 @@ snapshot_file_equal <- function(
     # We want to fail on CI since this suggests that the user has failed
     # to record the value locally
     if (fail_on_new) {
-      return(snapshot_fail(message, trace_env = trace_env))
+      snapshot_fail(message, trace_env = trace_env)
+    } else {
+      testthat_warn(message)
     }
-    testthat_warn(message)
+
     TRUE
   }
 }

R/test-that.R

@@ -110,7 +110,7 @@ test_code <- function(code, env, reporter = NULL, skip_on_empty = TRUE) {
   }
   handle_expectation <- function(e) {
     the$test_expectations <- the$test_expectations + 1L
-    register_expectation(e, 7)
+    register_expectation(e, 11)
     invokeRestart("muffle_expectation")
   }
   handle_warning <- function(e) {