lint fixes

Author: mjhuff

app-shell/src/main.ts

@@ -134,7 +134,7 @@ function createRendererLogger(): Logger {
   return logger
 }
 
-function installDevtools(): Promise<void | Logger> {
+function installDevtools(): Promise<Logger> {
   const extensions = [
     electronDevtoolsInstaller.REACT_DEVELOPER_TOOLS,
     electronDevtoolsInstaller.REDUX_DEVTOOLS,
@@ -158,6 +158,8 @@ function installDevtools(): Promise<void | Logger> {
       })
   } else {
     log.warn('could not resolve electron dev tools installer')
-    return Promise.reject('could not resolve electron dev tools installer')
+    return Promise.reject(
+      new Error('could not resolve electron dev tools installer')
+    )
   }
 }

app/src/__testing-utils__/matchers.ts

@@ -1,4 +1,3 @@
-import { when } from 'vitest-when'
 import type { Matcher } from '@testing-library/react'
 
 // Match things like <p>Some <strong>nested</strong> text</p>

app/src/atoms/text/__tests__/StyledText.test.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
-import { describe, it, expect, vi, beforeEach } from 'vitest'
+import { describe, it, expect } from 'vitest'
 import '@testing-library/jest-dom/vitest'
-import { fireEvent, screen } from '@testing-library/react'
+import { screen } from '@testing-library/react'
 import { TYPOGRAPHY } from '@opentrons/components'
 import { StyledText } from '../'
 import { renderWithProviders } from '../../../__testing-utils__'

app/src/molecules/PythonLabwareOffsetSnippet/__tests__/createSnippet.test.ts

@@ -1,7 +1,10 @@
 import { describe, it, expect } from 'vitest'
 import '@testing-library/jest-dom/vitest'
-import { transfer_settings } from '@opentrons/shared-data'
-import { ModuleModel, CompletedProtocolAnalysis } from '@opentrons/shared-data'
+import {
+  transfer_settings,
+  ModuleModel,
+  CompletedProtocolAnalysis,
+} from '@opentrons/shared-data'
 import { createSnippet } from '../createSnippet'
 
 const protocolWithMagTempTC = ({

app/src/organisms/AdvancedSettings/ClearUnavailableRobots.tsx

@@ -24,7 +24,7 @@ import { TertiaryButton } from '../../atoms/buttons'
 import { ERROR_TOAST, SUCCESS_TOAST } from '../../atoms/Toast'
 import { useToaster } from '../../organisms/ToasterOven'
 import { LegacyModal } from '../../molecules/LegacyModal'
-import { Portal, getTopPortalEl } from '../../App/portal'
+import { getTopPortalEl } from '../../App/portal'
 import {
   clearDiscoveryCache,
   getReachableRobots,
@@ -63,41 +63,43 @@ export function ClearUnavailableRobots(): JSX.Element {
   } = useConditionalConfirm(handleDeleteUnavailRobots, true)
   return (
     <>
-      {showConfirmDeleteUnavailRobots ? createPortal(
-          <LegacyModal
-            type="warning"
-            title={t('clear_unavailable_robots')}
-            onClose={cancelExit}
-          >
-            <StyledText as="p">{t('clearing_cannot_be_undone')}</StyledText>
-            <Flex
-              flexDirection={DIRECTION_ROW}
-              paddingTop={SPACING.spacing32}
-              justifyContent={JUSTIFY_FLEX_END}
+      {showConfirmDeleteUnavailRobots
+        ? createPortal(
+            <LegacyModal
+              type="warning"
+              title={t('clear_unavailable_robots')}
+              onClose={cancelExit}
             >
+              <StyledText as="p">{t('clearing_cannot_be_undone')}</StyledText>
               <Flex
-                paddingRight={SPACING.spacing4}
-                data-testid="AdvancedSettings_ConfirmClear_Cancel"
+                flexDirection={DIRECTION_ROW}
+                paddingTop={SPACING.spacing32}
+                justifyContent={JUSTIFY_FLEX_END}
               >
-                <Btn
-                  onClick={cancelExit}
-                  textTransform={TYPOGRAPHY.textTransformCapitalize}
-                  color={COLORS.blue50}
-                  fontWeight={TYPOGRAPHY.fontWeightSemiBold}
-                  marginRight={SPACING.spacing32}
+                <Flex
+                  paddingRight={SPACING.spacing4}
+                  data-testid="AdvancedSettings_ConfirmClear_Cancel"
                 >
-                  {t('shared:cancel')}
-                </Btn>
-              </Flex>
-              <Flex data-testid="AdvancedSettings_ConfirmClear_Proceed">
-                <AlertPrimaryButton onClick={confirmDeleteUnavailRobots}>
-                  {t('clear_confirm')}
-                </AlertPrimaryButton>
+                  <Btn
+                    onClick={cancelExit}
+                    textTransform={TYPOGRAPHY.textTransformCapitalize}
+                    color={COLORS.blue50}
+                    fontWeight={TYPOGRAPHY.fontWeightSemiBold}
+                    marginRight={SPACING.spacing32}
+                  >
+                    {t('shared:cancel')}
+                  </Btn>
+                </Flex>
+                <Flex data-testid="AdvancedSettings_ConfirmClear_Proceed">
+                  <AlertPrimaryButton onClick={confirmDeleteUnavailRobots}>
+                    {t('clear_confirm')}
+                  </AlertPrimaryButton>
+                </Flex>
               </Flex>
-            </Flex>
-          </LegacyModal>,
-          getTopPortalEl()
-      ) : null}
+            </LegacyModal>,
+            getTopPortalEl()
+          )
+        : null}
       <Flex
         alignItems={ALIGN_CENTER}
         justifyContent={JUSTIFY_SPACE_BETWEEN}

app/src/organisms/AdvancedSettings/__tests__/ClearUnavailableRobots.test.tsx

@@ -1,5 +1,5 @@
 import * as React from 'react'
-import { screen, fireEvent, waitFor } from '@testing-library/react'
+import { screen, fireEvent } from '@testing-library/react'
 import { describe, it, expect, vi, beforeEach } from 'vitest'
 import { useConditionalConfirm } from '@opentrons/components'
 import { i18n } from '../../../i18n'

app/src/organisms/ApplyHistoricOffsets/hooks/__tests__/getLabwareLocationCombos.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, expect, beforeEach } from 'vitest'
+import { describe, it, expect } from 'vitest'
 import {
   getLabwareDefURI,
   opentrons96PcrAdapterV1,

app/src/organisms/Devices/ProtocolRun/SetupLiquids/__tests__/LiquidDetailCard.test.tsx

@@ -1,6 +1,6 @@
 import * as React from 'react'
 import { fireEvent, screen } from '@testing-library/react'
-import { describe, it, beforeEach, vi, expect } from 'vitest'
+import { describe, it, beforeEach, vi, expect, Mock } from 'vitest'
 
 import { SPACING, COLORS } from '@opentrons/components'
 
@@ -16,8 +16,6 @@ import {
 import { getIsOnDevice } from '../../../../../redux/config'
 import { LiquidDetailCard } from '../LiquidDetailCard'
 
-import { Mock } from 'vitest'
-
 vi.mock('../../../../../redux/analytics')
 vi.mock('../../../../../redux/config')
 

app/src/organisms/Devices/ProtocolRun/SetupLiquids/__tests__/SetupLiquids.test.tsx

@@ -1,14 +1,13 @@
 import * as React from 'react'
 import { describe, it, beforeEach, vi } from 'vitest'
-import { screen } from '@testing-library/react'
+import { screen, fireEvent } from '@testing-library/react'
 
 import { renderWithProviders } from '../../../../../__testing-utils__'
 import { i18n } from '../../../../../i18n'
 import { SetupLiquids } from '../index'
 import { SetupLiquidsList } from '../SetupLiquidsList'
 import { SetupLiquidsMap } from '../SetupLiquidsMap'
 import { BackToTopButton } from '../../BackToTopButton'
-import { fireEvent } from '@testing-library/react'
 
 vi.mock('../SetupLiquidsList')
 vi.mock('../SetupLiquidsMap')

app/src/organisms/Devices/ProtocolRun/__tests__/SetupRobotCalibration.test.tsx

@@ -77,27 +77,27 @@ describe('SetupRobotCalibration', () => {
   })
 
   it('renders deck, pipette, and tip length calibration components', () => {
-    render()[0]
+    render()
     screen.getByText('Mock SetupDeckCalibration')
     screen.getByText('Mock SetupInstrumentCalibration')
     screen.getByText('Mock SetupTipLengthCalibration')
   })
 
   it('renders only pipette calibration component for Flex', () => {
     when(vi.mocked(useIsFlex)).calledWith(ROBOT_NAME).thenReturn(true)
-    render()[0]
+    render()
     expect(screen.queryByText('Mock SetupDeckCalibration')).toBeNull()
     screen.getByText('Mock SetupInstrumentCalibration')
     expect(screen.queryByText('Mock SetupTipLengthCalibration')).toBeNull()
   })
 
   it('changes Proceed CTA copy based on next step', () => {
-    render({ nextStep: 'labware_setup_step' })[0]
+    render({ nextStep: 'labware_setup_step' })
     screen.getByRole('button', { name: 'Proceed to labware' })
   })
 
   it('calls the expandStep function and tracks the analytics event on click', () => {
-    render()[0]
+    render()
     fireEvent.click(screen.getByRole('button', { name: 'Proceed to modules' }))
     expect(mockExpandStep).toHaveBeenCalled()
     expect(mockTrackEvent).toHaveBeenCalledWith({
@@ -107,7 +107,7 @@ describe('SetupRobotCalibration', () => {
   })
 
   it('does not call the expandStep function on click if calibration is not complete', () => {
-    render({ calibrationStatus: { complete: false } })[0]
+    render({ calibrationStatus: { complete: false } })
     const button = screen.getByRole('button', { name: 'Proceed to modules' })
     expect(button).toBeDisabled()
     fireEvent.click(button)
@@ -116,7 +116,7 @@ describe('SetupRobotCalibration', () => {
 
   it('does not call the expandStep function on click if run has started', () => {
     when(vi.mocked(useRunHasStarted)).calledWith(RUN_ID).thenReturn(true)
-    render()[0]
+    render()
     const button = screen.getByRole('button', { name: 'Proceed to modules' })
     expect(button).toBeDisabled()
     fireEvent.click(button)