Fix lint warnings

Author: kraenhansen

packages/compass-e2e-tests/helpers/commands/codemirror.ts

@@ -9,10 +9,13 @@ export async function getCodemirrorEditorText(
   // we have to find an instance of the editor and get the text directly from
   // its state
   const editorContents = await browser.execute(function (selector) {
-    const node =
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Accessing private Codemirror state
+    const node: any =
+      // eslint-disable-next-line no-restricted-globals
       document.querySelector(`${selector} [data-codemirror]`) ??
+      // eslint-disable-next-line no-restricted-globals
       document.querySelector(`${selector}[data-codemirror]`);
-    return (node as any)._cm.state.sliceDoc() as string;
+    return node._cm.state.sliceDoc() as string;
   }, selector);
   return editorContents;
 }
@@ -26,10 +29,12 @@ export async function getCodemirrorEditorTextAll(
   // its state
   const editorContents = await browser.execute(function (selector) {
     const editors = Array.from(
+      // eslint-disable-next-line no-restricted-globals
       document.querySelectorAll(`${selector} [data-codemirror]`)
     );
-    return editors.map((node) => {
-      return (node as any)._cm.state.sliceDoc() as string;
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Accessing private Codemirror state
+    return editors.map((node: any) => {
+      return node._cm.state.sliceDoc() as string;
     });
   }, selector);
   return editorContents;
@@ -42,10 +47,13 @@ export async function setCodemirrorEditorValue(
 ) {
   await browser.execute(
     function (selector, text) {
-      const node =
+      // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Accessing private Codemirror state
+      const node: any =
+        // eslint-disable-next-line no-restricted-globals
         document.querySelector(`${selector} [data-codemirror]`) ??
+        // eslint-disable-next-line no-restricted-globals
         document.querySelector(`${selector}[data-codemirror]`);
-      const editor = (node as any)._cm;
+      const editor = node._cm;
 
       editor.dispatch({
         changes: {

packages/compass-e2e-tests/helpers/commands/scroll-to-virtual-item.ts

@@ -7,6 +7,7 @@ type ItemConfig = {
     browser: CompassBrowser,
     selector: string
   ) => Promise<boolean>;
+  // eslint-disable-next-line no-restricted-globals
   getScrollContainer: (parent: Element | null) => ChildNode | null | undefined;
 };
 
@@ -23,6 +24,7 @@ const gridConfig: ItemConfig = {
     const length = await browser.$$(`${selector} [role="row"]`).length;
     return !!(rowCount && length);
   },
+  // eslint-disable-next-line no-restricted-globals
   getScrollContainer: (parent: Element | null) => {
     return parent?.firstChild;
   },
@@ -37,6 +39,7 @@ const treeConfig: ItemConfig = {
   ) => {
     return (await browser.$$(`${selector} [role="treeitem"]`).length) > 0;
   },
+  // eslint-disable-next-line no-restricted-globals
   getScrollContainer: (parent: Element | null) => {
     return parent?.firstChild?.firstChild;
   },
@@ -63,6 +66,7 @@ export async function scrollToVirtualItem(
   // scroll content
   const [scrollHeight, totalHeight] = await browser.execute(
     (selector, getScrollContainerString) => {
+      // eslint-disable-next-line no-restricted-globals
       const container = document.querySelector(selector);
       const scrollContainer = eval(getScrollContainerString)(container);
       const heightContainer = scrollContainer?.firstChild;
@@ -115,6 +119,7 @@ export async function scrollToVirtualItem(
       // scroll for another screen
       await browser.execute(
         (selector, nextScrollTop, getScrollContainerString) => {
+          // eslint-disable-next-line no-restricted-globals
           const container = document.querySelector(selector);
           const scrollContainer = eval(getScrollContainerString)(container);
           if (!scrollContainer) {

packages/compass-e2e-tests/helpers/commands/select-file.ts

@@ -8,7 +8,7 @@ export async function selectFile(
   // HACK: the <input type="file"> is not displayed so we can't interact
   // with it until we change that.
   await browser.execute((selector) => {
-    // eslint-disable-next-line no-undef
+    // eslint-disable-next-line no-restricted-globals
     const f = document.querySelector(selector);
     if (f) {
       f.removeAttribute('style');
@@ -24,7 +24,7 @@ export async function selectFile(
 
   // HACK: undo what we just did
   await browser.execute((selector) => {
-    // eslint-disable-next-line no-undef
+    // eslint-disable-next-line no-restricted-globals
     const f = document.querySelector(selector);
     if (f) {
       f.setAttribute('style', 'display: none');

packages/compass-e2e-tests/helpers/commands/set-export-filename.ts

@@ -13,6 +13,7 @@ export async function setExportFilename(
   await expect(fs.stat(filename)).to.be.rejected;
 
   await browser.execute(function (f) {
+    // eslint-disable-next-line no-restricted-globals
     document.dispatchEvent(
       new CustomEvent('selectExportFileName', { detail: f })
     );

packages/compass-e2e-tests/helpers/compass.ts

@@ -395,7 +395,8 @@ export class Compass {
 
   async stopBrowser(): Promise<void> {
     const logging: any[] = await this.browser.execute(function () {
-      return (window as any).logging;
+      // eslint-disable-next-line no-restricted-globals
+      return 'logging' in window && (window.logging as any);
     });
     const lines = logging.map((log) => JSON.stringify(log));
     const text = lines.join('\n');
@@ -1048,17 +1049,18 @@ export async function init(
     // larger window for more consistent results
     const [width, height] = await browser.execute(() => {
       // in case setWindowSize() below doesn't work
+      // eslint-disable-next-line no-restricted-globals
       window.resizeTo(window.screen.availWidth, window.screen.availHeight);
-
+      // eslint-disable-next-line no-restricted-globals
       return [window.screen.availWidth, window.screen.availHeight];
     });
     // getting available width=1512, height=944 in electron on mac which is arbitrary
     debug(`available width=${width}, height=${height}`);
     try {
       // window.resizeTo() doesn't work on firefox
       await browser.setWindowSize(width, height);
-    } catch (err: any) {
-      console.error(err?.stack);
+    } catch (err) {
+      console.error(err instanceof Error ? err.stack : err);
     }
   } else {
     await browser.execute(() => {

packages/compass-e2e-tests/helpers/telemetry.ts

@@ -36,7 +36,8 @@ function startFakeTelemetry(): Promise<Telemetry> {
     },
     pollForEvents: async (browser: CompassBrowser): Promise<void> => {
       tracking = await browser.execute(function () {
-        return (window as any).tracking;
+        // eslint-disable-next-line no-restricted-globals
+        return 'tracking' in window && (window.tracking as any);
       });
 
       neverFetched = false;

packages/compass-e2e-tests/tests/auto-connect.test.ts

@@ -231,6 +231,7 @@ describe('Automatically connecting from the command line', function () {
         }
       );
       await browser.execute(() => {
+        // eslint-disable-next-line no-restricted-globals
         location.reload();
       });
       await browser.waitForConnectionResult(
@@ -241,6 +242,7 @@ describe('Automatically connecting from the command line', function () {
       );
       await browser.disconnectAll();
       await browser.execute(() => {
+        // eslint-disable-next-line no-restricted-globals
         location.reload();
       });
     } catch (err: any) {

packages/compass-e2e-tests/tests/collection-aggregations-tab.test.ts

@@ -124,6 +124,7 @@ describe('Collection aggregations tab', function () {
     await browser.connectToDefaults();
     // set guide cue to not show up
     await browser.execute((key) => {
+      // eslint-disable-next-line no-restricted-globals
       localStorage.setItem(key, 'true');
     }, STAGE_WIZARD_GUIDE_CUE_STORAGE_KEY);
 

packages/compass-e2e-tests/tests/intercom.test.ts

@@ -44,7 +44,8 @@ describe('Intercom integration', function () {
     await compass.browser.waitUntil(
       () => {
         return compass.browser.execute(() => {
-          return typeof (window as any).Intercom === 'function';
+          // eslint-disable-next-line no-restricted-globals
+          return 'Intercom' in window && typeof window.Intercom === 'function';
         });
       },
       {