Copy public extracts upon export of data

cli/src/pixl_cli/_io.py

@@ -0,0 +1,140 @@
+#  Copyright (c) University College London Hospitals NHS Foundation Trust
+#
+#  Licensed under the Apache License, Version 2.0 (the "License");
+#  you may not use this file except in compliance with the License.
+#  You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+#  Unless required by applicable law or agreed to in writing, software
+#  distributed under the License is distributed on an "AS IS" BASIS,
+#  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#  See the License for the specific language governing permissions and
+#  limitations under the License.
+
+"""Reading and writing files from PIXL CLI."""
+import json
+from datetime import datetime
+from pathlib import Path
+
+import pandas as pd
+from core.omop import OmopExtract
+from core.patient_queue.message import Message, deserialise
+
+from pixl_cli._logging import logger
+from pixl_cli._utils import string_is_non_empty
+
+# instance of omop extract, can be overriden during testing
+extract = OmopExtract()
+
+
+def messages_from_state_file(filepath: Path) -> list[Message]:
+    """
+    Return messages from a state file path
+
+    :param filepath: Path for state file to be read
+    :return: A list of Message objects containing all the messages from the state file
+    """
+    logger.info(f"Creating messages from {filepath}")
+    if not filepath.exists():
+        raise FileNotFoundError
+    if filepath.suffix != ".state":
+        msg = f"Invalid file suffix for {filepath}. Expected .state"
+        raise ValueError(msg)
+
+    return [
+        deserialise(line) for line in Path.open(filepath).readlines() if string_is_non_empty(line)
+    ]
+
+
+def copy_public_parquet_and_build_messages(dir_path: Path) -> list[Message]:
+    """
+    Reads patient information from parquet files within directory structure
+    and transforms that into messages. Copies the public parquet files to extracts.
+
+    :param dir_path: Path for parquet directory containing private and public
+    files
+    """
+    public_dir = dir_path / "public"
+    private_dir = dir_path / "private"
+    log_file = dir_path / "extract_summary.json"
+
+    cohort_data = _check_and_parse_parquet(log_file, private_dir, public_dir)
+
+    expected_col_names = [
+        "PrimaryMrn",
+        "AccessionNumber",
+        "person_id",
+        "procedure_date",
+        "procedure_occurrence_id",
+    ]
+    logger.debug(
+        f"Extracting messages from {dir_path}. Expecting columns to include "
+        f"{expected_col_names}"
+    )
+
+    for col in expected_col_names:
+        if col not in list(cohort_data.columns):
+            msg = (
+                f"parquet files are expected to have at least {expected_col_names} as "
+                f"column names"
+            )
+            raise ValueError(msg)
+
+    (
+        mrn_col_name,
+        acc_num_col_name,
+        _,
+        dt_col_name,
+        procedure_occurrence_id,
+    ) = expected_col_names
+
+    omop_es_timestamp, project_name_slug = _parse_log_and_copy_parquet(dir_path, log_file)
+
+    messages = []
+
+    for _, row in cohort_data.iterrows():
+        # Create new dict to initialise message
+        message = Message(
+            mrn=row[mrn_col_name],
+            accession_number=row[acc_num_col_name],
+            study_date=row[dt_col_name],
+            procedure_occurrence_id=row[procedure_occurrence_id],
+            project_name=project_name_slug,
+            omop_es_timestamp=omop_es_timestamp,
+        )
+        messages.append(message)
+
+    if len(messages) == 0:
+        msg = f"Failed to find any messages in {dir_path}"
+        raise ValueError(msg)
+
+    logger.info(f"Created {len(messages)} messages from {dir_path}")
+    return messages
+
+
+def _check_and_parse_parquet(log_file: Path, private_dir: Path, public_dir: Path) -> pd.DataFrame:
+    for d in [public_dir, private_dir]:
+        if not d.is_dir():
+            err_str = f"{d} must exist and be a directory"
+            raise NotADirectoryError(err_str)
+    if not log_file.is_file():
+        err_str = f"{log_file} must exist and be a file"
+        raise FileNotFoundError(err_str)
+    # MRN in people.PrimaryMrn:
+    people = pd.read_parquet(private_dir / "PERSON_LINKS.parquet")
+    # accession number in accessions.AccessionNumber
+    accessions = pd.read_parquet(private_dir / "PROCEDURE_OCCURRENCE_LINKS.parquet")
+    # study_date is in procedure.procedure_date
+    procedure = pd.read_parquet(public_dir / "PROCEDURE_OCCURRENCE.parquet")
+    # joining data together
+    people_procedures = people.merge(procedure, on="person_id")
+    return people_procedures.merge(accessions, on="procedure_occurrence_id")
+
+
+def _parse_log_and_copy_parquet(dir_path: Path, log_file: Path) -> tuple[datetime, str]:
+    logs = json.load(log_file.open())
+    project_name = logs["settings"]["cdm_source_name"]
+    omop_es_timestamp = datetime.fromisoformat(logs["datetime"])
+    project_name_slug = extract.copy_to_exports(dir_path, project_name, omop_es_timestamp)
+    return omop_es_timestamp, project_name_slug

cli/src/pixl_cli/main.py

@@ -13,23 +13,21 @@
 #  limitations under the License.
 """PIXL command line interface functionality"""
 
-import datetime
 import json
 import os
 from operator import attrgetter
 from pathlib import Path
 from typing import Any, Optional
 
 import click
-import pandas as pd
 import requests
 import yaml
-from core.patient_queue.message import Message, deserialise
 from core.patient_queue.producer import PixlProducer
 from core.patient_queue.subscriber import PixlBlockingConsumer
 
+from ._io import copy_public_parquet_and_build_messages, messages_from_state_file
 from ._logging import logger, set_log_level
-from ._utils import clear_file, remove_file_if_it_exists, string_is_non_empty
+from ._utils import clear_file, remove_file_if_it_exists
 
 
 def _load_config(filename: str = "pixl_config.yml") -> dict:
@@ -90,18 +88,19 @@ def populate(parquet_dir: Path, *, restart: bool, queues: str) -> None:
             └── extract_summary.json
     """
     logger.info(f"Populating queue(s) {queues} from {parquet_dir}")
+    messages = copy_public_parquet_and_build_messages(parquet_dir)
+
     for queue in queues.split(","):
-        with PixlProducer(queue_name=queue, **config["rabbitmq"]) as producer:
-            state_filepath = state_filepath_for_queue(queue)
-            if state_filepath.exists() and restart:
-                logger.info(f"Extracting messages from state: {state_filepath}")
-                inform_user_that_queue_will_be_populated_from(state_filepath)
-                messages = messages_from_state_file(state_filepath)
-            elif parquet_dir is not None:
-                messages = messages_from_parquet(parquet_dir)
+        state_filepath = state_filepath_for_queue(queue)
+        if state_filepath.exists() and restart:
+            logger.info(f"Extracting messages from state: {state_filepath}")
+            inform_user_that_queue_will_be_populated_from(state_filepath)
+            messages = messages_from_state_file(state_filepath)
 
-            remove_file_if_it_exists(state_filepath)  # will be stale
-            producer.publish(sorted(messages, key=attrgetter("study_datetime")))
+        remove_file_if_it_exists(state_filepath)  # will be stale
+
+        with PixlProducer(queue_name=queue, **config["rabbitmq"]) as producer:
+            producer.publish(sorted(messages, key=attrgetter("study_date")))
 
 
 @cli.command()
@@ -285,110 +284,6 @@ def state_filepath_for_queue(queue_name: str) -> Path:
     return Path(f"{queue_name.replace('/', '_')}.state")
 
 
-def messages_from_state_file(filepath: Path) -> list[Message]:
-    """
-    Return messages from a state file path
-
-    :param filepath: Path for state file to be read
-    :return: A list of Message objects containing all the messages from the state file
-    """
-    logger.info(f"Creating messages from {filepath}")
-    if not filepath.exists():
-        raise FileNotFoundError
-    if filepath.suffix != ".state":
-        msg = f"Invalid file suffix for {filepath}. Expected .state"
-        raise ValueError(msg)
-
-    return [
-        deserialise(line) for line in Path.open(filepath).readlines() if string_is_non_empty(line)
-    ]
-
-
-def messages_from_parquet(dir_path: Path) -> list[Message]:
-    """
-    Reads patient information from parquet files within directory structure
-    and transforms that into messages.
-    :param dir_path: Path for parquet directory containing private and public
-    files
-    """
-    public_dir = dir_path / "public"
-    private_dir = dir_path / "private"
-    log_file = dir_path / "extract_summary.json"
-
-    for d in [public_dir, private_dir]:
-        if not d.is_dir():
-            err_str = f"{d} must exist and be a directory"
-            raise NotADirectoryError(err_str)
-
-    if not log_file.is_file():
-        err_str = f"{log_file} must exist and be a file"
-        raise FileNotFoundError(err_str)
-
-    # MRN in people.PrimaryMrn:
-    people = pd.read_parquet(private_dir / "PERSON_LINKS.parquet")
-    # accession number in accessions.AccesionNumber
-    accessions = pd.read_parquet(private_dir / "PROCEDURE_OCCURRENCE_LINKS.parquet")
-    # study_date is in procedure.procdure_date
-    procedure = pd.read_parquet(public_dir / "PROCEDURE_OCCURRENCE.parquet")
-    # joining data together
-    people_procedures = people.merge(procedure, on="person_id")
-    cohort_data = people_procedures.merge(accessions, on="procedure_occurrence_id")
-
-    expected_col_names = [
-        "PrimaryMrn",
-        "AccessionNumber",
-        "person_id",
-        "procedure_date",
-        "procedure_occurrence_id",
-    ]
-    logger.debug(
-        f"Extracting messages from {dir_path}. Expecting columns to include "
-        f"{expected_col_names}"
-    )
-
-    for col in expected_col_names:
-        if col not in list(cohort_data.columns):
-            msg = (
-                f"parquet files are expected to have at least {expected_col_names} as "
-                f"column names"
-            )
-            raise ValueError(msg)
-
-    (
-        mrn_col_name,
-        acc_num_col_name,
-        _,
-        dt_col_name,
-        procedure_occurrence_id,
-    ) = expected_col_names
-
-    # Get project name and OMOP ES timestamp from log file
-    logs = json.load(log_file.open())
-    project_name = logs["settings"]["cdm_source_name"]
-    omop_es_timestamp = datetime.datetime.fromisoformat(logs["datetime"])
-
-    messages = []
-
-    for _, row in cohort_data.iterrows():
-        # Create new dict to initialise message
-        message = Message(
-            mrn=row[mrn_col_name],
-            accession_number=row[acc_num_col_name],
-            study_datetime=row[dt_col_name],
-            procedure_occurrence_id=row[procedure_occurrence_id],
-            project_name=project_name,
-            omop_es_timestamp=omop_es_timestamp,
-        )
-        messages.append(message)
-
-    if len(messages) == 0:
-        msg = f"Failed to find any messages in {dir_path}"
-        raise ValueError(msg)
-
-    logger.info(f"Created {len(messages)} messages from {dir_path}")
-    return messages
-
-
 def queue_is_up() -> Any:
     """Checks if the queue is up"""
     with PixlProducer(queue_name="") as producer:

cli/tests/conftest.py

@@ -19,11 +19,17 @@
 from core.omop import OmopExtract
 
 
-@pytest.fixture()
-def omop_files(tmp_path_factory: pytest.TempPathFactory) -> OmopExtract:
-    """Create an OmopExtract instance using a temporary directory"""
+@pytest.fixture(autouse=True)
+def omop_files(tmp_path_factory: pytest.TempPathFactory, monkeypatch) -> OmopExtract:
+    """
+    Replace production extract instance with one writing to a tmpdir.
+
+    :returns OmopExtract: For direct use when the fixture is explicity called.
+    """
     export_dir = tmp_path_factory.mktemp("repo_base")
-    return OmopExtract(export_dir)
+    tmpdir_extract = OmopExtract(export_dir)
+    monkeypatch.setattr("pixl_cli._io.extract", tmpdir_extract)
+    return tmpdir_extract
 
 
 @pytest.fixture()

cli/tests/test_messages_from_parquet.py

@@ -17,7 +17,7 @@
 from pathlib import Path
 
 from core.patient_queue.message import Message
-from pixl_cli.main import messages_from_parquet
+from pixl_cli._io import copy_public_parquet_and_build_messages
 
 
 def test_messages_from_parquet(resources: Path) -> None:
@@ -26,40 +26,40 @@ def test_messages_from_parquet(resources: Path) -> None:
     The test data doesn't have any "difficult" cases in it, eg. people without procedures.
     """
     omop_parquet_dir = resources / "omop"
-    messages = messages_from_parquet(omop_parquet_dir)
+    messages = copy_public_parquet_and_build_messages(omop_parquet_dir)
     assert all(isinstance(msg, Message) for msg in messages)
 
     expected_messages = [
         Message(
             mrn="12345678",
             accession_number="12345678",
-            study_datetime=datetime.date.fromisoformat("2021-07-01"),
+            study_date=datetime.date.fromisoformat("2021-07-01"),
             procedure_occurrence_id=1,
-            project_name="Test Extract - UCLH OMOP CDM",
+            project_name="test-extract-uclh-omop-cdm",
             omop_es_timestamp=datetime.datetime.fromisoformat("2023-12-07T14:08:58"),
         ),
         Message(
             mrn="12345678",
             accession_number="ABC1234567",
-            study_datetime=datetime.date.fromisoformat("2021-07-01"),
+            study_date=datetime.date.fromisoformat("2021-07-01"),
             procedure_occurrence_id=2,
-            project_name="Test Extract - UCLH OMOP CDM",
+            project_name="test-extract-uclh-omop-cdm",
             omop_es_timestamp=datetime.datetime.fromisoformat("2023-12-07T14:08:58"),
         ),
         Message(
             mrn="987654321",
             accession_number="ABC1234560",
-            study_datetime=datetime.date.fromisoformat("2020-05-01"),
+            study_date=datetime.date.fromisoformat("2020-05-01"),
             procedure_occurrence_id=3,
-            project_name="Test Extract - UCLH OMOP CDM",
+            project_name="test-extract-uclh-omop-cdm",
             omop_es_timestamp=datetime.datetime.fromisoformat("2023-12-07T14:08:58"),
         ),
         Message(
             mrn="5020765",
             accession_number="MIG0234560",
-            study_datetime=datetime.date.fromisoformat("2015-05-01"),
+            study_date=datetime.date.fromisoformat("2015-05-01"),
             procedure_occurrence_id=4,
-            project_name="Test Extract - UCLH OMOP CDM",
+            project_name="test-extract-uclh-omop-cdm",
             omop_es_timestamp=datetime.datetime.fromisoformat("2023-12-07T14:08:58"),
         ),
     ]