fix(app): do not prompt module setup if no instrument is attached

[ahiuchingau]

Overview

We do not want to show the banner prompting user to setup their module unless an instrument is attached. This matches the ODD behavior.
Closes EXEC-1624

[CaseyBatten]

Wowie zowie. Makes sense to me, don't think we use pipette calibration for any of the module flows.

[vegano1]

We do need pipettes for module calibration though, I think this needs to be enforced for specific modules that require calibration like the tempdeck, thermo cycler, etc.

[CaseyBatten]

We do need pipettes for module calibration though, I think this needs to be enforced for specific modules that require calibration like the tempdeck, thermo cycler, etc.

Ah so we'll need to extend the null handling to cover the SECTIONS.PLACE_ADAPTER case as well. Looking through the remaining module wizard flows I don't think there are others that will require this, so the place adapter step followed by the probe attach and detach steps should give coverage.

[CaseyBatten]

Extended comment below regarding remaining coverage needed

[CaseyBatten]

Based on further feedback we'll need to add coverage on the SECTIONS.PLACE_ADAPTER case on line 218 here because the pipette is also used when probing calibration blocks.

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.

Project coverage is 24.76%. Comparing base (0aeacc6) to head (3719265).
Report is 5 commits behind head on edge.

Files with missing lines	Patch %	Lines
app/src/molecules/UpdateBanner/index.tsx	0.00%	9 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             edge   #18668      +/-   ##
==========================================
- Coverage   24.76%   24.76%   -0.01%     
==========================================
  Files        3284     3284              
  Lines      285457   285462       +5     
  Branches    28663    28661       -2     
==========================================
  Hits        70704    70704              
- Misses     214732   214737       +5     
  Partials       21       21              

Flag	Coverage Δ
protocol-designer	19.04% <0.00%> (-0.01%)	⬇️
step-generation	5.25% <0.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
app/src/molecules/UpdateBanner/index.tsx	0.00% <0.00%> (ø)

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.