updates

tests/test_changes.py

@@ -399,6 +399,7 @@ def test_detect_changes_tool_no_changes(self):
 
         # Patch _get_store to use our test store,
         # and get_changed_files/get_staged_and_unstaged to return empty.
+        original_close = self.store.close
         with (
             patch("code_review_graph.tools.review._get_store") as mock_get_store,
             patch("code_review_graph.tools.review.get_changed_files", return_value=[]),
@@ -414,13 +415,16 @@ def test_detect_changes_tool_no_changes(self):
             assert result["risk_score"] == 0.0
             assert result["changed_functions"] == []
             assert result["test_gaps"] == []
+        # Restore close method for teardown
+        self.store.close = original_close
 
     def test_detect_changes_tool_with_changes(self):
         """detect_changes_func returns full analysis for changed files."""
         from code_review_graph.tools import detect_changes_func
 
         self._add_func("my_func", path="/fake/repo/app.py", line_start=1, line_end=10)
 
+        original_close = self.store.close
         with (
             patch("code_review_graph.tools.review._get_store") as mock_get_store,
             patch("code_review_graph.tools.review.get_changed_files", return_value=["app.py"]),
@@ -438,3 +442,5 @@ def test_detect_changes_tool_with_changes(self):
             assert "risk_score" in result
             assert "test_gaps" in result
             assert "review_priorities" in result
+        # Restore close method for teardown
+        self.store.close = original_close

tests/test_tools.py

@@ -24,6 +24,7 @@ def setup_method(self):
 
     def teardown_method(self):
         self.store.close()
+        self.tmp.close()
         Path(self.tmp.name).unlink(missing_ok=True)
 
     def _seed_data(self):
@@ -201,6 +202,7 @@ def setup_method(self):
 
     def teardown_method(self):
         self.store.close()
+        self.tmp.close()
         Path(self.tmp.name).unlink(missing_ok=True)
 
     def test_finds_large_functions(self):