Add tolerance argument to set_state_inspector() (#2238)

With default that ignores minor differences.

Fixes #2237.

Author: mcol

NEWS.md

@@ -1,5 +1,6 @@
 # testthat (development version)
 
+* `set_state_inspector()` gains `tolerance` argument and ignores minor FP differences by default (@mcol, #2237).
 * `expect_vector()` fails, instead of erroring, if `object` is not a vector (@plietar, #2224).
 * New `vignette("mocking")` explains mocking in detail (#1265).
 * New `vignette("challenging-functions")` provides an index to other documentation organised by testing challenges (#1265).

R/test-state.R

@@ -42,7 +42,8 @@
 #' @export
 #' @param callback Either a zero-argument function that returns an object
 #'   capturing global state that you're interested in, or `NULL`.
-set_state_inspector <- function(callback) {
+#' @inheritParams waldo::compare
+set_state_inspector <- function(callback, tolerance = testthat_tolerance()) {
   if (
     !is.null(callback) &&
       !(is.function(callback) && length(formals(callback)) == 0)
@@ -51,11 +52,18 @@ set_state_inspector <- function(callback) {
   }
 
   the$state_inspector <- callback
+  the$state_inspector_tolerance <- tolerance
   invisible()
 }
 
 testthat_state_condition <- function(before, after, call) {
-  diffs <- waldo_compare(before, after, x_arg = "before", y_arg = "after")
+  diffs <- waldo_compare(
+    before,
+    after,
+    x_arg = "before",
+    y_arg = "after",
+    tolerance = the$state_inspector_tolerance
+  )
 
   if (length(diffs) == 0) {
     return(NULL)