New rule: no-commented-out-tests

[kangax]

There's already a rule to catch disabled tests. In our codebase, I'm seeing cases where people disable tests by commenting them out. While this could be easily caught during code reviews, it'd be nice to disallow this via a rule before it even makes it to the review.

Implementation-wise, we could probably detect cases of describe/it/etc. within comments and warn then.

[SimenB]

I'm not sure if it's possible to detect this. Normally we just ask for CallExpression, but in a comment it's just a string. Wanna try to provide PR for it?

[jeysal]

Throwing some regexes at comment contents to catch common cases sounds fine to me if someone wants to take a shot at implementing this 😄

[kangax]

@SimenB @jeysal here you go — lemme know what you think

[SimenB]

🎉 This issue has been resolved in version 22.6.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀