From 8640fa2942a9e815f740515301cc9b9c33b40ef2 Mon Sep 17 00:00:00 2001
From: "Richard O. Gregory" <richardgrecoson@gmail.com>
Date: Mon, 4 Nov 2019 18:05:14 +0000
Subject: [PATCH] Bug 1589333 - Enable ESLint for all of
 devtools/client/shared/ (manual changes). r=Standard8,jdescottes

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

--HG--
extra : moz-landing-system : lando
---
 .eslintignore                                   |  3 ---
 .../mochitest/test_GridElementWidthResizer.html |  5 ++++-
 .../test_GridElementWidthResizer_RTL.html       |  5 ++++-
 .../test/mochitest/test_HSplitBox_01.html       |  5 ++++-
 .../test/mochitest/test_accordion.html          |  2 ++
 .../test/mochitest/test_frame_01.html           |  3 +++
 .../test/mochitest/test_frame_02.html           |  3 +++
 .../test/mochitest/test_list_keyboard.html      |  4 ++--
 .../mochitest/test_notification_box_01.html     |  3 +++
 .../mochitest/test_notification_box_02.html     |  3 +++
 .../mochitest/test_notification_box_03.html     |  5 ++++-
 .../test_searchbox-with-autocomplete.html       |  1 -
 .../test/mochitest/test_sidebar_toggle.html     |  3 +++
 .../test/mochitest/test_tabs_accessibility.html |  3 +++
 .../test/mochitest/test_tabs_menu.html          |  3 +++
 .../test/mochitest/test_tree-view_01.html       |  6 +++---
 .../test/mochitest/test_tree-view_02.html       |  3 ++-
 .../components/test/mochitest/test_tree_01.html |  3 +++
 .../components/test/mochitest/test_tree_02.html |  3 +++
 .../components/test/mochitest/test_tree_03.html |  3 +++
 .../components/test/mochitest/test_tree_04.html |  3 +++
 .../components/test/mochitest/test_tree_07.html |  3 +++
 .../components/test/mochitest/test_tree_08.html |  3 +++
 .../components/test/mochitest/test_tree_09.html |  3 +++
 .../components/test/mochitest/test_tree_10.html |  5 ++++-
 .../components/test/mochitest/test_tree_11.html |  3 +++
 .../components/test/mochitest/test_tree_14.html |  4 ++--
 .../components/test/mochitest/test_tree_16.html |  2 --
 .../shared/test/doc_event-listeners-01.html     |  1 +
 .../shared/test/doc_event-listeners-03.html     | 17 +++++++++--------
 .../test/doc_inline-debugger-statement.html     |  3 +++
 .../shared/test/doc_native-event-handler.html   |  3 ++-
 32 files changed, 91 insertions(+), 28 deletions(-)

diff --git a/.eslintignore b/.eslintignore
index 834daf99c050..a6d5df740ded 100644
--- a/.eslintignore
+++ b/.eslintignore
@@ -56,9 +56,6 @@ browser/extensions/formautofill/phonenumberutils/PhoneNumberMetaData.jsm
 # devtools/ exclusions
 devtools/client/inspector/markup/test/doc_markup_events_*.html
 devtools/client/performance/components/test/test_jit_optimizations_01.html
-devtools/client/shared/test/*.html
-devtools/client/shared/components/test/mochitest/*.html
-!devtools/client/shared/components/test/mochitest/test_stack-trace.html
 devtools/server/tests/browser/storage-*.html
 !devtools/server/tests/browser/storage-unsecured-iframe.html
 devtools/server/tests/browser/stylesheets-nested-iframes.html
diff --git a/devtools/client/shared/components/test/mochitest/test_GridElementWidthResizer.html b/devtools/client/shared/components/test/mochitest/test_GridElementWidthResizer.html
index 40aafe7eb07c..3c02d755952b 100644
--- a/devtools/client/shared/components/test/mochitest/test_GridElementWidthResizer.html
+++ b/devtools/client/shared/components/test/mochitest/test_GridElementWidthResizer.html
@@ -75,6 +75,9 @@
     <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 const FUDGE_FACTOR = .5;
 function aboutEq(a, b) {
   dumpn(`Checking ${a} ~= ${b}`);
@@ -93,7 +96,7 @@ window.onload = async function () {
     const resizedA = [];
     const resizedD = [];
 
-    const scene = ReactDOM.render([
+    ReactDOM.render([
       dom.header({}, "header"),
       dom.aside({className: "a"}, "A"),
       GridElementWidthResizer({
diff --git a/devtools/client/shared/components/test/mochitest/test_GridElementWidthResizer_RTL.html b/devtools/client/shared/components/test/mochitest/test_GridElementWidthResizer_RTL.html
index 695d2148170a..d4243dfd0790 100644
--- a/devtools/client/shared/components/test/mochitest/test_GridElementWidthResizer_RTL.html
+++ b/devtools/client/shared/components/test/mochitest/test_GridElementWidthResizer_RTL.html
@@ -76,6 +76,9 @@
     <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 const FUDGE_FACTOR = .5;
 function aboutEq(a, b) {
   dumpn(`Checking ${a} ~= ${b}`);
@@ -94,7 +97,7 @@ window.onload = async function () {
     const resizedA = [];
     const resizedD = [];
 
-    const scene = ReactDOM.render([
+    ReactDOM.render([
       dom.header({}, "header"),
       dom.aside({className: "a"}, "A"),
       GridElementWidthResizer({
diff --git a/devtools/client/shared/components/test/mochitest/test_HSplitBox_01.html b/devtools/client/shared/components/test/mochitest/test_HSplitBox_01.html
index dd836694a5da..b992374ff023 100644
--- a/devtools/client/shared/components/test/mochitest/test_HSplitBox_01.html
+++ b/devtools/client/shared/components/test/mochitest/test_HSplitBox_01.html
@@ -24,6 +24,9 @@ Basic tests for the HSplitBox component.
 <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 const FUDGE_FACTOR = .1;
 function aboutEq(a, b) {
   dumpn(`Checking ${a} ~= ${b}`);
@@ -57,7 +60,7 @@ window.onload = async function () {
       return el;
     }
 
-    const box = await renderBox();
+    await renderBox();
 
     // Test that we properly rendered our two panes.
 
diff --git a/devtools/client/shared/components/test/mochitest/test_accordion.html b/devtools/client/shared/components/test/mochitest/test_accordion.html
index 8c1e4605da06..689db51e7bb1 100644
--- a/devtools/client/shared/components/test/mochitest/test_accordion.html
+++ b/devtools/client/shared/components/test/mochitest/test_accordion.html
@@ -22,6 +22,8 @@ Test that Accordion renders correctly.
 
 "use strict";
 
+/* global sinon */
+
 window.onload = async function() {
   try {
     const { button, div } = require("devtools/client/shared/vendor/react-dom-factories");
diff --git a/devtools/client/shared/components/test/mochitest/test_frame_01.html b/devtools/client/shared/components/test/mochitest/test_frame_01.html
index 50c4d718835b..63dea0488e7b 100644
--- a/devtools/client/shared/components/test/mochitest/test_frame_01.html
+++ b/devtools/client/shared/components/test/mochitest/test_frame_01.html
@@ -21,6 +21,9 @@ with optional columns, unknown and non-URL sources.
     <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 window.onload = async function () {
   try {
     const ReactDOM = browserRequire("devtools/client/shared/vendor/react-dom");
diff --git a/devtools/client/shared/components/test/mochitest/test_frame_02.html b/devtools/client/shared/components/test/mochitest/test_frame_02.html
index 351cd74ea47f..2884d619c104 100644
--- a/devtools/client/shared/components/test/mochitest/test_frame_02.html
+++ b/devtools/client/shared/components/test/mochitest/test_frame_02.html
@@ -16,6 +16,9 @@ Test that the frame component reacts to source-map pref changse.
 <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 window.onload = async function () {
   try {
     const Services = browserRequire("Services");
diff --git a/devtools/client/shared/components/test/mochitest/test_list_keyboard.html b/devtools/client/shared/components/test/mochitest/test_list_keyboard.html
index b2453c38dedc..7558404ed2f2 100644
--- a/devtools/client/shared/components/test/mochitest/test_list_keyboard.html
+++ b/devtools/client/shared/components/test/mochitest/test_list_keyboard.html
@@ -63,7 +63,7 @@ window.onload = function() {
 
     const tree = ReactDOM.render(list, document.body);
     const listEl = findRenderedDOMComponentWithClass(tree, "list");
-    const items = scryRenderedDOMComponentsWithTag(tree, "li");
+    scryRenderedDOMComponentsWithTag(tree, "li");
     const defaultFocus = listEl.ownerDocument.body;
 
     function blurEl(el) {
@@ -250,7 +250,7 @@ window.onload = function() {
     }];
 
     for (const test of tests) {
-      const { action, condition, event, state, name } = test;
+      const { action, event, state, name } = test;
 
       is(listEl, findRenderedDOMComponentWithClass(tree, "list"), "Sanity check");
 
diff --git a/devtools/client/shared/components/test/mochitest/test_notification_box_01.html b/devtools/client/shared/components/test/mochitest/test_notification_box_01.html
index 97c5b19d3383..2921d607c3b8 100644
--- a/devtools/client/shared/components/test/mochitest/test_notification_box_01.html
+++ b/devtools/client/shared/components/test/mochitest/test_notification_box_01.html
@@ -21,6 +21,9 @@ Test for Notification Box. The test is checking:
 <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 window.onload = async function () {
   try {
     const ReactDOM = browserRequire("devtools/client/shared/vendor/react-dom");
diff --git a/devtools/client/shared/components/test/mochitest/test_notification_box_02.html b/devtools/client/shared/components/test/mochitest/test_notification_box_02.html
index 6523417fea5d..f74194d1287f 100644
--- a/devtools/client/shared/components/test/mochitest/test_notification_box_02.html
+++ b/devtools/client/shared/components/test/mochitest/test_notification_box_02.html
@@ -17,6 +17,9 @@ Test for Notification Box. The test is checking:
 <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 window.onload = async function () {
   try {
     const ReactDOM = browserRequire("devtools/client/shared/vendor/react-dom");
diff --git a/devtools/client/shared/components/test/mochitest/test_notification_box_03.html b/devtools/client/shared/components/test/mochitest/test_notification_box_03.html
index e966e328458e..816456edd32b 100644
--- a/devtools/client/shared/components/test/mochitest/test_notification_box_03.html
+++ b/devtools/client/shared/components/test/mochitest/test_notification_box_03.html
@@ -17,6 +17,9 @@ Test for Notification Box. The test is checking:
 <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 window.onload = async function () {
   try {
     const ReactDOM = browserRequire("devtools/client/shared/vendor/react-dom");
@@ -29,7 +32,7 @@ window.onload = async function () {
     const notificationNode = ReactDOM.findDOMNode(notificationBox);
 
     let buttonCallbackExecuted = false;
-    var buttons = [{
+    const buttons = [{
       label: "Button1",
       callback: () => {
         buttonCallbackExecuted = true;
diff --git a/devtools/client/shared/components/test/mochitest/test_searchbox-with-autocomplete.html b/devtools/client/shared/components/test/mochitest/test_searchbox-with-autocomplete.html
index dd0e34679d50..7bec93bc4666 100644
--- a/devtools/client/shared/components/test/mochitest/test_searchbox-with-autocomplete.html
+++ b/devtools/client/shared/components/test/mochitest/test_searchbox-with-autocomplete.html
@@ -85,7 +85,6 @@ window.onload = async function () {
     },
     onChange: () => null,
   });
-  const { refs } = component;
 
   async function testSearchBoxWithAutocomplete() {
     ok(!$(".devtools-autocomplete-popup"), "Autocomplete list not visible");
diff --git a/devtools/client/shared/components/test/mochitest/test_sidebar_toggle.html b/devtools/client/shared/components/test/mochitest/test_sidebar_toggle.html
index c195498d64c3..0a31037a84d5 100644
--- a/devtools/client/shared/components/test/mochitest/test_sidebar_toggle.html
+++ b/devtools/client/shared/components/test/mochitest/test_sidebar_toggle.html
@@ -16,6 +16,9 @@ Test sidebar toggle button
 <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 window.onload = async function () {
   const SidebarToggle = browserRequire("devtools/client/shared/components/SidebarToggle.js");
 
diff --git a/devtools/client/shared/components/test/mochitest/test_tabs_accessibility.html b/devtools/client/shared/components/test/mochitest/test_tabs_accessibility.html
index 09df5fda7976..4d0ea6ef9639 100644
--- a/devtools/client/shared/components/test/mochitest/test_tabs_accessibility.html
+++ b/devtools/client/shared/components/test/mochitest/test_tabs_accessibility.html
@@ -16,6 +16,9 @@ Test tabs accessibility.
 <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 window.onload = async function () {
   try {
     const ReactDOM = browserRequire("devtools/client/shared/vendor/react-dom");
diff --git a/devtools/client/shared/components/test/mochitest/test_tabs_menu.html b/devtools/client/shared/components/test/mochitest/test_tabs_menu.html
index af955b512324..43e530d3d5d8 100644
--- a/devtools/client/shared/components/test/mochitest/test_tabs_menu.html
+++ b/devtools/client/shared/components/test/mochitest/test_tabs_menu.html
@@ -21,6 +21,9 @@ Test all-tabs menu.
 <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 window.onload = async function () {
   try {
     const ReactDOM = browserRequire("devtools/client/shared/vendor/react-dom");
diff --git a/devtools/client/shared/components/test/mochitest/test_tree-view_01.html b/devtools/client/shared/components/test/mochitest/test_tree-view_01.html
index 438cd9c61bda..393913df38be 100644
--- a/devtools/client/shared/components/test/mochitest/test_tree-view_01.html
+++ b/devtools/client/shared/components/test/mochitest/test_tree-view_01.html
@@ -35,7 +35,7 @@ window.onload = function() {
     const TreeView =
       browserRequire("devtools/client/shared/components/tree/TreeView");
 
-    const props = {
+    const _props = {
       ...TEST_TREE_VIEW_INTERFACE,
       renderValue: props => {
         return (props.value === "C" ?
@@ -47,7 +47,7 @@ window.onload = function() {
         );
       },
     };
-    const treeView = React.createElement(TreeView, props);
+    const treeView = React.createElement(TreeView, _props);
     const tree = ReactDOM.render(treeView, document.body);
     const treeViewEl = findRenderedDOMComponentWithClass(tree, "treeTable");
     const rows = scryRenderedDOMComponentsWithClass(tree, "treeRow");
@@ -251,7 +251,7 @@ window.onload = function() {
     }];
 
     for (const test of tests) {
-      const { action, condition, event, state, name } = test;
+      const { action, event, state, name } = test;
 
       info(name);
       if (event) {
diff --git a/devtools/client/shared/components/test/mochitest/test_tree-view_02.html b/devtools/client/shared/components/test/mochitest/test_tree-view_02.html
index 0fceae13b154..dc77b2cc7a12 100644
--- a/devtools/client/shared/components/test/mochitest/test_tree-view_02.html
+++ b/devtools/client/shared/components/test/mochitest/test_tree-view_02.html
@@ -74,7 +74,7 @@ window.onload = function() {
         }];
 
         for (const test of tests) {
-          const { action, condition, event, state, name } = test;
+          const { action, event, state, name } = test;
 
           info(name);
           if (event) {
@@ -98,6 +98,7 @@ window.onload = function() {
           if (label === "C") {
             return ["hide"];
           }
+          return [];
         }
       }
     };
diff --git a/devtools/client/shared/components/test/mochitest/test_tree_01.html b/devtools/client/shared/components/test/mochitest/test_tree_01.html
index de533354329c..0740c8957e60 100644
--- a/devtools/client/shared/components/test/mochitest/test_tree_01.html
+++ b/devtools/client/shared/components/test/mochitest/test_tree_01.html
@@ -17,6 +17,9 @@ depth.
 <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 window.onload = async function () {
   try {
     const ReactDOM = browserRequire("devtools/client/shared/vendor/react-dom");
diff --git a/devtools/client/shared/components/test/mochitest/test_tree_02.html b/devtools/client/shared/components/test/mochitest/test_tree_02.html
index 9bb1636fab5a..f53896557216 100644
--- a/devtools/client/shared/components/test/mochitest/test_tree_02.html
+++ b/devtools/client/shared/components/test/mochitest/test_tree_02.html
@@ -16,6 +16,9 @@ Test that collapsed subtrees aren't rendered.
 <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 window.onload = async function () {
   try {
     const ReactDOM = browserRequire("devtools/client/shared/vendor/react-dom");
diff --git a/devtools/client/shared/components/test/mochitest/test_tree_03.html b/devtools/client/shared/components/test/mochitest/test_tree_03.html
index e7d182d62041..6ebefa1fb73a 100644
--- a/devtools/client/shared/components/test/mochitest/test_tree_03.html
+++ b/devtools/client/shared/components/test/mochitest/test_tree_03.html
@@ -16,6 +16,9 @@ Test Tree's autoExpandDepth.
 <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 window.onload = async function () {
   try {
     const ReactDOM = browserRequire("devtools/client/shared/vendor/react-dom");
diff --git a/devtools/client/shared/components/test/mochitest/test_tree_04.html b/devtools/client/shared/components/test/mochitest/test_tree_04.html
index 599c2a41b0ee..2213f7249737 100644
--- a/devtools/client/shared/components/test/mochitest/test_tree_04.html
+++ b/devtools/client/shared/components/test/mochitest/test_tree_04.html
@@ -16,6 +16,9 @@ Test that we only render visible tree items.
 <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 window.onload = async function () {
   try {
     function getSpacerHeights() {
diff --git a/devtools/client/shared/components/test/mochitest/test_tree_07.html b/devtools/client/shared/components/test/mochitest/test_tree_07.html
index 9739304a11f1..2e763aaf2019 100644
--- a/devtools/client/shared/components/test/mochitest/test_tree_07.html
+++ b/devtools/client/shared/components/test/mochitest/test_tree_07.html
@@ -17,6 +17,9 @@ Test that arrows get the open attribute when their item's children are expanded.
 <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 window.onload = async function () {
   try {
     const ReactDOM = browserRequire("devtools/client/shared/vendor/react-dom");
diff --git a/devtools/client/shared/components/test/mochitest/test_tree_08.html b/devtools/client/shared/components/test/mochitest/test_tree_08.html
index 6f64ba840b71..cfdff8090d1d 100644
--- a/devtools/client/shared/components/test/mochitest/test_tree_08.html
+++ b/devtools/client/shared/components/test/mochitest/test_tree_08.html
@@ -18,6 +18,9 @@ other inputs.
 <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 window.onload = async function () {
   try {
     const ReactDOM = browserRequire("devtools/client/shared/vendor/react-dom");
diff --git a/devtools/client/shared/components/test/mochitest/test_tree_09.html b/devtools/client/shared/components/test/mochitest/test_tree_09.html
index d1f959c4429b..4d6a1010b5f5 100644
--- a/devtools/client/shared/components/test/mochitest/test_tree_09.html
+++ b/devtools/client/shared/components/test/mochitest/test_tree_09.html
@@ -17,6 +17,9 @@ Test that when an item in the Tree component is expanded or collapsed the approp
 <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 window.onload = async function () {
   try {
     const ReactDOM = browserRequire("devtools/client/shared/vendor/react-dom");
diff --git a/devtools/client/shared/components/test/mochitest/test_tree_10.html b/devtools/client/shared/components/test/mochitest/test_tree_10.html
index 8a220553d72b..7cda9e4348ab 100644
--- a/devtools/client/shared/components/test/mochitest/test_tree_10.html
+++ b/devtools/client/shared/components/test/mochitest/test_tree_10.html
@@ -17,6 +17,9 @@ Test that when an item in the Tree component is expanded or collapsed the approp
 <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 window.onload = async function () {
   try {
     const ReactDOM = browserRequire("devtools/client/shared/vendor/react-dom");
@@ -32,7 +35,7 @@ window.onload = async function () {
       return ReactDOM.render(Tree(treeProps), window.document.body);
     }
 
-    const tree = renderTree({ focused: "A" });
+    renderTree({ focused: "A" });
 
     isRenderedTree(document.body.textContent, [
       "A:true",
diff --git a/devtools/client/shared/components/test/mochitest/test_tree_11.html b/devtools/client/shared/components/test/mochitest/test_tree_11.html
index 0bed1a701e96..612a851018c1 100644
--- a/devtools/client/shared/components/test/mochitest/test_tree_11.html
+++ b/devtools/client/shared/components/test/mochitest/test_tree_11.html
@@ -29,6 +29,9 @@ Test that when an item in the Tree component is focused by arrow key, the view i
 <pre id="test">
 <script src="head.js" type="application/javascript"></script>
 <script type="application/javascript">
+
+'use strict'
+
 window.onload = async function () {
   try {
     const ReactDOM = browserRequire("devtools/client/shared/vendor/react-dom");
diff --git a/devtools/client/shared/components/test/mochitest/test_tree_14.html b/devtools/client/shared/components/test/mochitest/test_tree_14.html
index c6b838969e55..d68d87d6c525 100644
--- a/devtools/client/shared/components/test/mochitest/test_tree_14.html
+++ b/devtools/client/shared/components/test/mochitest/test_tree_14.html
@@ -29,7 +29,6 @@ window.onload = async function() {
       Simulate,
       findRenderedDOMComponentWithClass,
       findRenderedDOMComponentWithTag,
-      scryRenderedDOMComponentsWithTag,
     } = browserRequire("devtools/client/shared/vendor/react-dom-test-utils");
     const Tree = createFactory(
       browserRequire("devtools/client/shared/components/VirtualizedTree"));
@@ -209,7 +208,7 @@ window.onload = async function() {
     }];
 
     for (const test of tests) {
-      const { action, condition, event, props, name } = test;
+      const { action, event, props, name } = test;
 
       info(name);
       if (event) {
@@ -225,6 +224,7 @@ window.onload = async function() {
       if (test.activeElement) {
         const expected = typeof test.activeElement === "string" ?
           els[test.activeElement] : test.activeElement;
+        // eslint-disable-next-line no-debugger
         if (document.activeElement!==expected) {debugger;}
         is(document.activeElement, expected, "Focus is set correctly.");
       }
diff --git a/devtools/client/shared/components/test/mochitest/test_tree_16.html b/devtools/client/shared/components/test/mochitest/test_tree_16.html
index b361afdfe811..b70e63eadecc 100644
--- a/devtools/client/shared/components/test/mochitest/test_tree_16.html
+++ b/devtools/client/shared/components/test/mochitest/test_tree_16.html
@@ -36,8 +36,6 @@ window.onload = async function () {
   try {
     const ReactDOM = browserRequire("devtools/client/shared/vendor/react-dom");
     const { createFactory } = browserRequire("devtools/client/shared/vendor/react");
-    const { Simulate } =
-      browserRequire("devtools/client/shared/vendor/react-dom-test-utils");
     const Tree =
       createFactory(browserRequire("devtools/client/shared/components/VirtualizedTree"));
 
diff --git a/devtools/client/shared/test/doc_event-listeners-01.html b/devtools/client/shared/test/doc_event-listeners-01.html
index 85857702addf..5a9c2d53b6b6 100644
--- a/devtools/client/shared/test/doc_event-listeners-01.html
+++ b/devtools/client/shared/test/doc_event-listeners-01.html
@@ -16,6 +16,7 @@
 
       window.addEventListener("load", function() {
         function initialSetup(event) {
+          // eslint-disable-next-line no-debugger
           debugger;
           const button = document.querySelector("button");
           button.onclick = clickHandler;
diff --git a/devtools/client/shared/test/doc_event-listeners-03.html b/devtools/client/shared/test/doc_event-listeners-03.html
index 5afb7d663aec..2660f9f14144 100644
--- a/devtools/client/shared/test/doc_event-listeners-03.html
+++ b/devtools/client/shared/test/doc_event-listeners-03.html
@@ -20,6 +20,7 @@
         function initialSetup(event) {
           const button = document.getElementById("initialSetup");
           button.removeEventListener("click", initialSetup);
+          // eslint-disable-next-line no-debugger
           debugger;
         }
 
@@ -27,24 +28,24 @@
           window.foobar = "clicker";
         }
 
-        function handleEventClick() {
+        function HandleEventClick() {
           const button = document.getElementById("handleEventClick");
           // Create a long prototype chain to test for weird edge cases.
           button.addEventListener("click", Object.create(Object.create(this)));
         }
 
-        handleEventClick.prototype.handleEvent = function() {
-          window.foobar = "handleEventClick";
+        HandleEventClick.prototype.handleEvent = function() {
+          window.foobar = "HandleEventClick";
         };
 
-        function boundHandleEventClick() {
+        function BoundHandleEventClick() {
           const button = document.getElementById("boundHandleEventClick");
           this.handleEvent = this.handleEvent.bind(this);
           button.addEventListener("click", this);
         }
 
-        boundHandleEventClick.prototype.handleEvent = function() {
-          window.foobar = "boundHandleEventClick";
+        BoundHandleEventClick.prototype.handleEvent = function() {
+          window.foobar = "BoundHandleEventClick";
         };
 
         const button = document.getElementById("clicker");
@@ -52,8 +53,8 @@
         const boundClicker = clicker.bind(this).bind(this).bind(this);
         button.addEventListener("click", boundClicker);
 
-        new handleEventClick();
-        new boundHandleEventClick();
+        new HandleEventClick();
+        new BoundHandleEventClick();
 
         const initButton = document.getElementById("initialSetup");
         initButton.addEventListener("click", initialSetup);
diff --git a/devtools/client/shared/test/doc_inline-debugger-statement.html b/devtools/client/shared/test/doc_inline-debugger-statement.html
index f448a939cbd2..64aba53cf36c 100644
--- a/devtools/client/shared/test/doc_inline-debugger-statement.html
+++ b/devtools/client/shared/test/doc_inline-debugger-statement.html
@@ -13,7 +13,10 @@
 
     <script type="text/javascript">
       "use strict";
+      
+      /* exported runDebuggerStatement */
       function runDebuggerStatement() {
+        // eslint-disable-next-line no-debugger
         debugger;
       }
     </script>
diff --git a/devtools/client/shared/test/doc_native-event-handler.html b/devtools/client/shared/test/doc_native-event-handler.html
index 48f33195ea12..f08d7c01f1eb 100644
--- a/devtools/client/shared/test/doc_native-event-handler.html
+++ b/devtools/client/shared/test/doc_native-event-handler.html
@@ -8,10 +8,11 @@
     <script type="text/javascript">
       "use strict";
 
+      /* exported initialSetup */
       function initialSetup(event) {
+        // eslint-disable-next-line no-debugger
         debugger;
       }
-
       window.addEventListener("load", function() {});
     </script>
   </head>