-
Notifications
You must be signed in to change notification settings - Fork 103
Open
Description
What problem are you trying to solve?
RemoveTestPrefix is invoked by multiple composite "best practice" recipe paths. Yes, test was once a required method prefix and now is not, but removing it isn't a best practice it is more of a test naming preference. In fact, multiple teams I've worked on prefer to keep it so the IDE's method list clearly shows the test methods apart from the supporting methods in the test class (especially with a sorted list).
Have you considered any alternatives or workarounds?
It is large effort of creating multiple custom recipes to disable it (just try the assertj recipe).
Without the "excludeRecipe" feature, avoiding it is not a quick effort.
Any additional context
When a team wants to remove test as a prefix, they can run the single recipe.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels
Type
Projects
Status
No status