Restore files (Eslint 8.21.0)

vrigal · Jul 2, 2024 · f4791fb · f4791fb
1 parent 9a6d576
commit f4791fb
Show file tree

Hide file tree

Showing 57 changed files with 655 additions and 349 deletions.
diff --git a/tests/ui/job-view/Filtering_test.jsx b/tests/ui/job-view/Filtering_test.jsx
@@ -190,10 +190,12 @@ describe('Filtering', () => {
     });
 
     test('should have 1 push', async () => {
-      const { getAllByText, getAllByTestId, getByText, getByTitle } = render(
-        testApp(),
-        { legacyRoot: true },
-      );
+      const {
+        getAllByText,
+        getAllByTestId,
+        getByText,
+        getByTitle,
+      } = render(testApp(), { legacyRoot: true });
       const unfilteredPushes = await waitFor(() =>
         getAllByTestId('push-header'),
       );
@@ -492,8 +494,12 @@ describe('Filtering', () => {
     });
 
     test('Filters | Reset should get back to original set of jobs', async () => {
-      const { getAllByText, findAllByText, findByText, queryAllByText } =
-        render(testApp(), { legacyRoot: true });
+      const {
+        getAllByText,
+        findAllByText,
+        findByText,
+        queryAllByText,
+      } = render(testApp(), { legacyRoot: true });
       const symbolToRemove = 'yaml';
 
       await findAllByText('B');

diff --git a/tests/ui/job-view/Push_test.jsx b/tests/ui/job-view/Push_test.jsx
@@ -71,7 +71,8 @@ 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',
       },

diff --git a/tests/ui/job-view/details/PinBoard_test.jsx b/tests/ui/job-view/details/PinBoard_test.jsx
@@ -270,8 +270,12 @@ describe('DetailsPanel', () => {
   });
 
   test('classify and unclassify all jobs', async () => {
-    const { getByPlaceholderText, getByText, getByTitle, queryAllByTitle } =
-      render(testDetailsPanel(), { legacyRoot: true });
+    const {
+      getByPlaceholderText,
+      getByText,
+      getByTitle,
+      queryAllByTitle,
+    } = render(testDetailsPanel(), { legacyRoot: true });
 
     store.dispatch(pinJobs(jobList.data));
     store.dispatch(setSelectedJob(jobList.data[1], true));

diff --git a/tests/ui/perfherder/alerts-view/alerts_test.jsx b/tests/ui/perfherder/alerts-view/alerts_test.jsx
@@ -469,8 +469,11 @@ test("'Take' button hides when clicking on 'Unassigned' badge", async () => {
   alertSummary.assignee_email = 'test_user@mozilla.com';
   alertSummary.assignee_username = 'mozilla-ldap/test_user@mozilla.com';
 
-  const { getByText, queryByText, queryByPlaceholderText } =
-    alertsViewControls();
+  const {
+    getByText,
+    queryByText,
+    queryByPlaceholderText,
+  } = alertsViewControls();
 
   const unassignedBadge = await waitFor(() => getByText('Unassigned'));
 
@@ -658,8 +661,11 @@ 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'),
@@ -694,8 +700,11 @@ 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'),
@@ -737,8 +746,11 @@ 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'),
@@ -770,8 +782,11 @@ 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'),
@@ -804,8 +819,11 @@ 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'),

diff --git a/tests/ui/perfherder/compare-view/compare_table_test.jsx b/tests/ui/perfherder/compare-view/compare_table_test.jsx
@@ -346,8 +346,9 @@ 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),
@@ -376,8 +377,11 @@ 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));

diff --git a/tests/ui/perfherder/graphs-view/graphs_view_test.jsx b/tests/ui/perfherder/graphs-view/graphs_view_test.jsx
@@ -279,8 +279,12 @@ 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(
@@ -310,8 +314,12 @@ 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'));
 

diff --git a/tests/ui/perfherder/replicates_graph_test.jsx b/tests/ui/perfherder/replicates_graph_test.jsx
@@ -23,8 +23,31 @@ 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',
@@ -35,8 +58,31 @@ 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',
@@ -52,8 +98,26 @@ 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,
         },
@@ -70,12 +134,26 @@ 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',

diff --git a/tests/ui/push-health/details/DetailsPanel_test.jsx b/tests/ui/push-health/details/DetailsPanel_test.jsx
@@ -65,7 +65,8 @@ 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,
           },
         ],

diff --git a/ui/helpers/job.js b/ui/helpers/job.js
@@ -237,8 +237,7 @@ 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);

diff --git a/ui/helpers/performance.js b/ui/helpers/performance.js
@@ -1,4 +1,5 @@
 // This file may export additional functions.
+/* eslint-disable import/prefer-default-export */
 
 import TaskclusterModel from '../models/taskcluster';
 

diff --git a/ui/helpers/revision.js b/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)=.*/, '');

diff --git a/ui/helpers/sort.js b/ui/helpers/sort.js
@@ -1,3 +1,4 @@
+// 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

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

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