Try more comprehensive tomo metadata test

Author: stephen-riggs

tests/server/test_ispyb.py

@@ -78,6 +78,7 @@ def test_update_data_collection_group(mock_transport, ispyb_db_session: Session)
         table=DataCollectionGroup,
         insert_kwargs={
             "dataCollectionGroupId": 1,
+            "sessionId": 0,
             "experimentTypeId": 1,
         },
     )

tests/workflows/tomo/test_tomo_metadata.py

@@ -135,12 +135,16 @@ def test_register_search_map_update_with_all_parameters(
     assert sm_final_parameters.y_location is not None
 
     # Check this would have updated ispyb
-    mock_transport.do_update_search_map.assert_any_call(1, new_parameters)
-    new_parameters.x_location = sm_final_parameters.x_location
-    new_parameters.y_location = sm_final_parameters.y_location
-    new_parameters.height_on_atlas = int(40 * 7.8)
-    new_parameters.width_on_atlas = int(20 * 7.8)
-    mock_transport.do_update_search_map.assert_any_call(1, new_parameters)
+    update_args = mock_transport.do_update_search_map.call_args_list
+    assert len(update_args) == 2
+    assert update_args[0][0] == 1
+    assert update_args[1][0] == 1
+    assert update_args[0][1].x_location == new_parameters.x_location
+    assert update_args[0][1].y_location == new_parameters.y_location
+    assert update_args[1][1].x_location == sm_final_parameters.x_location
+    assert update_args[1][1].x_location == sm_final_parameters.y_location
+    assert update_args[1][1].height_on_atlas == int(40 * 7.8)
+    assert update_args[1][1].width_on_atlas == int(20 * 7.8)
 
 
 @mock.patch("murfey.workflows.tomo.tomo_metadata._transport_object")