Closed Bug 1465762 Opened 4 years ago Closed 4 years ago

Enable ESLint rule require-expected-throws-or-rejects for toolkit/**/extensions

Categories

(Toolkit :: Add-ons Manager, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla62
Tracking Status
firefox62 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

This is continuing roll-out of require-expected-throws-or-rejects as per bug 1452706.
Comment on attachment 8982205 [details]
Bug 1465762 - Enable ESLint rule require-expected-throws-or-rejects for toolkit/**/extensions.

https://reviewboard.mozilla.org/r/248188/#review254468

thanks!
Attachment #8982205 - Flags: review?(aswan) → review+
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6630148c9bea
Enable ESLint rule require-expected-throws-or-rejects for toolkit/**/extensions. r=aswan
That's kinda weird, they failed in debug mode, even though the exception message is the same... I'll try and figure something out.
Flags: needinfo?(standard8)
(In reply to Mark Banner (:standard8) from comment #7)
> That's kinda weird, they failed in debug mode, even though the exception
> message is the same... I'll try and figure something out.

Oh, that would be because of the `if (AppConstants.DEBUG)` wrapper around these tests in the code...
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8518549e6bb6
Enable ESLint rule require-expected-throws-or-rejects for toolkit/**/extensions. r=aswan
https://hg.mozilla.org/mozilla-central/rev/8518549e6bb6
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
Blocks: 1466497
No longer blocks: 1466497
Is manual testing required on this bug? If Yes, please provide some STR and the proper webextension(if required), if No set the “qe-verify-“ flag.
Flags: needinfo?(standard8)
No manual testing required, this affects unit tests only.
Flags: needinfo?(standard8)
Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.