Skip to content

Commit

Permalink
Fix tests that had become out of date. (#125)
Browse files Browse the repository at this point in the history
  • Loading branch information
JoeZiminski authored Oct 3, 2023
1 parent 1a8472d commit 534c7f0
Showing 1 changed file with 11 additions and 11 deletions.
22 changes: 11 additions & 11 deletions tests/test_integration/test_full_pipeline.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ def test_preprocessing_options_1(self, test_info):

for ses_name, run_name in preprocess_data.preprocessing_sessions_and_runs():
preprocess_data = preprocess.preprocess(
preprocess_data, ses_name, run_name, pp_steps, verbose=True
preprocess_data, ses_name, run_name, pp_steps
)
preprocess_data.save_preprocessed_data(ses_name, run_name, overwrite=True)

Expand All @@ -50,7 +50,7 @@ def test_preprocessing_options_2(self, test_info):

for ses_name, run_name in preprocess_data.preprocessing_sessions_and_runs():
preprocess_data = preprocess.preprocess(
preprocess_data, ses_name, run_name, pp_steps, verbose=True
preprocess_data, ses_name, run_name, pp_steps
)
preprocess_data.save_preprocessed_data(ses_name, run_name, overwrite=True)

Expand Down Expand Up @@ -164,7 +164,7 @@ def test_ses_concat_no_run_concat(self, test_info):
def test_existing_output_settings(self, test_info):
"""
In spikewrap existing preprocessed and sorting output data is
handled with options `fail_if_exists`, `load_if_exists` or
handled with options `fail_if_exists`, `skip_if_exists` or
`overwrite`. Check that the expected behaviour occurs when each of
these options is set.
Expand Down Expand Up @@ -202,12 +202,12 @@ def test_existing_output_settings(self, test_info):

file_paths = self.write_an_empty_file_in_outputs(test_info, ses_name, run_name)

# Test outputs are not overwritten if `load_if_exists`.
# Test outputs are not overwritten if `skip_if_exists`.
# Postprocessing is always deleted
self.run_full_pipeline(
*test_info,
existing_preprocessed_data="load_if_exists",
existing_sorting_output="load_if_exists",
existing_preprocessed_data="skip_if_exists",
existing_sorting_output="skip_if_exists",
overwrite_postprocessing=True,
sorter=DEFAULT_SORTER,
)
Expand All @@ -220,7 +220,7 @@ def test_existing_output_settings(self, test_info):
self.run_full_pipeline(
*test_info,
existing_preprocessed_data="fail_if_exists",
existing_sorting_output="load_if_exists",
existing_sorting_output="skip_if_exists",
overwrite_postprocessing=True,
sorter=DEFAULT_SORTER,
)
Expand All @@ -233,7 +233,7 @@ def test_existing_output_settings(self, test_info):
with pytest.raises(BaseException) as e:
self.run_full_pipeline(
*test_info,
existing_preprocessed_data="load_if_exists",
existing_preprocessed_data="skip_if_exists",
existing_sorting_output="fail_if_exists",
overwrite_postprocessing=True,
sorter=DEFAULT_SORTER,
Expand All @@ -245,8 +245,8 @@ def test_existing_output_settings(self, test_info):
with pytest.raises(BaseException) as e:
self.run_full_pipeline(
*test_info,
existing_preprocessed_data="load_if_exists",
existing_sorting_output="load_if_exists",
existing_preprocessed_data="skip_if_exists",
existing_sorting_output="skip_if_exists",
overwrite_postprocessing=False,
sorter=DEFAULT_SORTER,
)
Expand Down Expand Up @@ -617,7 +617,7 @@ def get_output_paths(
)

paths = {
"preprocessing": run_path / "preprocessed",
"preprocessing": run_path / "preprocessing",
"sorting_path": run_path / sorter / "sorting",
"postprocessing": run_path / sorter / "postprocessing",
}
Expand Down

0 comments on commit 534c7f0

Please sign in to comment.