fix tests for #6f02bb

Author: rickwierenga

pylabrobot/liquid_handling/backends/serializing_backend_tests.py

@@ -208,11 +208,11 @@ async def test_dispense96(self):
         "dispense": {
           "well_names": [well.name for well in self.plate.get_all_items()],
           "offset": serialize(Coordinate.zero()),
-          "volume": 10,
+          "volume": 10.0,
           "flow_rate": None,
           "liquid_height": None,
           "blow_out_air_volume": None,
-          "liquids": [[[None, 10]]] * 96,  # tuple, list of liquids per well, list of wells
+          "liquids": [[[None, 10.0]]] * 96,  # tuple, list of liquids per well, list of wells
           "tips": [serialize(tip) for tip in tips],
         }
       },

pylabrobot/liquid_handling/liquid_handler.py

@@ -1542,7 +1542,7 @@ async def drop_tips96(
       if not self.head96[i].has_tip:
         continue
       tip = self.head96[i].get_tip()
-      if tip.tracker.get_used_volume() > 0 and not allow_nonzero_volume:
+      if tip.tracker.get_used_volume() > 0 and not allow_nonzero_volume and does_volume_tracking():
         error = f"Cannot drop tip with volume {tip.tracker.get_used_volume()} on channel {i}"
         raise RuntimeError(error)
       if isinstance(resource, TipRack):
@@ -1942,7 +1942,6 @@ async def dispense96(
         liquids: List[Tuple[Optional[Liquid], float]]
         if tip.tracker.is_disabled or not does_volume_tracking():
           liquids = [(None, volume)]
-          all_liquids.append(liquids)
         else:
           liquids = tip.tracker.remove_liquid(volume=volume)
         reversed_liquids = list(reversed(liquids))

pylabrobot/liquid_handling/liquid_handler_tests.py

@@ -1,6 +1,7 @@
 import itertools
 import tempfile
 import unittest
+import unittest.mock
 from typing import Any, Dict, List, Optional, Union, cast
 
 import pytest
@@ -647,15 +648,23 @@ async def test_return_tips96(self):
       await self.lh.return_tips()
 
   async def test_aspirate_dispense96(self):
-    self.plate.get_item("A1").tracker.set_liquids([(None, 10)])
     await self.lh.pick_up_tips96(self.tip_rack)
     await self.lh.aspirate96(self.plate, volume=10)
-    for i in range(96):
-      self.assertTrue(self.lh.head96[i].has_tip)
-      self.assertEqual(self.lh.head96[i].get_tip().tracker.get_used_volume(), 10)
+    self.lh.backend.dispense96 = unittest.mock.AsyncMock()
     await self.lh.dispense96(self.plate, volume=10)
-    for i in range(96):
-      self.assertEqual(self.lh.head96[i].get_tip().tracker.get_used_volume(), 0)
+    self.lh.backend.dispense96.assert_called_with(
+      dispense=MultiHeadDispensePlate(
+        wells=self.plate.get_all_items(),
+        offset=Coordinate.zero(),
+        tips=[self.lh.head96[i].get_tip() for i in range(96)],
+        volume=10,
+        flow_rate=None,
+        liquid_height=None,
+        blow_out_air_volume=None,
+        liquids=[[(None, 10)] for _ in range(96)],
+        mix=None,
+      )
+    )
 
   async def test_transfer(self):
     t = self.tip_rack.get_item("A1").get_tip()