FR: Warn optionally on duplicate test names

[krlmlr]

Should testthat warn in this case, perhaps optionally?

library(testthat)

test_that("foo", {
  expect_true(TRUE)
})
#> Test passed 🎊

test_that("foo", {
  expect_true(TRUE)
})
#> Test passed 🎊

^{Created on 2023-08-23 with reprex v2.0.2}

Reference: tidyverse/dplyr#6901 (comment)

[hadley]

I think it would be fine for this to be an unconditional warning.

[hadley]

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.