relax tests on tidyselect error messages

[yjunechoe]

Summary

I just decided to relax all tests on tidyselect error messages in this PR - we already have tests for them where it matters (reading the eval errors on the agent), so testing for error messages directly from expect_() and test_() functions are a bit redundant.

There are just two tests in 75cf1fb where it might be nice to have snapshot tests for (to check whether we're re-throwing the most informative errors), but I'm leaning towards not doing that at least for now since it's not critical to pointblank's behavior.

I also factored out all tidyselect related functions out of utils.R into utils-tidyselect.R to keep track of them in one place.

devtools::test(filter = "tidyselect") now pass both release and dev versions of {tidyselect}

Related GitHub Issues and PRs

• Ref: pointblank and tidyselect 1.2.1 #526

Checklist

• I understand and agree to the Code of Conduct.
• I have listed any major changes in the NEWS.
• I have added testthat unit tests to tests/testthat for any new functionality.

Fixes: #526

[rich-iannone]

Thanks so much for getting this prepared so quickly! Will merge and then prepare this package for a patch release.

[rich-iannone]

LGTM!