Closed Bug 1463673 Opened 2 years ago Closed 2 years ago

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

Categories

(Toolkit :: General, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla62
Tracking Status
firefox62 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(5 files)

This is continuing roll-out of require-expected-throws-or-rejects as per bug 1452706.
Comment on attachment 8979856 [details]
Bug 1463673 - Member expressions should also be allowed as expected arguments to Assert.throws/rejects.

https://reviewboard.mozilla.org/r/246040/#review252284
Attachment #8979856 - Flags: review?(dtownsend) → review+
Comment on attachment 8979859 [details]
Bug 1463673 - Add the expected argument to Assert.throws/rejects for various tests in toolkit/.

https://reviewboard.mozilla.org/r/246046/#review252286
Attachment #8979859 - Flags: review?(dtownsend) → review+
Comment on attachment 8979860 [details]
Bug 1463673 - Enable ESLint rule require-expected-throws-or-rejects for toolkit/.

https://reviewboard.mozilla.org/r/246048/#review252288
Attachment #8979860 - Flags: review?(dtownsend) → review+
Comment on attachment 8979858 [details]
Bug 1463673 - Add the expected argument to Assert.throws/rejects for toolkit/components/places.

https://reviewboard.mozilla.org/r/246044/#review252534
Attachment #8979858 - Flags: review?(mak77) → review+
Comment on attachment 8979857 [details]
Bug 1463673 - Add the expected argument to Assert.throws/rejects for toolkit/components/telemetry.

https://reviewboard.mozilla.org/r/246042/#review252942

Thanks!
Attachment #8979857 - Flags: review?(gfritzsche) → review+
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d4bfc66b1fb9
Member expressions should also be allowed as expected arguments to Assert.throws/rejects. r=mossop
https://hg.mozilla.org/integration/autoland/rev/015e8f74e973
Add the expected argument to Assert.throws/rejects for toolkit/components/telemetry. r=gfritzsche
https://hg.mozilla.org/integration/autoland/rev/c9f558e4282c
Add the expected argument to Assert.throws/rejects for toolkit/components/places. r=mak
https://hg.mozilla.org/integration/autoland/rev/1350bcc2983e
Add the expected argument to Assert.throws/rejects for various tests in toolkit/. r=mossop
https://hg.mozilla.org/integration/autoland/rev/d73afee801c0
Enable ESLint rule require-expected-throws-or-rejects for toolkit/. r=mossop
Depends on: 1465385
No longer depends on: 1465385
You need to log in before you can comment on or make changes to this bug.