feat(QueryList): Return MultipleObjectsReturned, ObjectDoesNotExist

Author: tony

src/libvcs/_internal/query_list.py

@@ -14,6 +14,14 @@
 no_arg = object()
 
 
+class MultipleObjectsReturned(Exception):
+    """The requested object does not exist"""
+
+
+class ObjectDoesNotExist(Exception):
+    """The query returned multiple objects when only one was expected."""
+
+
 def keygetter(
     obj: Mapping[str, Any],
     path: str,
@@ -522,9 +530,9 @@ def get(
     ) -> Optional[T]:
         objs = self.filter(matcher=matcher, **kwargs)
         if len(objs) > 1:
-            raise Exception("Multiple objects returned")
+            raise MultipleObjectsReturned()
         elif len(objs) == 0:
             if default == no_arg:
-                raise Exception("No objects found")
+                raise ObjectDoesNotExist()
             return default
         return objs[0]

tests/_internal/test_query_list.py

@@ -3,7 +3,11 @@
 
 import pytest
 
-from libvcs._internal.query_list import QueryList
+from libvcs._internal.query_list import (
+    MultipleObjectsReturned,
+    ObjectDoesNotExist,
+    QueryList,
+)
 
 
 @dataclasses.dataclass
@@ -269,16 +273,16 @@ def test_filter(
             else:
                 assert qs.get(filter_expr) == expected_result[0]
         elif len(expected_result) > 1:
-            with pytest.raises(Exception) as e:
+            with pytest.raises(MultipleObjectsReturned) as e:
                 if isinstance(filter_expr, dict):
                     assert qs.get(**filter_expr) == expected_result
                 else:
                     assert qs.get(filter_expr) == expected_result
                 assert e.match("Multiple objects returned")
         elif len(expected_result) == 0:
-            with pytest.raises(Exception) as e:
+            with pytest.raises(ObjectDoesNotExist) as exc:
                 if isinstance(filter_expr, dict):
                     assert qs.get(**filter_expr) == expected_result
                 else:
                     assert qs.get(filter_expr) == expected_result
-            assert e.match("No objects found")
+            assert exc.match("No objects found")