Call of cleanup_descendants_tracking cleanup too much models about with_model

[cvincent]

👍 Just hit this issue upgrading from Rails 6.1 -> 7.0. For those Googling, the specific error message is: DescendantsTracker.clear was disabled because config.cache_classes = true.

[nertzy]

There's no reason. I think your idea is a good one.

[nertzy]

To give a little more context, we were running into issues years ago on an old project where with_model models would still exist in the descendants after a test was finished. We cleared the tracked descendants as a quick way to work around this problem.

[kakuzei]

Thank you for you answer.
How do you want to proceed to make this change available for everyone?

[marcocarvalho]

This is causing strange errors with STI.

I have one test that uses WithModel; after that test run, the next test that uses any STI will fail because one or more classes are not present on the descendants collection.

I was only able to spot this error because I turned on config.cache_classes

[nertzy]

Fixed in #44