chore(app): change default labware view to map (#17422)

Author: TamarZanzouri

app/src/organisms/ODD/ProtocolSetup/ProtocolSetupLabware/__tests__/ProtocolSetupLabware.test.tsx

@@ -101,6 +101,7 @@ describe('ProtocolSetupLabware', () => {
 
   it('renders the Labware Setup page', () => {
     render()
+    fireEvent.click(screen.getByRole('button', { name: 'List View' }))
     screen.getByText('Labware')
     screen.getByText('Labware name')
     screen.getByText('Location')
@@ -115,9 +116,10 @@ describe('ProtocolSetupLabware', () => {
 
   it('should toggle between map view and list view', () => {
     render()
+    expect(screen.queryByText('Map View')).toBeNull()
+    fireEvent.click(screen.getByRole('button', { name: 'List View' }))
     expect(screen.queryByText('List View')).toBeNull()
     fireEvent.click(screen.getByRole('button', { name: 'Map View' }))
-    expect(screen.queryByText('Map View')).toBeNull()
     fireEvent.click(screen.getByRole('button', { name: 'List View' }))
     screen.getByText('Labware')
     screen.getByText('Labware name')
@@ -126,6 +128,7 @@ describe('ProtocolSetupLabware', () => {
 
   it('sends a latch-close command when the labware latch is open and the button is clicked', () => {
     render()
+    fireEvent.click(screen.getByRole('button', { name: 'List View' }))
     fireEvent.click(screen.getByText('Labware Latch'))
     expect(mockCreateLiveCommand).toHaveBeenCalledWith({
       command: {
@@ -144,6 +147,7 @@ describe('ProtocolSetupLabware', () => {
       refetch: mockRefetch,
     } as any)
     render()
+    fireEvent.click(screen.getByRole('button', { name: 'List View' }))
     fireEvent.click(screen.getByText('Labware Latch'))
     expect(mockCreateLiveCommand).toHaveBeenCalledWith({
       command: {
@@ -162,15 +166,16 @@ describe('ProtocolSetupLabware', () => {
     )
 
     render()
+    fireEvent.click(screen.getByRole('button', { name: 'List View' }))
     screen.getByText('Opening...')
   })
 
   it('shows closing transition state of the labware latch button', () => {
     vi.mocked(useModulesQuery).mockReturnValue(
       mockUseModulesQueryClosing as any
     )
-
     render()
+    fireEvent.click(screen.getByRole('button', { name: 'List View' }))
     screen.getByText('Closing...')
   })
 
@@ -180,6 +185,7 @@ describe('ProtocolSetupLabware', () => {
     )
 
     render()
+    fireEvent.click(screen.getByRole('button', { name: 'List View' }))
     screen.getByText('Open')
   })
 })

app/src/organisms/ODD/ProtocolSetup/ProtocolSetupLabware/index.tsx

@@ -83,7 +83,7 @@ export function ProtocolSetupLabware({
   setIsConfirmed,
 }: ProtocolSetupLabwareProps): JSX.Element {
   const { t } = useTranslation('protocol_setup')
-  const [showMapView, setShowMapView] = useState<boolean>(false)
+  const [showMapView, setShowMapView] = useState<boolean>(true)
   const [
     showLabwareDetailsModal,
     setShowLabwareDetailsModal,