Open Bug 1681329 Opened 5 years ago Updated 4 years ago

Create an eslint rule enforcing that `reject()` calls must pass an error or parameter

Categories

(Developer Infrastructure :: Lint and Formatting, enhancement)

Product:
Developer Infrastructure
Component:
Lint and Formatting
Platform:
Desktop
All
Type:
enhancement

Tracking

(Not tracked)

People

(Reporter: Gijs, Unassigned)

References

Details

Kris, mconley and I just spent a while helping Scott deal with a weird intermittent failure, which turned out to be related to https://searchfox.org/mozilla-central/rev/71621bfa47a371f2b1ccfd33c704913124afb933/testing/mochitest/BrowserTestUtils/BrowserTestUtils.jsm#472 rejecting with no message or stack.

I don't know why there'd be no stack, but at least the "no message" part seems avoidable, and like something we should use linting to stop elsewhere, to make intermittent failures and debugging them less painful.

I expect there is no stack trace because there is no Error being generated.

Type: defect → enhancement
See Also: → 1677300
See Also: → 1684043
See Also: 1677300 →
Product: Firefox Build System → Developer Infrastructure
You need to log in before you can comment on or make changes to this bug.