fix(protocol-designer): get all labware defs instead of latest when z…

[jerader]

…ooming into slot

closes AUTH-1730

Overview

NOTE: cherry-picked from the edge commit

As documented in the ticket, there was a white screen if you upload a protocol to PD that has an older labware definition version and then try to zoom into that slot. The error specifically occurred with the highlight overlay that shows the labware display name.

This PR fixes it by refactoring the component to return all labware definitions instead of just latest. This does not affect selecting new labware though, that utility just returns the latest versions - this prevents users from selecting older versions with new steps/new deck setups

Test Plan and Hands on Testing

Upload the attached protocol and zoom into the thermocycler slot or the 1 well reservoir slot. See that it does not white screen.

Note that it looks like the labware selected isn't selected in the labware toolbox but that is because it is showing only latest defs. i think that is fine. the user will be regenerating their labware anyway if they make any changes

doItAllV8 (29).json

Changelog

• get all defs instead of latest defs
• fix test

Risk assessment

low, doesn't touch much code

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 21.49%. Comparing base (3f8243d) to head (58f83a3).
Report is 3 commits behind head on chore_release-pd-8.4.4.

Additional details and impacted files

@@                    Coverage Diff                     @@
##           chore_release-pd-8.4.4   #18223      +/-   ##
==========================================================
+ Coverage                   21.46%   21.49%   +0.02%     
==========================================================
  Files                        2748     2748              
  Lines                      212336   212336              
  Branches                     7224     7239      +15     
==========================================================
+ Hits                        45586    45635      +49     
+ Misses                     166743   166694      -49     
  Partials                        7        7              

Flag	Coverage Δ
protocol-designer	18.70% <100.00%> (+0.02%)	⬆️

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

Files with missing lines	Coverage Δ
.../pages/Designer/DeckSetup/SelectedHoveredItems.tsx	95.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

🚀 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.

[ncdiehl11]

🙌