From ada36c34d5e6c9497703ff3c7e1f9fcb62e407e1 Mon Sep 17 00:00:00 2001
From: Thomas Kluyver <thomas@kluyver.me.uk>
Date: Thu, 21 Dec 2023 09:39:23 +0000
Subject: [PATCH] Remove unused test_key parameter
---
tests/test_reference_runs/test_pre_deployment.py | 15 ++++-----------
1 file changed, 4 insertions(+), 11 deletions(-)
diff --git a/tests/test_reference_runs/test_pre_deployment.py b/tests/test_reference_runs/test_pre_deployment.py
index 1a0b92fec..ed4bb0fa4 100644
--- a/tests/test_reference_runs/test_pre_deployment.py
+++ b/tests/test_reference_runs/test_pre_deployment.py
@@ -150,7 +150,6 @@ def parse_config(
def validate_hdf5_files(
- test_key: str,
out_folder: pathlib.Path,
reference_folder: pathlib.Path,
cal_type: str,
@@ -318,12 +317,9 @@ def test_xfel_calibrate(test_key: str, val_dict: dict, release_test_config: Tupl
cmd = parse_config(cmd, cal_conf, out_folder)
if only_validate:
- validate_hdf5_files(
- test_key,
- out_folder,
- reference_folder,
- cal_type,
- )
+ assert validate_hdf5_files(
+ out_folder, reference_folder, cal_type
+ ), "HDF5 files changed - see details above"
return
if not use_slurm: # e.g. for Gitlab CI.
@@ -361,8 +357,5 @@ def test_xfel_calibrate(test_key: str, val_dict: dict, release_test_config: Tupl
# Stop tests at this point, if desired.
if not skip_numerical_validation:
assert validate_hdf5_files(
- test_key,
- out_folder,
- reference_folder,
- cal_type,
+ out_folder, reference_folder, cal_type
), "HDF5 files changed - see details above"
--
GitLab