You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The no-skipped-tests autofix replaces it.skip() with it()
In my opinion, this should not happen as this actually alters the way tests are run.
The rule is definitely valid, but I suggest not making this autofixable.
What do you guys think? I'd be happy to submit a PR.
The text was updated successfully, but these errors were encountered:
Personally I don’t have a strong opinion on that since I never use skipped tests. We already decided a while ago to remove the autofix feature from no-exclusive-tests because editors or IDEs automatically apply autofixes all the time and it becomes annoying when you want to run only one test from your IDE.
LukaPrebil
pushed a commit
to LukaPrebil/eslint-plugin-mocha
that referenced
this issue
Jul 28, 2020
Hi all,
The no-skipped-tests autofix replaces
it.skip()
withit()
In my opinion, this should not happen as this actually alters the way tests are run.
The rule is definitely valid, but I suggest not making this autofixable.
What do you guys think? I'd be happy to submit a PR.
The text was updated successfully, but these errors were encountered: