Open Bug 1389234 Opened 3 years ago Updated 2 years ago

Deprecate SimpleTest.requestFlakyTimeout in favor of the mozilla/no-arbitrary-setTimeout eslint rule

Categories

(Testing :: Mochitest, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

People

(Reporter: ahal, Unassigned)

References

Details

As discussed in bug 1385352, both requestFlakyTimeout and the eslint rule are accomplishing the same thing. We should have a single mechanism for preventing flaky timeouts.

I vote for the eslint rule because:

1) Doesn't need to be implemented per test harness (and reduces complexity)
2) It's a common tool that many devs are already familiar with
3) Devs can choose to disable it at the directory, file or line level

This bug should:
1) Remove the requestFlakyTimeout mechanism
2) Replace invocations of SimpleTest.requestFlakyTimeout with // eslint-disable
3) Enable mozilla/no-arbitrary-setTimeout on mochitest plain and chrome
I like eslint too, but keep in mind that some mochitests may not be linted, due to exclusions in .eslintignore.
You need to log in before you can comment on or make changes to this bug.