fix(eslint): Add no test.slow() eslint rule to functional tests - Githubissues

mozilla / fxa

Monorepo for Mozilla Accounts (formerly Firefox Accounts)

https://mozilla.github.io/ecosystem-platform/

Mozilla Public License 2.0

580 stars 210 forks source link

fix(eslint): Add no test.slow() eslint rule to functional tests #17111

Closed vbudhram closed 1 month ago

vbudhram commented 1 month ago

Because

In the last meeting we mentioned that future playwright tests should avoid using test.slow() and change default timeout

This pull request

Add new eslint rule to check for test.slow() usage, shows lint warning if you do use it

Issue that this pull request solves

Closes: # (issue number)

Checklist

[x] My commit is GPG signed.
[ ] If applicable, I have modified or added tests which pass locally.
[ ] I have added necessary documentation (if appropriate).
[ ] I have verified that my changes render correctly in RTL (if appropriate).

ashrivastava-qa commented 1 month ago

This is great! Thanks @vbudhram :)