Skip to content

Commit

Permalink
fix(api): typo in should_dodge_thermocycler (#6458)
Browse files Browse the repository at this point in the history
* add test that fails the thermocycler dodge
* fix typo
  • Loading branch information
amitlissack authored Sep 1, 2020
1 parent f1cbe81 commit 8e446e1
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 1 addition & 1 deletion api/src/opentrons/protocol_api/geometry.py
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ def split_loc_labware(
('4', '12'),
('12', '4'),
('4', '9'),
('9, 4'),
('9', '4'),
('4', '8'),
('8', '4'),
('1', '8'),
Expand Down
2 changes: 2 additions & 0 deletions api/tests/opentrons/protocol_api/test_geometry.py
Original file line number Diff line number Diff line change
Expand Up @@ -406,6 +406,8 @@ def test_should_dodge():
# with a tc loaded, some positions should require dodging
assert should_dodge_thermocycler(
deck, deck.position_for(12), deck.position_for(1))
assert should_dodge_thermocycler(
deck, deck.position_for(9), deck.position_for(4))
# but ones that weren't explicitly marked shouldn't
assert not should_dodge_thermocycler(
deck, deck.position_for(1), deck.position_for(2))
Expand Down

0 comments on commit 8e446e1

Please sign in to comment.