Sam

I guess its the abstract class yeah as it looks for the class name. This is a bug.

I've added a test for the inheritence. It's not that, as that test works. I would try pulling your tests out of the test factories.

Kinda curious, not entirely related, what are you using test factories for ?

Just checking if anything needs to be done here or if we can close this

Should afterProject run regardless? Maybe it should? afterSpec should not (I have a PR up for that).

> After making the changes, the errors have changed I am seeing this too sometimes. It is a config issue somewhere.

If it's been broken for 4 years I'll just remove it. We can update manually, which might not be perfect, but can't be worse than a no-op script :)

What do we want to do with this ?

would be fantastic to get this over the line

Can we close this if it's not possible ?