Skip to content

Commit

Permalink
update half tower test
Browse files Browse the repository at this point in the history
  • Loading branch information
fellen31 committed Apr 4, 2024
1 parent 2f0969e commit 937e442
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions tests/test_download.py
Original file line number Diff line number Diff line change
Expand Up @@ -617,15 +617,15 @@ def test_download_workflow_with_success(self, tmp_dir, mock_download_image, mock
def test_download_workflow_for_platform(self, tmp_dir, _):
download_obj = DownloadWorkflow(
pipeline="nf-core/rnaseq",
revision=("3.7", "3.9"),
revision=("3.7", "3.9", "1817d14"),
compress_type="none",
platform=True,
container_system="singularity",
)

download_obj.include_configs = False # suppress prompt, because stderr.is_interactive doesn't.

assert isinstance(download_obj.revision, list) and len(download_obj.revision) == 2
assert isinstance(download_obj.revision, list) and len(download_obj.revision) == 3
assert isinstance(download_obj.wf_sha, dict) and len(download_obj.wf_sha) == 0
assert isinstance(download_obj.wf_download_url, dict) and len(download_obj.wf_download_url) == 0

Expand All @@ -635,13 +635,12 @@ def test_download_workflow_for_platform(self, tmp_dir, _):
download_obj.pipeline,
download_obj.wf_revisions,
download_obj.wf_branches,
_
download_obj.wf_commits
) = nf_core.utils.get_repo_releases_branches_commits(download_obj.pipeline, wfs)

download_obj.get_revision_hash()

# download_obj.wf_download_url is not set for Seqera Platform downloads, but the sha values are
assert isinstance(download_obj.wf_sha, dict) and len(download_obj.wf_sha) == 2
# download_obj.wf_download_url is not set for tower downloads, but the sha values are
assert isinstance(download_obj.wf_sha, dict) and len(download_obj.wf_sha) == 3
assert isinstance(download_obj.wf_download_url, dict) and len(download_obj.wf_download_url) == 0

# The outdir for multiple revisions is the pipeline name and date: e.g. nf-core-rnaseq_2023-04-27_18-54
Expand Down

0 comments on commit 937e442

Please sign in to comment.