Bug 1781026 - Fix xpcshell no-unused-vars warnings in testing/xpcshell/example/. r=mossop

Differential Revision: https://phabricator.services.mozilla.com/D152621

.eslintrc.js

@@ -30,6 +30,8 @@ const xpcshellTestPaths = [
   "**/test*/unit*/**/",
   "**/test*/*/unit*/",
   "**/test*/xpcshell/**/",
+  // Special case for xpcshell harness.
+  "testing/xpcshell/example/unit/",
 ];
 
 const browserTestPaths = ["**/test*/**/browser*/"];
@@ -224,7 +226,6 @@ module.exports = {
         "dom/**",
         "netwerk/**",
         "security/manager/ssl/tests/unit/**",
-        "testing/xpcshell/**",
         "toolkit/components/**",
         "toolkit/modules/**",
       ],

testing/xpcshell/example/unit/test_get_file.js

@@ -3,8 +3,6 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
-var subscriptLoaded = false;
-
 function run_test() {
   var lf = do_get_file("file.txt");
   Assert.ok(lf.exists());

testing/xpcshell/example/unit/test_prefs_defaults.js

@@ -2,6 +2,9 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
+// Bug 1781025 - ESLint's use of multi-ini doesn't fully cope with
+// mozilla-central's use of .ini files
+// eslint-disable-next-line no-unused-vars
 function run_test() {
   /* import-globals-from prefs_test_common.js */
   load("prefs_test_common.js");

testing/xpcshell/example/unit/test_prefs_defaults_and_file.js

@@ -2,6 +2,9 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
+// Bug 1781025 - ESLint's use of multi-ini doesn't fully cope with
+// mozilla-central's use of .ini files
+// eslint-disable-next-line no-unused-vars
 function run_test() {
   /* import-globals-from prefs_test_common.js */
   load("prefs_test_common.js");

testing/xpcshell/example/unit/xpcshell.ini

@@ -12,8 +12,8 @@ support-files =
   location_load.js
   check_profile.js
   prefs_test_common.js
-# NOTE: Do NOT set prefs= here. If you do, move test_prefs_no_defaults.js and
-# test_prefs_no_defaults_with_file.js to a new file without "prefs =".
+# NOTE: Do NOT set prefs here. If you do, move test_prefs_no_defaults.js and
+# test_prefs_no_defaults_with_file.js to a new file without a pref definitions.
 
 [test_add_setup.js]
 [test_check_nsIException.js]