Skip to content
Snippets Groups Projects
Commit ada36c34 authored by Thomas Kluyver's avatar Thomas Kluyver
Browse files

Remove unused test_key parameter

parent d61bc8c9
No related branches found
No related tags found
1 merge request!933[Tests] clearer comparison of HDF5 files
...@@ -150,7 +150,6 @@ def parse_config( ...@@ -150,7 +150,6 @@ def parse_config(
def validate_hdf5_files( def validate_hdf5_files(
test_key: str,
out_folder: pathlib.Path, out_folder: pathlib.Path,
reference_folder: pathlib.Path, reference_folder: pathlib.Path,
cal_type: str, cal_type: str,
...@@ -318,12 +317,9 @@ def test_xfel_calibrate(test_key: str, val_dict: dict, release_test_config: Tupl ...@@ -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) cmd = parse_config(cmd, cal_conf, out_folder)
if only_validate: if only_validate:
validate_hdf5_files( assert validate_hdf5_files(
test_key, out_folder, reference_folder, cal_type
out_folder, ), "HDF5 files changed - see details above"
reference_folder,
cal_type,
)
return return
if not use_slurm: # e.g. for Gitlab CI. 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 ...@@ -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. # Stop tests at this point, if desired.
if not skip_numerical_validation: if not skip_numerical_validation:
assert validate_hdf5_files( assert validate_hdf5_files(
test_key, out_folder, reference_folder, cal_type
out_folder,
reference_folder,
cal_type,
), "HDF5 files changed - see details above" ), "HDF5 files changed - see details above"
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment