do not show module setup banner if no pipette is attached

Author: ahiuchingau

app/src/molecules/UpdateBanner/__tests__/UpdateBanner.test.tsx

@@ -123,6 +123,21 @@ describe('Module Update Banner', () => {
     expect(screen.queryByText('Setup module')).not.toBeInTheDocument()
   })
 
+  it('should not render module setup banner if pipette calibration is required', () => {
+    props = {
+      ...props,
+      updateType: 'setup',
+      calibratePipetteRequired: true,
+    }
+    render(props)
+    expect(
+      screen.queryByLabelText(
+        'ModuleCard_calibration_update_banner_test_number'
+      )
+    ).not.toBeInTheDocument()
+    expect(screen.queryByText('Setup module')).not.toBeInTheDocument()
+  })
+
   it('should not render a module setup link if pipette firmware update is required', () => {
     props = {
       ...props,

app/src/molecules/UpdateBanner/index.tsx

@@ -52,13 +52,13 @@ export const UpdateBanner = ({
   }
 
   const canProceed =
-    updateType === 'calibration'
-      ? !isEstopNotDisengaged &&
+    updateType === 'firmware'
+      ? true
+      : !isEstopNotDisengaged &&
         !isTooHot &&
         !attachPipetteRequired &&
         !calibratePipetteRequired &&
         !updatePipetteFWRequired
-      : true
 
   const getMessage = (): string => {
     switch (updateType) {
@@ -81,7 +81,12 @@ export const UpdateBanner = ({
 
   const isFlex = useIsFlex(robotName)
   // Only show banner for needing firmware update if robot is an OT-2
-  if (!isFlex && updateType !== 'firmware') return null
+  if (!isFlex && updateType !== 'firmware') {
+    return null
+  }
+  if (updateType === 'setup' && !canProceed) {
+    return null
+  }
   return (
     <Flex
       data-testid={`ModuleCard_${updateType}_update_banner_${serialNumber}`}