Add more testing to dicom archive loader last commit #1247
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This adds the following tests to
test_run_dicom_archive_loader.py
:mri_upload.number_of_minc_inserted
is correctmri_upload.number_of_minc_created
is correctdicom_archive.session_id
has been setmri_upload.insertion_complete
is set to TrueIt also modifies the DICOMs on the S3 bucket and the entries in the
tarchive
tables to add the localizer to the T1W to check that the localizer is indeed skipped because it is part of the series to exclude in the configuration settingexcluded_series_description
.Replaces #1244 that was polluted by a force-push