Hadley Wickham
Hadley Wickham
See also #1936
Maybe something like: ```R MyClass
We could also combine the two into a single function: ```R local_mocked_r6_class( MyClass, public = list( n = function() 2 ) ) ```
I think it would be fine for this to be an unconditional warning.
But this also doesn't seem that important to me; especially since it means that a reporter would have to keep track of every test that it's seen.
This looks like an rlang bug: https://github.com/r-lib/rlang/issues/1642 However maybe we should just use `deparse()` instead of `expr_deparse()`.
I tried using `deparse()` instead of `expr_deparse()` and I definitely did like the output (and it's likely to affect a bunch of existing snapshots). So we'll need to fix this...
Are you sure it's not the `appTitle`?
I don't have time to get this done today, I don't think it's that important, and we're likely to do a patch release in a few weeks, so I'm pushing...
Simpler reprex, albeit with a different error: ``` r library(rvest) html html_elements(xpath = '//p[contains(text(), "x")]') #> {xml_nodeset (1)} #> [1] x html |> html_elements(xpath = "//p[contains(text(), 'x')]") #> {xml_nodeset (1)}...