Bug 1330791 - Enable the no-native-reassign rule for eslint in /services and fix the associated error. r=markh
authorJared Wein <jwein@mozilla.com>
Thu, 12 Jan 2017 18:28:48 -0500
changeset 464597 3e1c799eb6f618b9d7e1176143148216c36a8f7a
parent 464596 ec582d2701280097f50b8474ba6a1f01faa3f50a
child 464598 d15f6cbb443d1979bff7d2e974448b6bb3dedcd9
push id42370
push usermwein@mozilla.com
push dateSat, 21 Jan 2017 03:10:58 +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;