Merge pull request #44 from REANNZ/fix-raise-invalid-uri

c00kiemon5ter · web-flow · commit 970abcbf73a3 · 2022-11-08T13:08:29.000+02:00
fix: ValueError should be raised, not returned
diff --git a/src/pyop/storage.py b/src/pyop/storage.py
@@ -93,7 +93,7 @@ def from_uri(cls, db_uri, collection, db_name=None, ttl=None, **kwargs):
                 alg=alg
             )
 
-        return ValueError(f"Invalid DB URI: {db_uri}")
+        raise ValueError(f"Invalid DB URI: {db_uri}")
 
     @classmethod
     def type(cls, db_uri):
@@ -105,7 +105,7 @@ def type(cls, db_uri):
         elif url.scheme == "stateless":
             return "stateless"
 
-        return ValueError(f"Invalid DB URI: {db_uri}")
+        raise ValueError(f"Invalid DB URI: {db_uri}")
 
     @property
     def ttl(self):
