-
Notifications
You must be signed in to change notification settings - Fork 179
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
refactor(app): move robot lights controls to state.robotControls (#4631)
- Loading branch information
Showing
26 changed files
with
1,044 additions
and
287 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
195 changes: 195 additions & 0 deletions
195
app/src/components/RobotSettings/__tests__/ControlsCard.test.js
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,195 @@ | ||
| // @flow | ||
| import * as React from 'react' | ||
| import { Provider } from 'react-redux' | ||
| import { mount } from 'enzyme' | ||
|
|
||
| import * as RobotControls from '../../../robot-controls' | ||
| import * as RobotAdmin from '../../../robot-admin' | ||
| import * as RobotSelectors from '../../../robot/selectors' | ||
| import { ControlsCard } from '../ControlsCard' | ||
| import { LabeledToggle, LabeledButton } from '@opentrons/components' | ||
| import { CONNECTABLE, REACHABLE } from '../../../discovery' | ||
|
|
||
| import type { State } from '../../../types' | ||
| import type { ViewableRobot } from '../../../discovery/types' | ||
|
|
||
| jest.mock('../../../robot-controls/selectors') | ||
| jest.mock('../../../robot/selectors') | ||
|
|
||
| const mockRobot: ViewableRobot = ({ | ||
| name: 'robot-name', | ||
| connected: true, | ||
| status: CONNECTABLE, | ||
| }: any) | ||
|
|
||
| const mockUnconnectableRobot: ViewableRobot = ({ | ||
| name: 'robot-name', | ||
| connected: true, | ||
| status: REACHABLE, | ||
| }: any) | ||
|
|
||
| const mockGetLightsOn: JestMockFn< | ||
| [State, string], | ||
| $Call<typeof RobotControls.getLightsOn, State, string> | ||
| > = RobotControls.getLightsOn | ||
|
|
||
| const mockGetIsRunning: JestMockFn< | ||
| [State], | ||
| $Call<typeof RobotSelectors.getIsRunning, State> | ||
| > = RobotSelectors.getIsRunning | ||
|
|
||
| describe('ControlsCard', () => { | ||
| let mockStore | ||
|
|
||
| const getDeckCalButton = wrapper => | ||
| wrapper | ||
| .find({ label: 'Calibrate deck' }) | ||
| .find(LabeledButton) | ||
| .find('button') | ||
|
|
||
| const getHomeButton = wrapper => | ||
| wrapper | ||
| .find({ label: 'Home all axes' }) | ||
| .find(LabeledButton) | ||
| .find('button') | ||
|
|
||
| const getRestartButton = wrapper => | ||
| wrapper | ||
| .find({ label: 'Restart robot' }) | ||
| .find(LabeledButton) | ||
| .find('button') | ||
|
|
||
| const getLightsButton = wrapper => | ||
| wrapper.find({ label: 'Lights' }).find(LabeledToggle) | ||
|
|
||
| beforeEach(() => { | ||
| mockStore = { | ||
| subscribe: () => {}, | ||
| getState: () => ({ mockState: true }), | ||
| dispatch: jest.fn(), | ||
| } | ||
| }) | ||
|
|
||
| afterEach(() => { | ||
| jest.resetAllMocks() | ||
| }) | ||
|
|
||
| test('calls fetchLights on mount', () => { | ||
| mount( | ||
| <Provider store={mockStore}> | ||
| <ControlsCard robot={mockRobot} calibrateDeckUrl="/deck/calibrate" /> | ||
| </Provider> | ||
| ) | ||
|
|
||
| expect(mockStore.dispatch).toHaveBeenCalledWith( | ||
| RobotControls.fetchLights(mockRobot.name) | ||
| ) | ||
| }) | ||
|
|
||
| test('calls updateLights with toggle on button click', () => { | ||
| mockGetLightsOn.mockReturnValue(true) | ||
|
|
||
| const wrapper = mount( | ||
| <Provider store={mockStore}> | ||
| <ControlsCard robot={mockRobot} calibrateDeckUrl="/deck/calibrate" /> | ||
| </Provider> | ||
| ) | ||
|
|
||
| getLightsButton(wrapper).invoke('onClick')() | ||
|
|
||
| expect(mockStore.dispatch).toHaveBeenCalledWith( | ||
| RobotControls.updateLights(mockRobot.name, false) | ||
| ) | ||
| }) | ||
|
|
||
| test('calls restartRobot on button click', () => { | ||
| const wrapper = mount( | ||
| <Provider store={mockStore}> | ||
| <ControlsCard robot={mockRobot} calibrateDeckUrl="/deck/calibrate" /> | ||
| </Provider> | ||
| ) | ||
|
|
||
| getRestartButton(wrapper).invoke('onClick')() | ||
|
|
||
| expect(mockStore.dispatch).toHaveBeenCalledWith( | ||
| RobotAdmin.restartRobot(mockRobot.name) | ||
| ) | ||
| }) | ||
|
|
||
| // TODO(mc, 2019-12-17): enable test when POST /robot/home is epic-based | ||
| test.skip('calls home on button click', () => { | ||
| const wrapper = mount( | ||
| <Provider store={mockStore}> | ||
| <ControlsCard robot={mockRobot} calibrateDeckUrl="/deck/calibrate" /> | ||
| </Provider> | ||
| ) | ||
|
|
||
| getHomeButton(wrapper).invoke('onClick')() | ||
|
|
||
| // expect(mockStore.dispatch).toHaveBeenCalledWith(RobotControls.home(mockRobot.name)) | ||
| }) | ||
|
|
||
| test('DC, home, and restart buttons enabled if connected and not running', () => { | ||
| mockGetIsRunning.mockReturnValue(false) | ||
|
|
||
| const wrapper = mount( | ||
| <Provider store={mockStore}> | ||
| <ControlsCard | ||
| robot={mockUnconnectableRobot} | ||
| calibrateDeckUrl="/deck/calibrate" | ||
| /> | ||
| </Provider> | ||
| ) | ||
|
|
||
| expect(getDeckCalButton(wrapper).prop('disabled')).toBe(true) | ||
| expect(getHomeButton(wrapper).prop('disabled')).toBe(true) | ||
| expect(getRestartButton(wrapper).prop('disabled')).toBe(true) | ||
| }) | ||
|
|
||
| test('DC, home, and restart buttons disabled if not connectable', () => { | ||
| const wrapper = mount( | ||
| <Provider store={mockStore}> | ||
| <ControlsCard | ||
| robot={mockUnconnectableRobot} | ||
| calibrateDeckUrl="/deck/calibrate" | ||
| /> | ||
| </Provider> | ||
| ) | ||
|
|
||
| expect(getDeckCalButton(wrapper).prop('disabled')).toBe(true) | ||
| expect(getHomeButton(wrapper).prop('disabled')).toBe(true) | ||
| expect(getRestartButton(wrapper).prop('disabled')).toBe(true) | ||
| }) | ||
|
|
||
| test('DC, home, and restart buttons disabled if not connected', () => { | ||
| const mockRobot: ViewableRobot = ({ | ||
| name: 'robot-name', | ||
| connected: false, | ||
| status: CONNECTABLE, | ||
| }: any) | ||
|
|
||
| const wrapper = mount( | ||
| <Provider store={mockStore}> | ||
| <ControlsCard robot={mockRobot} calibrateDeckUrl="/deck/calibrate" /> | ||
| </Provider> | ||
| ) | ||
|
|
||
| expect(getDeckCalButton(wrapper).prop('disabled')).toBe(true) | ||
| expect(getHomeButton(wrapper).prop('disabled')).toBe(true) | ||
| expect(getRestartButton(wrapper).prop('disabled')).toBe(true) | ||
| }) | ||
|
|
||
| test('DC, home, and restart buttons disabled if protocol running', () => { | ||
| mockGetIsRunning.mockReturnValue(true) | ||
|
|
||
| const wrapper = mount( | ||
| <Provider store={mockStore}> | ||
| <ControlsCard robot={mockRobot} calibrateDeckUrl="/deck/calibrate" /> | ||
| </Provider> | ||
| ) | ||
|
|
||
| expect(getDeckCalButton(wrapper).prop('disabled')).toBe(true) | ||
| expect(getHomeButton(wrapper).prop('disabled')).toBe(true) | ||
| expect(getRestartButton(wrapper).prop('disabled')).toBe(true) | ||
| }) | ||
| }) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.