Run automatic linting

tests/ui/job-view/Filtering_test.jsx

@@ -468,12 +468,8 @@ describe('Filtering', () => {
     });
 
     test('Filters | Reset should get back to original set of jobs', async () => {
-      const {
-        getAllByText,
-        findAllByText,
-        findByText,
-        queryAllByText,
-      } = render(testApp());
+      const { getAllByText, findAllByText, findByText, queryAllByText } =
+        render(testApp());
       const symbolToRemove = 'yaml';
 
       await findAllByText('B');

tests/ui/job-view/Push_test.jsx

@@ -71,8 +71,7 @@ describe('Transformations', () => {
         expected: 'devtools/client/framework/browser-toolbox/test/browser.ini',
       },
       {
-        path:
-          'devtools/client/framework/browser-toolbox/test/browser_browser_toolbox.js',
+        path: 'devtools/client/framework/browser-toolbox/test/browser_browser_toolbox.js',
         expected:
           'devtools/client/framework/browser-toolbox/test/browser_browser_toolbox.js',
       },

tests/ui/job-view/details/PinBoard_test.jsx

@@ -262,12 +262,8 @@ describe('DetailsPanel', () => {
   });
 
   test('classify and unclassify all jobs', async () => {
-    const {
-      getByPlaceholderText,
-      getByText,
-      getByTitle,
-      queryAllByTitle,
-    } = render(testDetailsPanel());
+    const { getByPlaceholderText, getByText, getByTitle, queryAllByTitle } =
+      render(testDetailsPanel());
 
     store.dispatch(pinJobs(jobList.data));
     store.dispatch(setSelectedJob(jobList.data[1], true));

tests/ui/perfherder/alerts-view/alerts_test.jsx

@@ -467,11 +467,8 @@ test("'Take' button hides when clicking on 'Unassigned' badge", async () => {
   alertSummary.assignee_email = '[email protected]';
   alertSummary.assignee_username = 'mozilla-ldap/[email protected]';
 
-  const {
-    getByText,
-    queryByText,
-    queryByPlaceholderText,
-  } = alertsViewControls();
+  const { getByText, queryByText, queryByPlaceholderText } =
+    alertsViewControls();
 
   const unassignedBadge = await waitFor(() => getByText('Unassigned'));
 
@@ -659,11 +656,8 @@ const assertAlertsAreInOrder = async (alertsInOrder, alertTableRows) => {
 };
 
 test(`table data can be sorted in descending order by 'Test'`, async () => {
-  const {
-    getAllByLabelText,
-    getByTestId,
-    getAllByTitle,
-  } = alertsViewControls();
+  const { getAllByLabelText, getByTestId, getAllByTitle } =
+    alertsViewControls();
 
   let alertTableRows = await waitFor(() =>
     getAllByLabelText('Alert table row'),
@@ -698,11 +692,8 @@ test(`table data can be sorted in descending order by 'Test'`, async () => {
 });
 
 test(`table data can be sorted in ascending order by 'Platform'`, async () => {
-  const {
-    getByTestId,
-    getAllByLabelText,
-    getAllByTitle,
-  } = alertsViewControls();
+  const { getByTestId, getAllByLabelText, getAllByTitle } =
+    alertsViewControls();
 
   let alertTableRows = await waitFor(() =>
     getAllByLabelText('Alert table row'),
@@ -744,11 +735,8 @@ test(`table data cannot be sorted by 'Tags & Options'`, async () => {
 });
 
 test(`table data can be sorted in ascending order by 'Confidence'`, async () => {
-  const {
-    getAllByLabelText,
-    getByTestId,
-    getAllByTitle,
-  } = alertsViewControls();
+  const { getAllByLabelText, getByTestId, getAllByTitle } =
+    alertsViewControls();
 
   let alertTableRows = await waitFor(() =>
     getAllByLabelText('Alert table row'),
@@ -780,11 +768,8 @@ test(`table data can be sorted in ascending order by 'Confidence'`, async () =>
 });
 
 test(`table data can be sorted in ascending order by 'Magnitude of Change'`, async () => {
-  const {
-    getAllByLabelText,
-    getByTestId,
-    getAllByTitle,
-  } = alertsViewControls();
+  const { getAllByLabelText, getByTestId, getAllByTitle } =
+    alertsViewControls();
 
   let alertTableRows = await waitFor(() =>
     getAllByLabelText('Alert table row'),
@@ -817,11 +802,8 @@ test(`table data can be sorted in ascending order by 'Magnitude of Change'`, asy
 });
 
 test('Data can be sorted only by one column', async () => {
-  const {
-    getAllByLabelText,
-    getByTestId,
-    getAllByTitle,
-  } = alertsViewControls();
+  const { getAllByLabelText, getByTestId, getAllByTitle } =
+    alertsViewControls();
 
   let alertTableRows = await waitFor(() =>
     getAllByLabelText('Alert table row'),

tests/ui/perfherder/compare-view/compare_table_test.jsx

@@ -343,9 +343,8 @@ test('filters that are not enabled are removed from URL params', async () => {
 });
 
 test('page parameter updates with value 2 when clicking on the second page', async () => {
-  const { getByText, findAllByLabelText } = compareTableControls(
-    compareResults,
-  );
+  const { getByText, findAllByLabelText } =
+    compareTableControls(compareResults);
 
   const result1 = await waitFor(() =>
     getByText(compareTablesControlsResults[0].name),
@@ -374,11 +373,8 @@ test('page parameter updates with value 2 when clicking on the second page', asy
 });
 
 test('text input filter results should differ when filter button(s) are selected', async () => {
-  const {
-    getByText,
-    getByPlaceholderText,
-    queryByText,
-  } = compareTableControls();
+  const { getByText, getByPlaceholderText, queryByText } =
+    compareTableControls();
 
   const result1 = await waitFor(() => getByText(result[0].name));
   const result2 = await waitFor(() => getByText(result[1].name));

tests/ui/perfherder/graphs-view/graphs_view_test.jsx

@@ -278,12 +278,8 @@ test('Using select query param displays tooltip for correct datapoint with repli
 });
 
 test('InputFilter from TestDataModal can filter by application name', async () => {
-  const {
-    getByText,
-    getByTestId,
-    getByPlaceholderText,
-    getByTitle,
-  } = graphsViewControls();
+  const { getByText, getByTestId, getByPlaceholderText, getByTitle } =
+    graphsViewControls();
 
   const { name, application, projectName, platform } = seriesData[0];
   const fullTestName = projectName.concat(
@@ -313,12 +309,8 @@ test('InputFilter from TestDataModal can filter by application name', async () =
 
 test('Changing the platform dropdown while filtered by text in the Test Data Modal displays expected tests', async () => {
   mockShowModal.mockClear();
-  const {
-    getByText,
-    getByPlaceholderText,
-    getByTitle,
-    getByTestId,
-  } = graphsViewControls();
+  const { getByText, getByPlaceholderText, getByTitle, getByTestId } =
+    graphsViewControls();
 
   fireEvent.click(getByText('Add test data'));
 

tests/ui/perfherder/replicates_graph_test.jsx

@@ -23,31 +23,8 @@ const mockedReplicateData = {
           lowerIsBetter: true,
           name: SUBTEST_NAME,
           replicates: [
-            500,
-            345,
-            366,
-            358,
-            355,
-            365,
-            354,
-            354,
-            363,
-            360,
-            389,
-            357,
-            273,
-            376,
-            350,
-            381,
-            373,
-            356,
-            351,
-            348,
-            271,
-            354,
-            351,
-            360,
-            385,
+            500, 345, 366, 358, 355, 365, 354, 354, 363, 360, 389, 357, 273,
+            376, 350, 381, 373, 356, 351, 348, 271, 354, 351, 360, 385,
           ],
           shouldAlert: false,
           unit: 'ms',
@@ -58,31 +35,8 @@ const mockedReplicateData = {
           lowerIsBetter: true,
           name: 'tablemutation.html',
           replicates: [
-            89,
-            88,
-            90,
-            83,
-            87,
-            87,
-            88,
-            86,
-            85,
-            84,
-            85,
-            88,
-            102,
-            83,
-            100,
-            86,
-            100,
-            80,
-            87,
-            88,
-            82,
-            85,
-            84,
-            83,
-            86,
+            89, 88, 90, 83, 87, 87, 88, 86, 85, 84, 85, 88, 102, 83, 100, 86,
+            100, 80, 87, 88, 82, 85, 84, 83, 86,
           ],
           shouldAlert: false,
           unit: 'ms',
@@ -98,26 +52,8 @@ const mockedReplicateData = {
         {
           name: 'ts_paint',
           replicates: [
-            844,
-            832,
-            842,
-            825,
-            849,
-            816,
-            857,
-            841,
-            849,
-            840,
-            846,
-            872,
-            864,
-            837,
-            830,
-            843,
-            841,
-            854,
-            841,
-            858,
+            844, 832, 842, 825, 849, 816, 857, 841, 849, 840, 846, 872, 864,
+            837, 830, 843, 841, 854, 841, 858,
           ],
           value: 842,
         },
@@ -134,26 +70,12 @@ const mockedReplicateData = {
           lowerIsBetter: true,
           name: '',
           replicates: [
-            552.9000000000001,
-            561.6599999999999,
-            558.1199999999999,
-            565.0799999999999,
-            554.94,
-            599.9399999999998,
-            600.9200000000001,
-            559.22,
-            552.0600000000002,
-            559.4000000000001,
-            603.0999999999999,
-            586.4199999999998,
-            587.3200000000002,
-            606.3800000000001,
-            587.1000000000001,
-            592.9200000000001,
-            542.9199999999998,
-            553.98,
-            556.4200000000001,
-            576.0999999999999,
+            552.9000000000001, 561.6599999999999, 558.1199999999999,
+            565.0799999999999, 554.94, 599.9399999999998, 600.9200000000001,
+            559.22, 552.0600000000002, 559.4000000000001, 603.0999999999999,
+            586.4199999999998, 587.3200000000002, 606.3800000000001,
+            587.1000000000001, 592.9200000000001, 542.9199999999998, 553.98,
+            556.4200000000001, 576.0999999999999,
           ],
           shouldAlert: false,
           unit: 'ms',

tests/ui/push-health/details/DetailsPanel_test.jsx

@@ -65,8 +65,7 @@ describe('DetailsPanel', () => {
               },
               line_number: 18841,
             },
-            line:
-              '22:42:53     INFO - TEST-UNEXPECTED-FAIL | devtools/client/debugger/test/mochitest/browser_dbg-sourcemaps.js | Uncaught exception - at chrome://mochitests/content/browser/devtools/client/debugger/test/mochitest/helpers.js:358 - TypeError: can\'t access property "wrapClass", lineInfo is null',
+            line: '22:42:53     INFO - TEST-UNEXPECTED-FAIL | devtools/client/debugger/test/mochitest/browser_dbg-sourcemaps.js | Uncaught exception - at chrome://mochitests/content/browser/devtools/client/debugger/test/mochitest/helpers.js:358 - TypeError: can\'t access property "wrapClass", lineInfo is null',
             line_number: 18841,
           },
         ],

ui/helpers/job.js

@@ -237,7 +237,8 @@ export const getTaskRunStr = (job) => `${job.task_id}.${job.retry_id}`;
 
 // This matches as taskId, optionally followed by `.` or`-` and a runId.
 // We support `-` for backwards compatability with the original format used.
-const taskRunPattern = /^([A-Za-z0-9_-]{8}[Q-T][A-Za-z0-9_-][CGKOSWaeimquy26-][A-Za-z0-9_-]{10}[AQgw])(?:[-.]([0-9]+))?$/;
+const taskRunPattern =
+  /^([A-Za-z0-9_-]{8}[Q-T][A-Za-z0-9_-][CGKOSWaeimquy26-][A-Za-z0-9_-]{10}[AQgw])(?:[-.]([0-9]+))?$/;
 
 export const getTaskRun = function getTaskRun(taskRunStr) {
   const match = taskRunPattern.exec(taskRunStr);

ui/helpers/performance.js

@@ -1,5 +1,4 @@
 // This file may export additional functions.
-/* eslint-disable import/prefer-default-export */
 
 import TaskclusterModel from '../models/taskcluster';
 

ui/helpers/revision.js

@@ -24,7 +24,7 @@ export const getRevisionTitle = function getRevisionTitle(revisions) {
      *  Strip out unwanted things like additional lines, trychooser
      *  syntax, request flags, mq cruft, whitespace, and punctuation
      */
-    // eslint-disable-next-line prefer-destructuring
+
     title = title.split('\n')[0];
     title = title.replace(/\btry: .*/, '');
     title = title.replace(/\b(r|sr|f|a)=.*/, '');

ui/helpers/sort.js

@@ -1,4 +1,3 @@
-// eslint-disable-next-line import/prefer-default-export
 export const sortAlphaNum = (a, b) => {
   // Implement a better alphanumeric sort so that mochitest-10
   // is sorted after mochitest 9, not mochitest-1

ui/helpers/taskcluster.js

@@ -80,7 +80,6 @@ const taskcluster = (() => {
         userCredentials[_rootUrl] &&
         moment(userCredentials[_rootUrl].expires).isAfter(moment())
       ) {
-        // eslint-disable-next-line no-promise-executor-return
         return resolve(userCredentials[_rootUrl]);
       }
 

ui/helpers/utils.js

@@ -1,5 +1,3 @@
-/* eslint-disable no-bitwise */
-
 export const hashFunction = (someString) => {
   // Borrowed from https://github.com/darkskyapp/string-hash
   let hash = 5381;

ui/infra-compare/InfraCompareTableView.jsx

@@ -66,12 +66,8 @@ export default class InfraCompareTableView extends React.Component {
 
   getInfraData = async () => {
     const { getQueryParams, getDisplayResults } = this.props;
-    const {
-      originalProject,
-      originalRevision,
-      newProject,
-      newRevision,
-    } = this.props.validated;
+    const { originalProject, originalRevision, newProject, newRevision } =
+      this.props.validated;
     const { timeRange, failureMessages } = this.state;
 
     this.setState({ loading: true });
@@ -143,13 +139,8 @@ export default class InfraCompareTableView extends React.Component {
       pageTitle,
     } = this.props.validated;
     const { jobsNotDisplayed } = this.props;
-    const {
-      compareResults,
-      loading,
-      failureMessages,
-      timeRange,
-      tabTitle,
-    } = this.state;
+    const { compareResults, loading, failureMessages, timeRange, tabTitle } =
+      this.state;
     const compareDropdowns = [];
     const params = {
       originalProject,