Better advice for mocked base functions.

If the function is used as an argument somewhere, binding NULL to it causes that call to fail. Instead bind the actual function, and then mock that. I think it might also be necessary to do the same for the other default packages, but I'm not certain. Will likely know better at the end of TDD!
r-lib · Aug 15, 2024 · 193d4da · 193d4da
1 parent f060cce
commit 193d4da
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/R/mock2.R b/R/mock2.R
@@ -48,12 +48,12 @@
 #'
 #' To mock a function in the base package, you need to make sure that you
 #' have a binding for this function in your package. It's easiest to do this
-#' by binding the value to `NULL`. For example, if you wanted to mock
+#' by binding the name to the function. For example, if you wanted to mock
 #' `interactive()` in your package, you'd need to include this code somewhere
 #' in your package:
 #'
 #' ```R
-#' interactive <- NULL
+#' interactive <- interactive
 #' ```
 #'
 #' Why is this necessary? `with_mocked_bindings()` and `local_mocked_bindings()`

diff --git a/man/local_mocked_bindings.Rd b/man/local_mocked_bindings.Rd