Bug 1330791 - Enable the no-native-reassign rule for eslint in /services and fix the associated error. r?markh draft
authorJared Wein <jwein@mozilla.com>
Thu, 12 Jan 2017 18:28:48 -0500
changeset 464220 f6287a48deccbaf1dd4507f7f97c47770fbdb468
parent 464219 63dd9535905c50cb0063b0004f97b116cea0bf7f
child 464360 551d49330f318fa82fe45271089512d54e575d58
push id42305
push userjwein@mozilla.com
push dateFri, 20 Jan 2017 17:40:06 +0000
reviewersmarkh
bugs1330791
milestone53.0a1
Bug 1330791 - Enable the no-native-reassign rule for eslint in /services and fix the associated error. r?markh MozReview-Commit-ID: 65EsZGCJPMD
services/.eslintrc.js
services/sync/tests/unit/test_status.js
--- a/services/.eslintrc.js
+++ b/services/.eslintrc.js
@@ -5,12 +5,11 @@ module.exports = {
     "../toolkit/.eslintrc.js"
   ],
   rules: {
     /* These rules are only set to warn temporarily
        until they get fixed, at which point their
        respective line in this file should be removed. */
     "consistent-return": "warn",
     "no-func-assign": "warn",
-    "no-native-reassign": "warn",
     "no-nested-ternary": "warn",
   }
 };
--- a/services/sync/tests/unit/test_status.js
+++ b/services/sync/tests/unit/test_status.js
@@ -68,17 +68,17 @@ function run_test() {
 
 
   // Check resetSync
   Status.sync = SYNC_FAILED;
   Status.resetSync();
 
   do_check_eq(Status.service, STATUS_OK);
   do_check_eq(Status.sync, SYNC_SUCCEEDED);
-  for (name in Status.engines) {
+  if (Status.engines.length) {
     do_throw("Status.engines should be empty.");
   }
 
 
   // Check resetBackoff
   Status.enforceBackoff = true;
   Status.backOffInterval = 4815162342;
   Status.backOffInterval = 42;