fix(app): Don't render ProtocolRunHeader TerminalBanner on CurrentRun

app/src/organisms/Devices/ProtocolRun/ProtocolRunHeader.tsx

@@ -364,7 +364,7 @@ export function ProtocolRunHeader({
         CANCELLABLE_STATUSES.includes(runStatus) ? (
           <Banner type="warning">{t('shared:close_robot_door')}</Banner>
         ) : null}
-        {isRunCurrent ? (
+        {mostRecentRunId === runId ? (
           <TerminalRunBanner
             {...{
               runStatus,
@@ -455,7 +455,7 @@ export function ProtocolRunHeader({
         ) : null}
         {showDropTipWizard &&
         pipettesWithTip[0]?.specs != null &&
-        isRunCurrent ? (
+        mostRecentRunId === runId ? (
           <DropTipWizard
             robotType={isFlex ? FLEX_ROBOT_TYPE : OT2_ROBOT_TYPE}
             mount={pipettesWithTip[0].mount}

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

@@ -1054,26 +1054,4 @@ describe('ProtocolRunHeader', () => {
       expect(queryByText('Tips may be attached.')).not.toBeInTheDocument()
     })
   })
-
-  it('does not show the drop tip banner when the run is not over', async () => {
-    when(mockUseRunQuery)
-      .calledWith(RUN_ID)
-      .mockReturnValue({
-        data: {
-          data: {
-            ...mockIdleUnstartedRun,
-            current: false,
-            status: RUN_STATUS_SUCCEEDED,
-          },
-        },
-      } as UseQueryResult<Run>)
-    when(mockUseRunStatus)
-      .calledWith(RUN_ID)
-      .mockReturnValue(RUN_STATUS_SUCCEEDED)
-
-    const [{ queryByText }] = render()
-    await waitFor(() => {
-      expect(queryByText('Tips may be attached.')).not.toBeInTheDocument()
-    })
-  })
 })

app/src/organisms/ProtocolUpload/hooks/__tests__/useMostRecentRunId.test.tsx

@@ -31,6 +31,15 @@ describe('useMostRecentRunId hook', () => {
 
     const { result } = renderHook(useMostRecentRunId)
 
+    expect(result.current).toBeNull()
+  })
+  it('should return null if no run data exists', async () => {
+    when(mockUseAllRunsQuery)
+      .calledWith()
+      .mockReturnValue({ data: { data: null } } as any)
+
+    const { result } = renderHook(useMostRecentRunId)
+
     expect(result.current).toBeNull()
   })
 })

app/src/organisms/ProtocolUpload/hooks/useMostRecentRunId.ts

@@ -1,8 +1,9 @@
 import { useAllRunsQuery } from '@opentrons/react-api-client'
+import last from 'lodash/last'
 
 export function useMostRecentRunId(): string | null {
   const { data: allRuns } = useAllRunsQuery()
-  return allRuns != null && allRuns.data.length > 0
-    ? allRuns.data[allRuns.data.length - 1].id
+  return allRuns != null && allRuns.data?.length > 0
+    ? last(allRuns.data)?.id ?? null
     : null
 }