Skip to content

Conversation

@averissimo
Copy link
Contributor

@averissimo averissimo commented Jun 4, 2025

Changes description

  • it() call passes the parent.frame() down to internal describe_it()
  • Adds test case

@hadley
Copy link
Member

hadley commented Jul 23, 2025

Hmmm, reading the implementation of describe() , it's not clear to me that using :: in this way is really desired or supported. But I guess your fix doesn't make anything worse.

@hadley hadley merged commit b8a4c47 into r-lib:main Jul 23, 2025
13 checks passed
schloerke added a commit to schloerke/testthat that referenced this pull request Jul 25, 2025
* upstream/HEAD: (250 commits)
  Implement `expect_shape()` (r-lib#1469)
  Re-enable catch tests on windows (r-lib#2104)
  Move digest to suggests (r-lib#2105)
  Support emscripten in `skip_on_os()` (r-lib#2103)
  Add `skip_unless_r()` (r-lib#2094)
  Require R 4.1 (r-lib#2101)
  Allow unquoting first arg for `expect_s4_class()` (r-lib#2065)
  typo fix (r-lib#2051)
  Explicitly pass `parent.frame()` in `it()` (r-lib#2086)
  Update vignette advice for migrating to {testthat} 3e (r-lib#2080)
  New praise messages (r-lib#1974)
  move '!' outside aggregation (r-lib#2067)
  Handle deprecation of std::uncaught_exception() (r-lib#2097)
  perf: use `anyNA()` (r-lib#2058)
  Upkeep (r-lib#2099)
  Implement `mock_output_sequence()` (r-lib#2061)
  Increment version number to 3.2.3.9000
  Increment version number to 3.2.3
  Update CRAN comments
  Finish off r-lib#2046
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

describe_it doesn't find the right environment when it() is prefixed with testthat::

2 participants