Skip to content

Commit

Permalink
Adapt tests for the new Pipelines Download CLI.
Browse files Browse the repository at this point in the history
  • Loading branch information
MatthiasZepper committed Sep 27, 2024
1 parent c59282c commit 88decd2
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 9 deletions.
3 changes: 1 addition & 2 deletions nf_core/__main__.py
Original file line number Diff line number Diff line change
Expand Up @@ -2110,8 +2110,7 @@ def command_download(
outdir,
compress,
force,
tower,
platform,
platform or tower,
download_configuration,
tag,
container_system,
Expand Down
6 changes: 1 addition & 5 deletions nf_core/commands_pipelines.py
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,6 @@ def pipelines_download(
outdir,
compress,
force,
tower,
platform,
download_configuration,
tag,
Expand All @@ -185,16 +184,13 @@ def pipelines_download(
"""
from nf_core.pipelines.download import DownloadWorkflow

if tower:
log.warning("[red]The `-t` / `--tower` flag is deprecated. Please use `--platform` instead.[/]")

dl = DownloadWorkflow(
pipeline,
revision,
outdir,
compress,
force,
tower or platform, # True if either specified
platform,
download_configuration,
tag,
container_system,
Expand Down
4 changes: 2 additions & 2 deletions tests/test_cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,7 @@ def test_cli_download(self, mock_dl):
"compress": "tar.gz",
"force": None,
"platform": None,
"download-configuration": None,
"download-configuration": "yes",
"tag": "3.12=testing",
"container-system": "singularity",
"container-library": "quay.io",
Expand All @@ -188,7 +188,7 @@ def test_cli_download(self, mock_dl):
params["compress"],
"force" in params,
"platform" in params,
"download-configuration" in params,
params["download-configuration"],
(params["tag"],),
params["container-system"],
(params["container-library"],),
Expand Down

0 comments on commit 88decd2

Please sign in to comment.