[Tests] Validate metadata for retrieved constants before validating the corrected files

Description

This MR addresses: https://git.xfel.eu/calibration/pycalibration/-/issues/89

In case the retrieved constants are different, the tests should realize this before validating the corrected files. This help in understanding the reason for the test failure instead of spending time investigating why the files were not validated and have different values.

How Has This Been Tested?

• Tested locally after changing parameters of one test to get the wrong Calibration constants.
• CI-InProgress

Relevant Documents (optional)

Types of changes

• Test (additional or refactored tests)

Checklist:

Reviewers

@kluyvert @schmidtp

tests/test_reference_runs/test_pre_deployment.py

@@ -375,17 +375,20 @@ def test_xfel_calibrate(
         if time_counter > time_to_wait:
             assert False, f"{test_key} failure, report doesn't exists."
     LOGGER.info("Report found.")
-    # For corrections validate same calibration constants were retrieved.
 
-    metadata_file = f"calibration_metadata_{cal_conf['karabo-id']}"
-    assert compare_metadata_ccvs(
-        out_folder / metadata_file,
-        reference_folder / metadata_file,
-        "retrieved-constants",
-    ), "Found difference in the metadata for the retrieved Constants."
+    if cal_type.lower() == "correct":
+        # For corrections validate calibration constants
+        metadata_file = f"calibration_metadata_{cal_conf['karabo-id']}.yml"
+        assert compare_metadata_ccvs(
+            out_folder / metadata_file,
+            reference_folder / metadata_file,
+            "retrieved-constants",
+        ), "Found difference in the metadata for the retrieved Constants."
+        LOGGER.info("Retrieved CCVs validated.")
 
     # Stop tests at this point, if desired.
     if not skip_numerical_validation:
         assert validate_hdf5_files(
             out_folder,  reference_folder, cal_type
         ), "HDF5 files changed - see details above"
+        LOGGER.info("H5 Files validated.")