feat: avoid creation of nested datasets (#796)

* feat: avoid creation of nested datasets

Author: mohammad-alisafaee

renku/core/management/datasets.py

@@ -115,6 +115,12 @@ def with_dataset(self, name=None, identifier=None):
         clean_up_required = False
 
         if dataset is None:
+            # Avoid nested datasets: name mustn't have '/' in it
+            if len(Path(name).parts) > 1:
+                raise errors.ParameterError(
+                    'Dataset name {} is not valid.'.format(name)
+                )
+
             clean_up_required = True
             dataset_ref = None
             identifier = str(uuid.uuid4())

tests/cli/test_datasets.py

@@ -182,6 +182,14 @@ def test_dataset_create_exception_refs(runner, project, client):
     assert 'a' in result.output
 
 
+@pytest.mark.parametrize('name', ['dataset/new', '/dataset', 'dataset/'])
+def test_dataset_name_is_valid(client, runner, project, name):
+    """Test dataset name has no '/' character to avoid nested datasets."""
+    result = runner.invoke(cli, ['dataset', 'create', name])
+    assert 2 == result.exit_code
+    assert 'is not valid' in result.output
+
+
 @pytest.mark.parametrize('output_format', DATASETS_FORMATS.keys())
 def test_datasets_list_empty(output_format, runner, project):
     """Test listing without datasets."""