(CI-Test): cache test not finding cache.db, create tmp cache.db in test folder and cleanup

Author: Matars

src/gitfetch/cache.py

@@ -12,17 +12,20 @@
 class CacheManager:
     """Manages local caching of GitHub data using SQLite."""
 
-    CACHE_DIR = Path.home() / ".local" / "share" / "gitfetch"
-    DB_FILE = CACHE_DIR / "cache.db"
-
-    def __init__(self, cache_expiry_minutes: int = 15):
+    def __init__(self, cache_expiry_minutes: int = 15,
+                 cache_dir: Optional[Path] = None):
         """
         Initialize the cache manager.
 
         Args:
             cache_expiry_minutes: Minutes before cache expires (default: 15)
+            cache_dir: Directory to store cache files
+                (default: ~/.local/share/gitfetch)
         """
         self.cache_expiry_minutes = cache_expiry_minutes
+        self.CACHE_DIR = cache_dir or (
+            Path.home() / ".local" / "share" / "gitfetch")
+        self.DB_FILE = self.CACHE_DIR / "cache.db"
         self._ensure_cache_dir()
         self._init_database()
 

tests/test_cache.py

@@ -15,9 +15,8 @@ class TestCacheManager:
     def setup_method(self):
         """Set up test fixtures."""
         self.temp_dir = tempfile.mkdtemp()
-        with patch('src.gitfetch.cache.CacheManager.CACHE_DIR',
-                   Path(self.temp_dir)):
-            self.cache_manager = CacheManager(cache_expiry_minutes=15)
+        self.cache_manager = CacheManager(
+            cache_expiry_minutes=15, cache_dir=Path(self.temp_dir))
 
     def teardown_method(self):
         """Clean up test fixtures."""
@@ -60,10 +59,9 @@ def test_stale_cache_retrieval(self):
 
     def test_cache_minutes_parameter(self):
         """Test that cache_expiry_minutes parameter works."""
-        with patch('src.gitfetch.cache.CacheManager.CACHE_DIR',
-                   Path(self.temp_dir)):
-            cache_manager = CacheManager(cache_expiry_minutes=30)
-            assert cache_manager.cache_expiry_minutes == 30
+        cache_manager = CacheManager(
+            cache_expiry_minutes=30, cache_dir=Path(self.temp_dir))
+        assert cache_manager.cache_expiry_minutes == 30
 
     def test_cache_timestamp_update(self):
         """Test that caching updates the timestamp."""