Non standard testthat file names fail to test correctly

[david-lyder]

The test output shows that no tests were run: "Tests": { "Ok": 0, "Skipped": 0, "Failed": 0, "Unknown": 1, "Output": "", "Available": false } However, folders are present, but the file was missed because the name is test-all.rout, which does not match the convention expected.

[dpastoor]

Background:

Initially it seemed sufficient to parse files with a name like test-all.rout* to find the status of all executed tests. Later, files with different names were discovered that also contain test status.

proposed heuristic:

• *.Rout means successful test, *.Rout.fail means failed test
• check if explicitly testthat.Rout/Rout.fail --> known testthat test suite
• if single Rout/Rout.fail file, but not named testthat.Rout, check if that is a testthat style file (has library(testthat) bytes in the file somewhere) - should capture shiny as an example
• if multiple then shouldn't be testthat, instead Rout/Rout.fail should be general tests (eg RcppTOML/data.table)

[dpastoor]

for the heuristic, suggest using globbing via https://github.com/gobwas/glob