Explicitly pass parent.frame() in it() (#2086)

Fixes #2085

Author: averissimo

NEWS.md

@@ -1,5 +1,6 @@
 # testthat (development version)
 
+* `it()` now finds the correct evaluation environment in more cases (@averissimo, #2085).
 * Fixed an issue preventing compilation from succeeding due to deprecation / removal of `std::uncaught_exception()` (@kevinushey, #2047).
 
 # testthat 3.2.3

R/describe.R

@@ -94,5 +94,5 @@ it <- function(description, code = NULL) {
   check_string(description, allow_empty = FALSE)
 
   code <- substitute(code)
-  describe_it(description, code)
+  describe_it(description, code, env = parent.frame())
 }

tests/testthat/test-describe.R

@@ -37,6 +37,12 @@ describe("describe", {
     expect_equal(1, someInternalVariable)
   })
 
+  # prefix is needed to test this use case
+  testthat::it("should be possible to use variables from outer environments with package prefix", {
+    expect_equal(1, someExternalVariable)
+    expect_equal(1, someInternalVariable)
+  })
+
   it("should not be possible to access variables from other specs (1)", {
     some_test_var <- 5
   })