fix(cli): fix project export download for CLI

nejch · JohnVillalovos · commit 5d1486785793 · 2022-06-01T08:00:51.000-07:00
Since ac1c619 we delete parent arg keys from the args dict so this has been trying to access the wrong attribute.
diff --git a/gitlab/v4/cli.py b/gitlab/v4/cli.py
@@ -116,7 +116,7 @@ def do_custom(self) -> Any:
 
     def do_project_export_download(self) -> None:
         try:
-            project = self.gl.projects.get(int(self.args["project_id"]), lazy=True)
+            project = self.gl.projects.get(self.parent_args["project_id"], lazy=True)
             export_status = project.exports.get()
             if TYPE_CHECKING:
                 assert export_status is not None
diff --git a/tests/functional/cli/test_cli_projects.py b/tests/functional/cli/test_cli_projects.py
@@ -1,3 +1,6 @@
+import subprocess
+import time
+
 import pytest
 import responses
 
@@ -25,3 +28,34 @@ def test_project_registry_delete_in_bulk(
     ]
     ret = ret = script_runner.run(*cmd)
     assert ret.success
+
+
+@pytest.fixture
+def project_export(project):
+    export = project.exports.create()
+    export.refresh()
+
+    count = 0
+    while export.export_status != "finished":
+        time.sleep(0.5)
+        export.refresh()
+        count += 1
+        if count == 30:
+            raise Exception("Project export taking too much time")
+
+    return export
+
+
+def test_project_export_download(gitlab_config, project_export):
+    cmd = [
+        "gitlab",
+        "--config-file",
+        gitlab_config,
+        "project-export",
+        "download",
+        "--project-id",
+        str(project_export.id),
+    ]
+
+    export = subprocess.run(cmd, capture_output=True, check=True)
+    assert export.returncode == 0
