[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

scripts/test-local.sh

@@ -206,14 +206,14 @@ check_docker() {
         echo ""
         echo "After starting Docker, wait a few seconds and try running this script again."
         echo ""
-        
+
         # Show the actual docker error for debugging
         echo "Technical details:"
         docker ps 2>&1 | sed 's/^/  /'
         echo ""
         exit 1
     fi
-    
+
     print_message $GREEN "✓ Docker is installed and running"
 }
 
@@ -488,13 +488,13 @@ main() {
     # Run pytest
     # Build pytest command
     PYTEST_CMD="pytest"
-    
+
     # Add test files if specified
     if [ -n "$TEST_FILES" ]; then
         PYTEST_CMD="$PYTEST_CMD $TEST_FILES"
         print_message $BLUE "Test files specified: $TEST_FILES"
     fi
-    
+
     # Add markers if needed (only if no specific test files were given)
     if [ -z "$TEST_FILES" ]; then
         # Check if we selected all cores - if so, run all tests without marker filtering
@@ -510,20 +510,20 @@ main() {
         all_cores="memory mongo pickle redis sql"
         selected_sorted=$(echo "$SELECTED_CORES" | tr ' ' '\n' | sort | tr '\n' ' ' | xargs)
         all_sorted=$(echo "$all_cores" | tr ' ' '\n' | sort | tr '\n' ' ' | xargs)
-        
+
         if [ "$selected_sorted" != "$all_sorted" ]; then
             PYTEST_CMD="$PYTEST_CMD -m \"$pytest_markers\""
         fi
     fi
-    
+
     # Add verbose flag if needed
     if [ "$VERBOSE" = true ]; then
         PYTEST_CMD="$PYTEST_CMD -v"
     fi
-    
+
     # Add coverage options
     PYTEST_CMD="$PYTEST_CMD --cov=cachier --cov-report=$COVERAGE_REPORT"
-    
+
     # Print and run the command
     print_message $BLUE "Running: $PYTEST_CMD"
     eval $PYTEST_CMD

tests/test_config.py

@@ -10,13 +10,19 @@
 def test_set_default_params_deprecated():
     """Test that set_default_params shows deprecation warning."""
     # Test lines 103-111: deprecation warning
-    with pytest.warns(DeprecationWarning, match="set_default_params.*deprecated.*set_global_params"):
+    with pytest.warns(
+        DeprecationWarning,
+        match="set_default_params.*deprecated.*set_global_params",
+    ):
         set_default_params(stale_after=60)
 
 
 def test_get_default_params_deprecated():
     """Test that get_default_params shows deprecation warning."""
     # Test lines 143-151: deprecation warning
-    with pytest.warns(DeprecationWarning, match="get_default_params.*deprecated.*get_global_params"):
+    with pytest.warns(
+        DeprecationWarning,
+        match="get_default_params.*deprecated.*get_global_params",
+    ):
         params = get_default_params()
-        assert params is not None
+        assert params is not None

tests/test_main.py

@@ -17,33 +17,40 @@ def test_cli_group():
 def test_set_max_workers_command():
     """Test the set_max_workers command."""
     runner = CliRunner()
-    
+
     # First check if the command exists in the CLI
     result = runner.invoke(cli, ["--help"])
     assert result.exit_code == 0
-    
+
     # The command decorator syntax in __main__.py is incorrect
     # It should be @cli.command() or @cli.command("command-name")
     # Currently it's using the description as the command name
     # So the command is registered with a long name
-    
+
     # Test with the actual registered command name
-    result = runner.invoke(cli, ["Limits the number of worker threads used by cachier.", "4"])
+    result = runner.invoke(
+        cli, ["Limits the number of worker threads used by cachier.", "4"]
+    )
     assert result.exit_code == 0
-    
+
     # Test with invalid input (non-integer)
-    result = runner.invoke(cli, ["Limits the number of worker threads used by cachier.", "invalid"])
+    result = runner.invoke(
+        cli,
+        ["Limits the number of worker threads used by cachier.", "invalid"],
+    )
     assert result.exit_code != 0
-    
+
     # Test without argument
-    result = runner.invoke(cli, ["Limits the number of worker threads used by cachier."])
+    result = runner.invoke(
+        cli, ["Limits the number of worker threads used by cachier."]
+    )
     assert result.exit_code != 0
 
 
 def test_set_max_workers_function():
     """Test the set_max_workers function directly."""
     # This tests the function import and ensures it's callable
     # The actual functionality is tested in core tests
-    
+
     # Verify the function is callable
-    assert callable(set_max_workers)
+    assert callable(set_max_workers)

tests/test_memory_core.py

@@ -3,7 +3,6 @@
 import hashlib
 import queue
 import threading
-from datetime import timedelta
 from datetime import datetime, timedelta
 from random import random
 from time import sleep, time

tests/test_pickle_core.py

@@ -11,15 +11,15 @@
 #     realpath,
 #     dirname
 # )
+import hashlib
 import os
-import sys
-import time
 import pickle
-import hashlib
+import sys
 import tempfile
 import threading
-from pathlib import Path
+import time
 from datetime import datetime, timedelta
+from pathlib import Path
 from random import random
 from time import sleep, time
 from unittest.mock import MagicMock, Mock, patch
@@ -35,8 +35,7 @@
 import pandas as pd
 
 from cachier import cachier
-from cachier.config import _global_params
-from cachier.config import CacheEntry
+from cachier.config import CacheEntry, _global_params
 from cachier.cores.pickle import _PickleCore
 
 

tests/test_redis_core.py

@@ -3,10 +3,10 @@
 import hashlib
 import queue
 import threading
-from random import random
-from time import sleep
 import warnings
 from datetime import datetime, timedelta
+from random import random
+from time import sleep
 from unittest.mock import MagicMock, Mock, patch
 
 import pandas as pd

tests/test_util.py

@@ -18,9 +18,9 @@ def test_parse_bytes_invalid_format():
     # Test line 15: ValueError for invalid format
     with pytest.raises(ValueError, match="Invalid size value: invalid"):
         parse_bytes("invalid")
-    
+
     with pytest.raises(ValueError, match="Invalid size value: 123XB"):
         parse_bytes("123XB")
-    
+
     with pytest.raises(ValueError, match="Invalid size value: abc123"):
-        parse_bytes("abc123")
+        parse_bytes("abc123")