Enable new eslint rules for Jasmine tests
The following discussion from !21825 (merged) should be addressed:
-
@mikegreiling started a discussion: (+1 comment) jasmine/new-line-before-expect: off jasmine/new-line-between-declarations: off jasmine/no-promise-without-done-fail: off jasmine/prefer-jasmine-matcher: off jasmine/prefer-toHaveBeenCalledWith: off
These are all new rules added in the latest version of
eslint-plugin-jasmine
. The first three are able to be fixed automatically, and the later two need more manual care. There are hundreds of violations and the may not all be worth addressing. We should review them in turn and create separate MRs.
Edited by Mike Greiling