Creating an inner-class prompts a complete test detection process
-
Eric Lefevre-Ardant
I think it does that when renaming a class, too (not surprisingly).
-
benrady (at gmail)
Yes, it's merely annoying when it re-indexes after creating a new top level
class. I can see how that would be very aggravating when working with a
library that makes heavy use of inner classes. -
benrady (at gmail)
At least part of the problem here lies in DefaultInfinitestCore.findChangedClassFiles(), which assumes that if any files were removed (or renamed, which the changeDetector treats at the same thing) that the entire index should be reloaded.
I think simply treating the removed class as changed (thereby running any dependent tests) and removing it from the index would be sufficient, but some exploratory testing after making the change would probably be warranted.
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.