New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New rule: avoid-prefixing-tests-with #273
Comments
It would also help if you want to remove an accidental space: it(' should remove space in the beginning', () => { ... I think that adding a rule like that which is fixable (removes the unwanted string) would be nice. |
Sure, PR welcome! 👍 |
Shall I start working on it? |
@suganya-sangith if you've got the time, go for it! I'd love to see this rule, but haven't had the time to finish off my implementation. Feel free to tag me in the PR, and I'll review it 🙂 Name-wise, I think |
Basically, this would catch things like
it('it should throw an error', () => { ...
I figure it would be best as a more generic rule, that took a list of strings that should be considered as undesired to have at the start of the messages for test calls, with "it" being the default.
(The rule name could be improved :/)
The text was updated successfully, but these errors were encountered: