Attempt to run test on this workflow with the Murfey and ISPyB databases

Author: tieneupin

tests/workflows/clem/test_register_preprocessing_results.py

@@ -5,13 +5,15 @@
 import pytest
 from pytest_mock import MockerFixture
 
+import murfey.util.db as MurfeyDB
+from murfey.server.run import _set_up_transport
 from murfey.workflows.clem.register_preprocessing_results import (
     _register_clem_image_series,
     _register_dcg_and_atlas,
     _register_grid_square,
     run,
 )
-from tests.conftest import ExampleVisit
+from tests.conftest import ExampleVisit, get_or_create_db_entry
 
 visit_name = f"{ExampleVisit.proposal_code}{ExampleVisit.proposal_number}-{ExampleVisit.visit_number}"
 processed_dir_name = "processed"
@@ -20,9 +22,14 @@
 
 
 @pytest.fixture
-def preprocessing_messages(tmp_path: Path):
+def rsync_basepath(tmp_path: Path):
+    return tmp_path / "data"
+
+
+@pytest.fixture
+def preprocessing_messages(rsync_basepath: Path):
     # Make directory to where data for current grid is stored
-    visit_dir = tmp_path / "data" / "2020" / visit_name
+    visit_dir = rsync_basepath / "2020" / visit_name
     processed_dir = visit_dir / processed_dir_name
     grid_dir = processed_dir / grid_name
     grid_dir.mkdir(parents=True, exist_ok=True)
@@ -147,4 +154,66 @@ def test_run(
     assert mock_align_and_merge_call.call_count == len(preprocessing_messages) * len(
         colors
     )
-    assert run
+
+
+def test_run_with_db(
+    mocker: MockerFixture,
+    preprocessing_messages: list[dict[str, Any]],
+    rsync_basepath: Path,
+    murfey_db_session,
+    seed_murfey_db,
+    ispyb_db_session_factory,
+    ispyb_db_session,
+    seed_ispyb_db,
+):
+    # Mock out module-level functions
+    mock_get_security_config = mocker.patch("murfey.server.ispyb.get_security_config")
+    mock_get_security_config.return_value = MagicMock()
+    mock_url = mocker.patch("murfey.server.ispyb.url")
+    mock_url.return_value = MagicMock()
+    mock_create_engine = mocker.patch("murfey.server.ispyb.create_engine")
+    mock_create_engine.return_value = MagicMock()
+
+    # Mock out the machine config used in the helper sanitisation function
+    mock_get_machine_config = mocker.patch("murfey.workflows.clem.get_machine_config")
+    mock_machine_config = MagicMock()
+    mock_machine_config.rsync_basepath = rsync_basepath
+    mock_get_machine_config.return_value = {
+        ExampleVisit.instrument_name: mock_machine_config,
+    }
+    # Mock the ISPyB connection
+    mock_ispyb_sessionmaker = mocker.patch("murfey.server.ispyb.sessionmaker")
+    mock_ispyb_sessionmaker.return_value = ispyb_db_session_factory
+    mock_ispyb_sessionmaker.return_value.return_value = ispyb_db_session
+
+    # Populate the Murfey database
+    murfey_session_entry: MurfeyDB.Session = get_or_create_db_entry(
+        murfey_db_session,
+        MurfeyDB.Session,
+        lookup_kwargs={"id": ExampleVisit.murfey_session_id},
+    )
+    murfey_session_entry.visit = visit_name
+    murfey_session_entry.name = visit_name
+    murfey_session_entry.instrument_name = ExampleVisit.instrument_name
+    murfey_db_session.add(murfey_session_entry)
+    murfey_db_session.commit()
+
+    # Mock the align and merge workflow call
+    mock_align_and_merge_call = mocker.patch(
+        "murfey.workflows.clem.register_preprocessing_results.submit_cluster_request"
+    )
+
+    # Set up the transport manager
+    _set_up_transport("PikaTransport")
+
+    # Run the function
+    for message in preprocessing_messages:
+        result = run(
+            message=message,
+            murfey_db=murfey_db_session,
+        )
+        assert result == {"success": True}
+    assert mock_align_and_merge_call.call_count == len(preprocessing_messages) * len(
+        colors
+    )
+    murfey_db_session.close()