feat(skore-local-project): Factorize the project "deletion test" in a decorator

skore-local-project/src/skore_local_project/project.py

@@ -4,6 +4,7 @@
 
 import io
 import os
+from functools import wraps
 from operator import itemgetter
 from pathlib import Path
 from types import SimpleNamespace
@@ -35,6 +36,21 @@ class Metadata(TypedDict):  # noqa: D101
         predict_time: float
 
 
+def ensure_project_is_not_deleted(method):
+    """Ensure project is not deleted, before executing any other operation."""
+
+    @wraps(method)
+    def wrapper(self, *args, **kwargs):
+        if self.name not in self._Project__projects_storage:
+            raise RuntimeError(
+                f"Skore could not proceed because {repr(self)} does not exist anymore."
+            )
+
+        return method(self, *args, **kwargs)
+
+    return wrapper
+
+
 class Project:
     r"""
     API to manage a collection of key-report pairs persisted in a local storage.
@@ -168,6 +184,7 @@ def pickle(report: EstimatorReport | CrossValidationReport) -> tuple[str, bytes]
 
         return pickle_hash, pickle_bytes
 
+    @ensure_project_is_not_deleted
     def put(self, key: str, report: EstimatorReport | CrossValidationReport):
         """
         Put a key-report pair to the local project.
@@ -202,12 +219,6 @@ def put(self, key: str, report: EstimatorReport | CrossValidationReport):
                 f"ort` (found '{type(report)}')"
             )
 
-        if self.name not in self.__projects_storage:
-            raise RuntimeError(
-                f"Bad condition: {repr(self)} does not exist anymore, "
-                f"it had to be removed."
-            )
-
         pickle_hash, pickle_bytes = Project.pickle(report)
 
         if pickle_hash not in self.__artifacts_storage:
@@ -224,13 +235,9 @@ def put(self, key: str, report: EstimatorReport | CrossValidationReport):
         )
 
     @property
+    @ensure_project_is_not_deleted
     def reports(self):
         """Accessor for interaction with the persisted reports."""
-        if self.name not in self.__projects_storage:
-            raise RuntimeError(
-                f"Bad condition: {repr(self)} does not exist anymore, "
-                f"it had to be removed."
-            )
 
         def get(id: str) -> EstimatorReport:
             """Get a persisted report by its id."""

skore-local-project/tests/unit/test_project.py

@@ -192,8 +192,9 @@ def test_put_exception(self, tmp_path, regression):
         with raises(
             RuntimeError,
             match=re.escape(
-                f"Bad condition: {repr(project)} does not exist anymore, "
-                f"it had to be removed.",
+                f"Skore could not proceed because "
+                f"Project(mode='local', name='<project>', workspace='{tmp_path}') "
+                f"does not exist anymore."
             ),
         ):
             project.put("<key>", regression)
@@ -296,8 +297,9 @@ def test_reports_exception(self, tmp_path):
         with raises(
             RuntimeError,
             match=re.escape(
-                f"Bad condition: {repr(project)} does not exist anymore, "
-                f"it had to be removed.",
+                f"Skore could not proceed because "
+                f"Project(mode='local', name='<project>', workspace='{tmp_path}') "
+                f"does not exist anymore."
             ),
         ):
             project.reports  # noqa: B018