-
Notifications
You must be signed in to change notification settings - Fork 12
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
Looking for rule ideas! 💡 #1
Comments
Rules around skipped and focused tests would be useful - for example, being able to make a focused test a lint error. (Skipped tests actually already are a lint error if you have |
It would help us on CI if we have some rule like " no only on scenarios" |
Could there be a rule for no We sometimes add this to scenarios during debugging, but would like to make sure it's not checked in. |
@GLuchtenberg @poshaughnessy I've added #11 and #12 for skipped and exclusive tests, respectively. |
Thanks @textbook I'll check! |
One of my colleagues just put out a pr for no pause if this is something might be interested in: #16 |
Great idea! I will check it out. Thanks.
…On Tue, Nov 19, 2019, 11:39 George Griffiths ***@***.***> wrote:
One of my colleagues just put out a pr for no pause if this is something
might be interested in:
https://github.com/poenneby/eslint-plugin-codeceptjs/pulls
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#1?email_source=notifications&email_token=AAPXJEA3YHPITCYJUC362SLQUO66BA5CNFSM4C2A6HEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEENW3PA#issuecomment-555445692>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAPXJEDB24RCXTKX4AWCC73QUO66BANCNFSM4C2A6HEA>
.
|
@poenneby I would like to see a rule |
This plugin is extremely simple as it currently only defines the globals in CodeceptJS.
Here are some ideas of rules:
prefer_page_objects - Prevent the use of✅I
withinScenario
Scenario
s withoutFeature
function callyield
within aScenario
while missing generator function*
declarationFeel free to suggest others!
The text was updated successfully, but these errors were encountered: