Update tests

Author: awolverp

.gitignore

@@ -4,3 +4,4 @@ __pycache__
 *.so
 /.coverage
 /.pytest_cache
+/htmlcov

python/cachebox/_cachebox.py

@@ -96,7 +96,7 @@ def capacity(self) -> int:
     def __len__(self) -> int:
         return len(self._raw)
 
-    def __sizeof__(self):
+    def __sizeof__(self):  # pragma: no cover
         return self._raw.__sizeof__()
 
     def __contains__(self, key: KT) -> bool:
@@ -147,10 +147,10 @@ def setdefault(self, key: KT, default: typing.Optional[DT] = None) -> typing.Uni
         """
         return self._raw.setdefault(key, default)
 
-    def popitem(self) -> typing.NoReturn:
+    def popitem(self) -> typing.NoReturn:  # pragma: no cover
         raise NotImplementedError()
 
-    def drain(self) -> typing.NoReturn:
+    def drain(self) -> typing.NoReturn:  # pragma: no cover
         raise NotImplementedError()
 
     def update(self, iterable: typing.Union["Cache", dict, tuple, typing.Generator]) -> None:
@@ -181,13 +181,13 @@ def __delitem__(self, key: KT) -> None:
 
     def __eq__(self, other) -> bool:
         if not isinstance(other, Cache):
-            return False
+            return False  # pragma: no cover
 
         return self._raw == other._raw
 
     def __ne__(self, other) -> bool:
         if not isinstance(other, Cache):
-            return False
+            return False  # pragma: no cover
 
         return self._raw != other._raw
 
@@ -268,7 +268,7 @@ def capacity(self) -> int:
     def __len__(self) -> int:
         return len(self._raw)
 
-    def __sizeof__(self):
+    def __sizeof__(self):  # pragma: no cover
         return self._raw.__sizeof__()
 
     def __contains__(self, key: KT) -> bool:
@@ -307,7 +307,7 @@ def popitem(self) -> typing.Tuple[KT, VT]:
         except _core.CoreKeyError:
             raise KeyError() from None
 
-    def drain(self, n: int) -> int:
+    def drain(self, n: int) -> int:  # pragma: no cover
         if n == 0:
             return 0
 
@@ -342,13 +342,13 @@ def __delitem__(self, key: KT) -> None:
 
     def __eq__(self, other) -> bool:
         if not isinstance(other, FIFOCache):
-            return False
+            return False  # pragma: no cover
 
         return self._raw == other._raw
 
     def __ne__(self, other) -> bool:
         if not isinstance(other, FIFOCache):
-            return False
+            return False  # pragma: no cover
 
         return self._raw != other._raw
 

python/tests/mixin.py

@@ -26,7 +26,7 @@ def __hash__(self) -> int:
         return self.val
 
 
-def getsizeof(obj, use_sys=True):
+def getsizeof(obj, use_sys=True):  # pragma: no cover
     try:
         if use_sys:
             return sys.getsizeof(obj)
@@ -36,7 +36,7 @@ def getsizeof(obj, use_sys=True):
         return len(obj)
 
 
-class _TestMixin:
+class _TestMixin:  # pragma: no cover
     CACHE: typing.Type[BaseCacheImpl]
 
     KWARGS: dict = {}
@@ -100,15 +100,6 @@ def test___len__(self):
         assert len(cache) == 10
         assert cache.is_full()
 
-    def test___sizeof__(self):
-        cache = self.CACHE(10, **self.KWARGS, capacity=10)
-
-        # all classes have to implement __sizeof__
-        # __sizeof__ returns exactly allocated memory size by cache
-        # but sys.getsizeof add also garbage collector overhead to that, so sometimes
-        # sys.getsizeof is greater than __sizeof__
-        getsizeof(cache, False)
-
     def test___bool__(self):
         cache = self.CACHE(1, **self.KWARGS, capacity=1)
 
@@ -146,16 +137,22 @@ def test___setitem__(self):
         del cache[2]
         del cache[3]
 
+        with pytest.raises(KeyError):
+            del cache["error"]
+
         cache[0]
 
         with pytest.raises(KeyError):
             cache[2]
 
     def test___repr__(self):
-        cache = self.CACHE(2, **self.KWARGS, capacity=2)
+        cache = self.CACHE(100, **self.KWARGS, capacity=2)
         assert str(cache) == repr(cache)
         assert repr(cache).startswith(self.CACHE.__name__)
 
+        cache.update({i: i for i in range(100)})
+        assert str(cache) == repr(cache)
+
     def test_insert(self):
         cache = self.CACHE(5, **self.KWARGS, capacity=5)
 

python/tests/test_caches.py

@@ -88,3 +88,5 @@ def test_first_last(self):
 
         assert obj.first() == 1
         assert obj.last() == 10
+        assert obj.first(-1) == obj.last()
+        assert obj.first(-10000) is None