359 lint errors in the ui-users tests

Description

if you edit ui-users/package.json's "lint" script to

"lint": "eslint *.js lib settings test || true",

Then running yarn lint yields 359 errors.

Since "tests are code", we really ought to be keeping this code up at the same quality as the main code.

(I imagine the same is true of the tests in ui-items and elsewhere, but I won't bother filing separate issues for each module at this stage.)

Environment

None

Potential Workaround

None

Checklist

hide

TestRail: Results

Activity

Show:

Charles LedvinaJanuary 10, 2018 at 9:59 PM

3.5 hours later-- 0 lint errors.

Charles LedvinaJanuary 10, 2018 at 6:26 PM

Eeeek!

Mike TaylorJanuary 10, 2018 at 8:57 AM

Assigned to Charles, since he is the "product owner" for UI tests.

Mike TaylorSeptember 21, 2017 at 1:33 PM

Quick check: ui-items is better, only 121 lint errors.

Done

Details

Assignee

Reporter

Labels

Priority

Fix versions

TestRail: Cases

Open TestRail: Cases

TestRail: Runs

Open TestRail: Runs

Created September 21, 2017 at 1:32 PM
Updated July 3, 2019 at 8:35 PM
Resolved January 10, 2018 at 9:59 PM
TestRail: Cases
TestRail: Runs

Flag notifications