remove redundant mocks + tests

Opentrons · Dec 6, 2023 · 6060a8f · 6060a8f
1 parent 4676a72
commit 6060a8f
Showing 1 changed file with 1 addition and 69 deletions.
diff --git a/app/src/organisms/Devices/ProtocolRun/__tests__/BackToTopButton.test.tsx b/app/src/organisms/Devices/ProtocolRun/__tests__/BackToTopButton.test.tsx
@@ -9,11 +9,6 @@ import {
   useTrackEvent,
   ANALYTICS_PROTOCOL_PROCEED_TO_RUN,
 } from '../../../../redux/analytics'
-import {
-  useRunCalibrationStatus,
-  useRunHasStarted,
-  useUnmatchedModulesForProtocol,
-} from '../../hooks'
 import { BackToTopButton } from '../BackToTopButton'
 
 jest.mock('@opentrons/components', () => {
@@ -24,17 +19,7 @@ jest.mock('@opentrons/components', () => {
   }
 })
 jest.mock('../../../../redux/analytics')
-jest.mock('../../hooks')
 
-const mockUseUnmatchedModulesForProtocol = useUnmatchedModulesForProtocol as jest.MockedFunction<
-  typeof useUnmatchedModulesForProtocol
->
-const mockUseRunCalibrationStatus = useRunCalibrationStatus as jest.MockedFunction<
-  typeof useRunCalibrationStatus
->
-const mockUseRunHasStarted = useRunHasStarted as jest.MockedFunction<
-  typeof useRunHasStarted
->
 const mockUseTrackEvent = useTrackEvent as jest.MockedFunction<
   typeof useTrackEvent
 >
@@ -62,20 +47,6 @@ let mockTrackEvent: jest.Mock
 
 describe('BackToTopButton', () => {
   beforeEach(() => {
-    when(mockUseUnmatchedModulesForProtocol)
-      .calledWith(ROBOT_NAME, RUN_ID)
-      .mockReturnValue({
-        missingModuleIds: [],
-        remainingAttachedModules: [],
-      })
-
-    when(mockUseRunCalibrationStatus)
-      .calledWith(ROBOT_NAME, RUN_ID)
-      .mockReturnValue({
-        complete: true,
-      })
-    when(mockUseRunHasStarted).calledWith(RUN_ID).mockReturnValue(false)
-
     mockTrackEvent = jest.fn()
     when(mockUseTrackEvent).calledWith().mockReturnValue(mockTrackEvent)
   })
@@ -102,46 +73,7 @@ describe('BackToTopButton', () => {
     })
   })
 
-  it('should not disabled with modules not connected tooltip when there are missing moduleIds', () => {
-    when(mockUseUnmatchedModulesForProtocol)
-      .calledWith(ROBOT_NAME, RUN_ID)
-      .mockReturnValue({
-        missingModuleIds: ['temperatureModuleV1'],
-        remainingAttachedModules: [],
-      })
-    const { getByRole } = render()
-    const button = getByRole('button', { name: 'Back to top' })
-    expect(button).not.toBeDisabled()
-  })
-  it('should not be disabled with modules not connected and calibration not completed tooltip if missing cal and moduleIds', async () => {
-    when(mockUseUnmatchedModulesForProtocol)
-      .calledWith(ROBOT_NAME, RUN_ID)
-      .mockReturnValue({
-        missingModuleIds: ['temperatureModuleV1'],
-        remainingAttachedModules: [],
-      })
-    when(mockUseRunCalibrationStatus)
-      .calledWith(ROBOT_NAME, RUN_ID)
-      .mockReturnValue({
-        complete: false,
-      })
-    const { getByRole } = render()
-    const button = getByRole('button', { name: 'Back to top' })
-    expect(button).not.toBeDisabled()
-  })
-  it('should not be disabled with calibration not complete tooltip when calibration not complete', async () => {
-    when(mockUseRunCalibrationStatus)
-      .calledWith(ROBOT_NAME, RUN_ID)
-      .mockReturnValue({
-        complete: false,
-      })
-    const { getByRole } = render()
-    const button = getByRole('button', { name: 'Back to top' })
-    expect(button).not.toBeDisabled()
-  })
-  it('should not be disabled with protocol run started tooltip when run has started', async () => {
-    when(mockUseRunHasStarted).calledWith(RUN_ID).mockReturnValue(true)
-
+  it('should always be enabled', () => {
     const { getByRole } = render()
     const button = getByRole('button', { name: 'Back to top' })
     expect(button).not.toBeDisabled()