RSpec LOC don't contribute to `rails stats`

[CuddlyBunion341]

What Ruby, Rails and RSpec versions are you using?

Ruby version: 3.4.3
Rails version: 8.0.2
RSpec version: 3.13.5

Observed behaviour

When running bin/rails stats the rspec tests are not included in the report:

bin/rails stats
+----------------------+--------+--------+---------+---------+-----+-------+
| Name                 |  Lines |    LOC | Classes | Methods | M/C | LOC/M |
+----------------------+--------+--------+---------+---------+-----+-------+
| Controllers          |    818 |    634 |      18 |      94 |   5 |     4 |
| Helpers              |     97 |     72 |       0 |      13 |   0 |     3 |
| Jobs                 |      9 |      2 |       1 |       0 |   0 |     0 |
| Models               |    800 |    595 |      25 |      49 |   1 |    10 |
| Mailers              |     16 |     12 |       2 |       1 |   0 |    10 |
| Channels             |     22 |     16 |       1 |       2 |   2 |     6 |
| Views                |   2488 |   2155 |       0 |       1 |   0 |  2153 |
| Stylesheets          |   1359 |   1125 |       0 |       0 |   0 |     0 |
| JavaScript           |    428 |    342 |       0 |       0 |   0 |     0 |
| Libraries            |     40 |     34 |       0 |       0 |   0 |     0 |
+----------------------+--------+--------+---------+---------+-----+-------+
| Total                |   6077 |   4987 |      47 |     160 |   3 |    29 |
+----------------------+--------+--------+---------+---------+-----+-------+
  Code LOC: 4987     Test LOC: 0     Code to Test Ratio: 1:0.0

I'd expect rspec to be handled in the code statistics calculator

Expected behaviour

I'd expect the Test LOC to be present as well as the correct ratio and test groups such as Model Tests.

Can you provide an example reproduction?

Todo

[pirj]

I’m not familiar with the stats generator. Can it be RSpec code is ignored because ot doesn’t match the pattern?

[JonRowe]

The cleanest solution would be to propose a patch to Rails to allow the stat reporter to report spec files, we could modify the ruby regex and/or monkey patch it to detect spec files differently, or we could sub class it and override the rake task but both of those seem like brittle ideas to me.

If Rails can't be convinced to support us, maybe Rails could be persuaded of a halfway step where "minitest" becomes "tests" and checks both _test.rb and _spec.rb, and we modify the regex for RSpec specifics.

This is unlikely to be a priority for me but if someone wants to take this up either with Rails or seeing what we can do here please open a PR