Project: Don't funnel ::load() errors into RuntimeErrors

Instead, let the exceptions propagate up to the caller.  For one thing,
this allows us to write more specific tests :^)

ultratrace2/model/project.py

@@ -21,14 +21,10 @@ def save(self):
 
     @classmethod
     def load(cls, save_file: str) -> "Project":
-        try:
-            with open(save_file, "rb") as fp:
-                project = pickle.load(fp)
-                assert isinstance(project, Project)
-                return project
-        except Exception as e:
-            logger.error(e)
-            raise RuntimeError(str(e))
+        with open(save_file, "rb") as fp:
+            project = pickle.load(fp)
+            assert isinstance(project, Project)
+            return project
 
     @classmethod
     def get_by_path(cls, root_path: str) -> "Project":
@@ -51,7 +47,8 @@ def get_by_path(cls, root_path: str) -> "Project":
         save_file = cls.get_save_file(root_path)
         try:
             return cls.load(save_file)
-        except RuntimeError:
+        except Exception as e:
+            logger.warning(e)
             logger.info(
                 f"Unable to find existing project at {root_path}, creating new one..."
             )

ultratrace2/model/tests/test_project.py

@@ -8,10 +8,10 @@
 @pytest.mark.parametrize(
     "save_file,error",
     [
-        ("", RuntimeError),
-        ("/path/to/nowhere", RuntimeError),
-        ("/dev/null", RuntimeError),  # pickle cannot open
-        ("/etc/sudoers", RuntimeError),  # not readable
+        ("", FileNotFoundError),
+        ("/path/to/nowhere", FileNotFoundError),
+        ("/dev/null", EOFError),  # pickle cannot open
+        ("/etc/sudoers", PermissionError),  # not readable
     ],
 )
 def test_load_project_invalid(save_file: str, error: Exception) -> None: