feat: use project name as default path when cloning (#857)

Author: mohammad-alisafaee

renku/core/management/clone.py

@@ -23,6 +23,7 @@
 
 from renku.core import errors
 from renku.core.management.githooks import install
+from renku.core.models.git import GitURL
 
 
 def clone(
@@ -38,7 +39,7 @@ def clone(
     """Clone Renku project repo, install Git hooks and LFS."""
     from renku.core.management.client import LocalClient
 
-    path = path or '.'
+    path = path or GitURL.parse(url).name
     # Clone the project
     if skip_smudge:
         os.environ['GIT_LFS_SKIP_SMUDGE'] = '1'

tests/cli/test_integration_datasets.py

@@ -927,3 +927,20 @@ def test_renku_clone(runner, monkeypatch):
             result = runner.invoke(cli, ['run', 'touch', 'output'])
             assert 'is not configured' in result.output
             assert 1 == result.exit_code
+
+
+@pytest.mark.integration
+@pytest.mark.parametrize(
+    'path,expected_path', [('', 'datasets-test'), ('.', '.'),
+                           ('new-name', 'new-name')]
+)
+def test_renku_clone_uses_project_name(
+    runner, monkeypatch, path, expected_path
+):
+    """Test renku clone uses project name as target-path by default."""
+    REMOTE = 'https://dev.renku.ch/gitlab/virginiafriedrich/datasets-test.git'
+
+    with runner.isolated_filesystem() as project_path:
+        result = runner.invoke(cli, ['clone', REMOTE, path])
+        assert 0 == result.exit_code
+        assert (Path(project_path) / expected_path / 'Dockerfile').exists()