feat: resolve paths in datasamples spec

Signed-off-by: ThibaultFy <50656860+ThibaultFy@users.noreply.github.com>

substra/sdk/schemas.py

@@ -158,19 +158,17 @@ def exclusive_paths(cls, values):  # noqa: N805
     @pydantic.model_validator(mode="before")
     def resolve_paths(cls, values):  # noqa: N805
         """Resolve given path is relative."""
-
         if "paths" in values:
             paths = []
             for path in values["paths"]:
-                if not path.is_absolute():
-                    paths.append(path.resolve())
-                else:
-                    paths.append(path)
+                path = pathlib.Path(path)
+                paths.append(path.resolve()) if not path.is_absolute() else paths.append(path)
+
             values["paths"] = paths
 
         elif "path" in values:
-            if not values["path"].is_absolute():
-                values["path"] = values["path"].resolve()
+            path = pathlib.Path(values["path"])
+            values["path"] = path.resolve() if not path.is_absolute() else path
 
         return values
 

tests/sdk/test_schemas.py

@@ -0,0 +1,20 @@
+import pathlib
+import uuid
+
+import pytest
+
+from substra.sdk.schemas import DataSampleSpec
+
+
+@pytest.mark.parametrize("path", [pathlib.Path() / "data", "./data", pathlib.Path().cwd() / "data"])
+def test_datasample_spec_resolve_path(path):
+    datasample_spec = DataSampleSpec(path=path, data_manager_keys=[str(uuid.uuid4())])
+
+    assert datasample_spec.path == pathlib.Path().cwd() / "data"
+
+
+def test_datasample_spec_resolve_paths():
+    paths = [pathlib.Path() / "data", "./data", pathlib.Path().cwd() / "data"]
+    datasample_spec = DataSampleSpec(paths=paths, data_manager_keys=[str(uuid.uuid4())])
+
+    assert all([path == pathlib.Path().cwd() / "data" for path in datasample_spec.paths])