Fix checksum message check, improve checksum message, update test case

config/ad_config.py

@@ -64,7 +64,7 @@
     AD_LOGDIR = os.path.join(RUNFOLDERS, "automate_demultiplexing_logfiles")
     MAIL_SETTINGS = MAIL_SETTINGS | {  # Add test mail recipients
         "pipeline_started_subj": f"{SCRIPT_MODE}. ALERT: Started pipeline for %s",
-        "binfx_recipient": MAIL_SETTINGS["binfx_email"],
+        "binfx_recipient": "[email protected]",
         # Oncology email address for email alerts
         "oncology_ops_email": "[email protected]",
         "wes_samplename_emaillist": ["[email protected]"],
@@ -361,7 +361,7 @@ class DemultiplexConfig(PanelConfig):
         "lane_metrics_suffix": LANE_METRICS_SUFFIX,
         "cd_success": "picard.illumina.CollectIlluminaLaneMetrics done",
         "demultiplex_success": DEMULTIPLEX_SUCCESS,
-        "checksums_already_assessed": "Checksums already assessed by AS: %s",  # Written to file by AS
+        "checksums_assessed": "Checksums assessed by AS: %s",  # Written to file by AS
         "checksums_match": "Checksums match",  # Success message written to md5checksum file by integrity check scripts
         "checksums_do_not_match": "Checksums do not match",  # Failure message written to md5sum file by integrity check scripts
         "samplesheet_success": "Samplesheet check successful with no errors identified: %s",

demultiplex/demultiplex.py

@@ -519,7 +519,7 @@ def prior_ic(self, checksums: list) -> Optional[bool]:
             :return (Optional[bool]):   Returns true if the checksum file has previously
                                         been checked for the success message by the script
         """
-        if DemultiplexConfig.STRINGS["checksums_already_assessed"] in checksums[-1]:
+        if DemultiplexConfig.STRINGS["checksums_assessed"].split(':')[0] in checksums[-1]:
             self.demux_rf_logger.info(
                 self.demux_rf_logger.log_msgs["checksumfile_checked"]
             )
@@ -577,7 +577,7 @@ def write_checksums_assessed(self) -> None:
         write_lines(
             self.rf_obj.checksumfile_path,
             "a",
-            DemultiplexConfig.STRINGS['checksums_already_assessed'] % datetime.datetime.now(),
+            DemultiplexConfig.STRINGS['checksums_assessed'] % datetime.datetime.now(),
         )
 
     def checksums_do_not_match_message(self, checksums: list) -> Optional[bool]:

test/test_demultiplex.py

@@ -493,7 +493,7 @@ def checksums_checked(self):
         """
         return [
             "Checksums match after 1 hours",
-            "Checksums already assessed by AS",
+            "Checksums assessed by AS",
         ]
 
     @pytest.fixture(scope="function")