ESlint rule 'no-arbitrary-setTimeout' enabled on xpcshell tests
ahalberstadt at mozilla.com
Fri Aug 11 13:15:28 UTC 2017
This is now also enabled on browser-chrome tests. Bug 1389234 has been
filed to track deprecating SimpleTest.requestFlakyTimeout on
mochitest-plain and chrome in favour of this new rule.
On Fri, Jul 28, 2017 at 12:02 PM Andrew Halberstadt <
ahalberstadt at mozilla.com> wrote:
> Ah, good to know! I'll file a follow-up to enable the eslint rule on
> browser/a11y/chrome. Maybe eventually we can just replace the
> requestFlakyTimeout mechanism with this eslint rule. I decided to punt on
> that as I'm not sure if eslint is running on 100% of mochitests yet.
> On Fri, Jul 28, 2017 at 11:56 AM Felipe G <felipc at gmail.com> wrote:
>> I'll note that requestFlakyTimeout is only enabled for mochitest-plain at
>> the moment:
>> So browser-chrome / a11y / chrome tests are still able to use non-0
>> On Fri, Jul 28, 2017 at 12:48 PM, Andrew Halberstadt <
>> ahalberstadt at mozilla.com> wrote:
>>> As part of a larger effort to reduce oranges, we are starting to lint our
>>> tests for common causes of intermittent failures. One low-hanging fruit
>>> preventing setTimeout with an arbitrary value (aka non-zero) as opposed
>>> waiting for an appropriate event. The mochitest harness already prevents
>>> this in the harness itself (SimpleTest.requestFlakyTimeout), so this rule
>>> is only enabled on xpcshell tests for now.
>>> If you need to use a flaky setTimeout for some reason, you can disable
>>> rule at the directory level, file level or line level:
>>> It has been disabled in the following files due to pre-existing
>>> Let me know if you think this should be enabled on any other test suites.
>>> dev-platform mailing list
>>> dev-platform at lists.mozilla.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the firefox-dev