fix: add missing close() calls and sync test files with main branch

Author: github-actions[bot]

tests/test_cache.py

@@ -37,6 +37,7 @@ def test_cache_basic():
     assert cache.get("none_key") is None
 
     assert cache.number_of_items == 8
+    cache.close()
 
 
 def test_cache_miss():
@@ -46,6 +47,7 @@ def test_cache_miss():
     cache.set("key1", "value1")
     assert cache.get("key1") == "value1"
     assert cache.get("key2") is CACHE_MISS
+    cache.close()
 
 
 def test_cache_overwrite():
@@ -56,6 +58,7 @@ def test_cache_overwrite():
     cache.set("key", "value2")
     assert cache.get("key") == "value2"
     assert cache.number_of_items == 1
+    cache.close()
 
 
 def test_cache_ttl():
@@ -88,6 +91,7 @@ def test_cache_ttl():
 
     time.sleep(0.2)
     assert cache.get("key4") is CACHE_MISS
+    cache.close()
 
 
 def test_cache_ttl_thread():
@@ -128,6 +132,7 @@ def custom_deserializer(value):
     # Should raise ValueError for non-string values
     with pytest.raises(ValueError, match="Failed to serialize"):
         cache.set("key2", 42)
+    cache.close()
 
 
 def test_cache_serialization_error():
@@ -138,6 +143,7 @@ def failing_serializer(value):
 
     with pytest.raises(ValueError, match="Failed to serialize value of type"):
         cache.set("key", "value")
+    cache.close()
 
 
 def test_cache_deserialization_error():
@@ -151,6 +157,7 @@ def failing_deserializer(value):
 
     with pytest.raises(ValueError, match="Failed to deserialize value"):
         cache.get("key")
+    cache.close()
 
 
 def test_cache_complex_objects():
@@ -174,6 +181,7 @@ def test_cache_complex_objects():
     }
     cache.set("complex", complex_obj)
     assert cache.get("complex") == complex_obj
+    cache.close()
 
 
 def test_cache_lru_behavior():
@@ -196,6 +204,7 @@ def test_cache_lru_behavior():
     assert cache.get("key2") is CACHE_MISS  # Should be evicted
     assert cache.get("key3") == "value3"  # Should still be there
     assert cache.get("key4") == "value4"  # Should be there
+    cache.close()
 
 
 def test_cache_lru_behavior_with_set_overwrite():
@@ -218,6 +227,7 @@ def test_cache_lru_behavior_with_set_overwrite():
     assert cache.get("key2") is CACHE_MISS  # Should be evicted
     assert cache.get("key3") == "value3"  # Should still be there
     assert cache.get("key4") == "value4"  # Should be there
+    cache.close()
 
 
 def test_cache_size_limit():
@@ -236,6 +246,7 @@ def test_cache_size_limit():
 
     # Large value should be rejected (exceeds size_limit_in_bytes / 2)
     assert cache.get("large") is CACHE_MISS
+    cache.close()
 
 
 def test_cache_delete():
@@ -256,6 +267,7 @@ def test_cache_delete():
     # Delete non-existent key should do nothing
     assert cache.delete("nonexistent") is False
     assert cache.number_of_items == 1
+    cache.close()
 
 
 def test_cache_clean_expired():
@@ -277,6 +289,7 @@ def test_cache_clean_expired():
     assert cache.number_of_items == 1
     assert cache.get("key1") is CACHE_MISS
     assert cache.get("key2") == "value2"
+    cache.close()
 
 
 def test_cache_close():
@@ -321,6 +334,7 @@ def test_cache_default_serializer_pickle():
     assert isinstance(retrieved, dict)
     retrieved_dict = cast(dict[str, Any], retrieved)
     assert retrieved_dict["level1"]["level2"]["level3"]["list"][3]["nested"] == "dict"
+    cache.close()
 
 
 def test_cache_multiple_operations():
@@ -345,6 +359,7 @@ def test_cache_multiple_operations():
     # Verify unchanged values
     for i in range(25, 50):
         assert cache.get(f"key{i}") == f"value{i}"
+    cache.close()
 
 
 def test_cache_empty_values():
@@ -364,6 +379,7 @@ def test_cache_empty_values():
 
     cache.set("false", False)
     assert cache.get("false") is False
+    cache.close()
 
 
 def test_cache_default_ttl_sentinel():
@@ -382,6 +398,7 @@ def test_cache_default_ttl_sentinel():
 
     time.sleep(0.2)
     assert cache.get("key2") == "value2"  # Should still be there
+    cache.close()
 
 
 def test_cache_concurrent_get_set_delete_clear():

tests/test_storage.py

@@ -147,6 +147,7 @@ def test_storage_expiration_manual():
     checked, deleted = storage._clean_expired()
     assert checked == 81
     assert deleted == 80
+    storage.close()
 
 
 def test_storage_expiration_thread():
@@ -191,6 +192,7 @@ def test_storage_too_many_items():
         if i % 10 == 0:
             # These keys should be still in the storage (thanks to the LRU behavior)
             assert storage.get(f"key{i}") == f"v{i}".encode()
+    storage.close()
 
 
 def test_storage_various():
@@ -210,6 +212,7 @@ def test_storage_various():
 
     # Get an expired key should return CACHE_MISS
     assert storage.get("key1") is CACHE_MISS
+    storage.close()
 
 
 def test_storage_delete_return_value():
@@ -254,6 +257,7 @@ def test_max_size_limit():
     assert storage.number_of_items > 10
     assert storage.size_in_bytes > 3500
     assert storage.size_in_bytes < 4096
+    storage.close()
 
 
 def test_overwrite_existing_key_size_tracking():
@@ -491,6 +495,12 @@ def test_storage_invalid_parameters():
     with pytest.raises(ValueError, match="expiration_thread_delay must be positive"):
         Storage[bytes](expiration_thread_delay=-1.0)
 
+    with pytest.raises(
+        ValueError,
+        match="expiration_thread_max_checks_per_iteration cannot be negative",
+    ):
+        Storage[bytes](expiration_thread_max_checks_per_iteration=-1)
+
 
 def test_overwrite_lru_item_size_tracking():
     """Size tracking stays correct when set() overwrites a key that gets evicted as the LRU item.