Parsing of test failures is slightly unclear

[hadley]

✓  Running ‘statsWrapper_tests.R’ [11s/12s] (23.7s)
E  Running ‘timeProjectedNetwork_tests.R’ (25.1s)
   Running the tests in ‘tests/statsWrapper_tests.R’ failed.

I think it should be something like this?

E  Running ‘statsWrapper_tests.R’ [11s/12s] (23.7s)
✓  Running ‘timeProjectedNetwork_tests.R’ (25.1s)
  
Running the tests in ‘tests/statsWrapper_tests.R’ failed.

[gaborcsardi]

Is this on GHA? Can you link to the build?

[hadley]

It was a failing testthat revdep, but I made a minimal reprex at https://github.com/hadley/testfails

─  checking tests ...
✓  Running ‘a.R’
✓  Running ‘b.R’ (394ms)
E  Running ‘c.R’ (601ms)
   Running the tests in ‘tests/b.R’ failed.
   Complete output:
     > stop("error")
     Error: error
     Execution halted

So looking at more closely I think it should probably be:

─  checking tests ...
✓  Running ‘a.R’
✓  Running ‘b.R’ (394ms)
✓  Running ‘c.R’ (601ms)
E  Running the tests in ‘tests/b.R’ failed.
   Complete output:
     > stop("error")
     Error: error
     Execution halted

[gaborcsardi]

Thanks!

[gaborcsardi]

Ah, the R CMD check output for tests is different when you are using --as-cran.