feat(api): re-add api datasets commands using new database backend (#…

…2296)

renku/api/models/dataset.py

@@ -42,7 +42,8 @@
 from operator import attrgetter
 
 from renku.api.models.project import ensure_project_context
-from renku.core.management.dataset.datasets_provenance import DatasetsProvenance
+from renku.core.management.command_builder.database_dispatcher import DatabaseDispatcher
+from renku.core.metadata.gateway.dataset_gateway import DatasetGateway
 from renku.core.models import dataset as core_dataset
 
 
@@ -85,8 +86,12 @@ def list(project):
         client = project.client
         if not client or not client.has_graph_files():
             return []
-        datasets_provenance = DatasetsProvenance()
-        return [Dataset._from_dataset(d) for d in datasets_provenance.datasets]
+        database_dispatcher = DatabaseDispatcher()
+        database_dispatcher.push_database_to_stack(client.database_path)
+        dataset_gateway = DatasetGateway()
+        dataset_gateway.database_dispatcher = database_dispatcher
+
+        return [Dataset._from_dataset(d) for d in dataset_gateway.get_all_datasets()]
 
     def __getattribute__(self, name):
         dataset = object.__getattribute__(self, "_dataset")

tests/api/test_dataset.py

@@ -24,7 +24,6 @@
 from renku.api import Dataset, Project
 
 
-@pytest.mark.skip(reason="not implemented with new metadata yet, reenable later")
 def test_list_datasets(client_with_datasets):
     """Test listing datasets within a project context."""
     with Project():
@@ -33,7 +32,6 @@ def test_list_datasets(client_with_datasets):
         assert {"dataset-1", "dataset-2"} == {d.name for d in datasets}
 
 
-@pytest.mark.skip(reason="not implemented with new metadata yet, reenable later")
 def test_list_datasets_outside_a_context(client_with_datasets):
     """Test listing datasets outside a project context."""
     datasets = Dataset.list()
@@ -48,7 +46,6 @@ def test_list_datasets_outside_a_renku_project(directory_tree):
     assert [] == Dataset.list()
 
 
-@pytest.mark.skip(reason="not implemented with new metadata yet, reenable later")
 @pytest.mark.parametrize(
     "dataset, files_paths",
     [