Rule request: mocha-avoid-skip

[jesskelsall]

Pretty straightforward, disallow use of Mocha's .skip() method on tests.

Like with mocha-avoid-only, its benefit would be to stop temporary code making it into a build, preventing tests from being ran.

I'd imagine this isn't too different to mocha-avoid-only to implement.

[JoshuaKGoldberg]

Indeed, very similar to the existing mocha-avoid-only rule and a great idea for the same reasons. It would be easiest IMO to add to that rule and rename it to something more generic (it's also valid for other test runners with the same variable names).

Perhaps test-restriction-methods? test-restrictors? 😕

[reduckted]

Just linking #455 here since you mentioned it could be generalized.

[JoshuaKGoldberg]

☠️ It's time! ☠️

Per #876, this repository is no longer accepting feature pull requests. TSLint is being deprecated and we recommend you switch to https://typescript-eslint.io.

Thanks for open sourcing with us, everyone!