testthat vs test-all files, redundant? #1246
Description
Currently there are two entry points to the {testthat} suite:
https://github.com/tidymodels/broom/blob/d7ad27062b6c19d3a04f3e02c037182fbf8ccb4a/tests/test-all.R
https://github.com/tidymodels/broom/blob/d7ad27062b6c19d3a04f3e02c037182fbf8ccb4a/tests/testthat.R
AFAICT they're redundant. It looks like testthat.R was likely added by {usethis} during the transition to testthat 3e (#1228).
One tiny difference is that test-all doesn't call library(broom). I'm not sure it matters, though, because IINM {testthat} will expose all package symbols (even un-exported ones) anyway.
Should one of these be removed?