Remove a TODO and update a snapshot test!

Author: DavisVaughan

tests/testthat/_snaps/pick.md

@@ -213,7 +213,7 @@
       filter(df, pick(x, y)$x)
     Condition
       Error in `filter()`:
-      i In argument: `asNamespace("dplyr")$dplyr_pick_tibble(x = x, y = y)$x`.
+      i In argument: `pick(x, y)$x`.
       Caused by error:
       ! `..1` must be a logical vector, not a double vector.
 

tests/testthat/test-pick.R

@@ -492,7 +492,6 @@ test_that("`filter()` with `pick()` that uses invalid tidy-selection errors", {
 test_that("`filter()` that doesn't use `pick()` result correctly errors", {
   df <- tibble(x = c(1, 2, NA, 3), y = c(2, NA, 5, 3))
 
-  # TODO: Can we improve on the `In argument:` expression in the expansion case?
   expect_snapshot(error = TRUE, {
     filter(df, pick(x, y)$x)
   })