Skip to content

test(app): fix distribute test for quick transfer step commands#18446

Merged
ncdiehl11 merged 1 commit intoedgefrom
app_fix-broken-test
May 28, 2025
Merged

test(app): fix distribute test for quick transfer step commands#18446
ncdiehl11 merged 1 commit intoedgefrom
app_fix-broken-test

Conversation

@ncdiehl11
Copy link
Copy Markdown
Collaborator

@ncdiehl11 ncdiehl11 commented May 28, 2025

Fixes distribute test for quick transfer step commands. We needed to use the constant for 'source_well' as SOURCE_WELL_BLOWOUT_DESTINATION rather than string literal 'source'

Also adds correct commands for moving back to source well and blowing out there in Python commands

@ncdiehl11 ncdiehl11 self-assigned this May 28, 2025
@ncdiehl11 ncdiehl11 requested a review from jerader May 28, 2025 21:02
@ncdiehl11 ncdiehl11 marked this pull request as ready for review May 28, 2025 21:02
@ncdiehl11 ncdiehl11 requested a review from a team as a code owner May 28, 2025 21:02
Copy link
Copy Markdown
Contributor

@koji koji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

passed the all cases

Copy link
Copy Markdown
Collaborator

@jerader jerader left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ty!

@ncdiehl11 ncdiehl11 merged commit 568cae0 into edge May 28, 2025
36 checks passed
pipette.move_to(mock_labware_2["B1"].bottom())
pipette.move_to(mock_labware_1["A1"].top())
pipette.flow_rate.blow_out = 50
pipette.blow_out()
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just checking: a blowout is what we expect here?

@ncdiehl11 ncdiehl11 deleted the app_fix-broken-test branch May 29, 2025 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants