Ruff

Author: MoonBoi9001

src/utils/circuit_breaker.py

@@ -44,13 +44,13 @@ def _get_failure_timestamps(self) -> List[datetime]:
         # If the log file does not exist, return an empty list
         if not self.log_file.exists():
             return []
-        
+
         # If the log file exists, read and parse all timestamps
         try:
             with self.log_file.open("r") as f:
                 timestamps = [datetime.fromisoformat(line.strip()) for line in f if line.strip()]
             return timestamps
-        
+
         # If there is an error reading or parsing the log file, log the error and return an empty list
         except (IOError, ValueError) as e:
             logger.error(f"Error reading or parsing circuit breaker log file {self.log_file}: {e}")
@@ -105,7 +105,7 @@ def record_failure(self) -> None:
 
             # Log the success
             logger.warning("Circuit breaker has recorded a failure.")
-        
+
         # If there is an error appending the timestamp to the log file, log the error
         except IOError as e:
             logger.error(f"Failed to record failure to circuit breaker log {self.log_file}: {e}")

tests/test_circuit_breaker.py

@@ -1,6 +1,7 @@
 """
 Unit tests for the CircuitBreaker utility.
 """
+
 from datetime import datetime, timedelta
 from unittest.mock import MagicMock, mock_open, patch
 
@@ -12,7 +13,7 @@
 @pytest.fixture
 def mock_path():
     """Fixture to mock the Path object for file system interactions."""
-    with patch("src.utils.circuit_breaker.Path") as mock_path_cls:
+    with patch("src.utils.circuit_breaker.Path"):
         mock_instance = MagicMock()
         mock_instance.exists.return_value = False
         mock_instance.open = mock_open()
@@ -117,4 +118,4 @@ def test_reset_does_nothing_if_log_file_does_not_exist(breaker: CircuitBreaker,
     """
     mock_path.exists.return_value = False
     breaker.reset()
-    mock_path.unlink.assert_not_called() 
+    mock_path.unlink.assert_not_called()