[pre-commit.ci] pre-commit suggestions (#236)

* [pre-commit.ci] pre-commit suggestions

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.6.0 → v5.0.0](pre-commit/pre-commit-hooks@v4.6.0...v5.0.0)
- [github.com/crate-ci/typos: v1.22.9 → v1.26.0](crate-ci/typos@v1.22.9...v1.26.0)
- [github.com/pre-commit/mirrors-prettier: v3.1.0 → v4.0.0-alpha.8](pre-commit/mirrors-prettier@v3.1.0...v4.0.0-alpha.8)
- [github.com/astral-sh/ruff-pre-commit: v0.5.0 → v0.6.9](astral-sh/ruff-pre-commit@v0.5.0...v0.6.9)
- [github.com/tox-dev/pyproject-fmt: 2.1.3 → 2.2.4](tox-dev/pyproject-fmt@2.1.3...2.2.4)
- [github.com/abravalheri/validate-pyproject: v0.18 → v0.20.2](abravalheri/validate-pyproject@v0.18...v0.20.2)

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

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

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Author: pre-commit-ci[bot]

.pre-commit-config.yaml

@@ -9,7 +9,7 @@ ci:
 
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.6.0
+    rev: v5.0.0
     hooks:
       - id: end-of-file-fixer
       - id: trailing-whitespace
@@ -23,7 +23,7 @@ repos:
       - id: detect-private-key
 
   - repo: https://github.com/crate-ci/typos
-    rev: v1.22.9
+    rev: v1.26.0
     hooks:
       - id: typos
         # empty to do not write fixes
@@ -47,7 +47,7 @@ repos:
         args: ["--number"]
 
   - repo: https://github.com/pre-commit/mirrors-prettier
-    rev: v3.1.0
+    rev: v4.0.0-alpha.8
     hooks:
       - id: prettier
         files: \.(json|yml|yaml|toml)
@@ -60,7 +60,7 @@ repos:
   #      - id: mypy
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.5.0
+    rev: v0.6.9
     hooks:
       # use black formatting
       - id: ruff-format
@@ -71,11 +71,11 @@ repos:
         args: ["--fix"]
 
   - repo: https://github.com/tox-dev/pyproject-fmt
-    rev: 2.1.3
+    rev: 2.2.4
     hooks:
       - id: pyproject-fmt
         additional_dependencies: [tox]
   - repo: https://github.com/abravalheri/validate-pyproject
-    rev: v0.18
+    rev: v0.20.2
     hooks:
       - id: validate-pyproject

tests/test_core_lookup.py

@@ -1,6 +1,7 @@
 """Testing a few basic cachier interfaces."""
 
 import pytest
+
 from cachier import cachier, get_default_params
 from cachier.cores.mongo import MissingMongetter
 

tests/test_defaults.py

@@ -5,9 +5,9 @@
 import threading
 import time
 
-import cachier
 import pytest
 
+import cachier
 from tests.test_mongo_core import _test_mongetter
 
 MONGO_DELTA = datetime.timedelta(seconds=3)
@@ -65,7 +65,7 @@ def global_test_2():
     assert global_test_2.cache_dpath() is not None
 
 
-@pytest.mark.mongo()
+@pytest.mark.mongo
 def test_mongetter_default_param():
     cachier.set_default_params(mongetter=_test_mongetter)
 

tests/test_general.py

@@ -10,16 +10,16 @@
 from random import random
 from time import sleep, time
 
-import cachier
 import pytest
+
+import cachier
 from cachier.core import (
     DEFAULT_MAX_WORKERS,
     MAX_WORKERS_ENVAR_NAME,
     _get_executor,
     _max_workers,
     _set_max_workers,
 )
-
 from tests.test_mongo_core import (
     _test_mongetter,
 )

tests/test_memory_core.py

@@ -9,6 +9,7 @@
 
 import pandas as pd
 import pytest
+
 from cachier import cachier
 
 
@@ -19,7 +20,7 @@ def _takes_2_seconds(arg_1, arg_2):
     return f"arg_1:{arg_1}, arg_2:{arg_2}"
 
 
-@pytest.mark.memory()
+@pytest.mark.memory
 def test_memory_core():
     """Basic memory core functionality."""
     _takes_2_seconds.clear_cache()
@@ -31,7 +32,7 @@ def test_memory_core():
     _takes_2_seconds.clear_cache()
 
 
-@pytest.mark.memory()
+@pytest.mark.memory
 def test_memory_core_keywords():
     """Basic memory core functionality with keyword arguments."""
     _takes_2_seconds.clear_cache()
@@ -53,7 +54,7 @@ def _stale_after_seconds(arg_1, arg_2):
     return random()
 
 
-@pytest.mark.memory()
+@pytest.mark.memory
 def test_stale_after():
     """Testing the stale_after functionality."""
     _stale_after_seconds.clear_cache()
@@ -74,7 +75,7 @@ def _stale_after_next_time(arg_1, arg_2):
     return random()
 
 
-@pytest.mark.memory()
+@pytest.mark.memory
 def test_stale_after_next_time():
     """Testing the stale_after with next_time functionality."""
     _stale_after_next_time.clear_cache()
@@ -103,7 +104,7 @@ def _random_num_with_arg(a):
     return random()
 
 
-@pytest.mark.memory()
+@pytest.mark.memory
 def test_overwrite_cache():
     """Tests that the overwrite feature works correctly."""
     _random_num.clear_cache()
@@ -127,7 +128,7 @@ def test_overwrite_cache():
     _random_num_with_arg.clear_cache()
 
 
-@pytest.mark.memory()
+@pytest.mark.memory
 def test_ignore_cache():
     """Tests that the ignore_cache feature works correctly."""
     _random_num.clear_cache()
@@ -165,7 +166,7 @@ def _calls_takes_time(res_queue):
     res_queue.put(res)
 
 
-@pytest.mark.memory()
+@pytest.mark.memory
 def test_memory_being_calculated():
     """Testing memory core handling of being calculated scenarios."""
     _takes_time.clear_cache()
@@ -199,7 +200,7 @@ def _calls_being_calc_next_time(res_queue):
     res_queue.put(res)
 
 
-@pytest.mark.memory()
+@pytest.mark.memory
 def test_being_calc_next_time():
     """Testing memory core handling of being calculated scenarios."""
     _takes_time.clear_cache()
@@ -241,7 +242,7 @@ def _delete_cache(arg_1, arg_2):
     return random() + arg_1 + arg_2
 
 
-@pytest.mark.memory()
+@pytest.mark.memory
 def test_clear_being_calculated():
     """Test memory core clear `being calculated` functionality."""
     _takes_time.clear_cache()
@@ -264,7 +265,7 @@ def test_clear_being_calculated():
     assert res1 != res2
 
 
-@pytest.mark.memory()
+@pytest.mark.memory
 def test_clear_being_calculated_with_empty_cache():
     """Test memory core clear `being calculated` functionality."""
     _takes_time.clear_cache()
@@ -281,7 +282,7 @@ def _error_throwing_func(arg1):
     return 7
 
 
-@pytest.mark.memory()
+@pytest.mark.memory
 def test_error_throwing_func():
     # with
     res1 = _error_throwing_func(4)
@@ -290,7 +291,7 @@ def test_error_throwing_func():
     assert res1 == res2
 
 
-@pytest.mark.memory()
+@pytest.mark.memory
 def test_callable_hash_param():
     def _hash_func(args, kwargs):
         def _hash(obj):

tests/test_mongo_core.py

@@ -14,13 +14,14 @@
 import pymongo
 import pytest
 from birch import Birch  # type: ignore[import-not-found]
-from cachier import cachier
-from cachier.cores.base import RecalculationNeeded
-from cachier.cores.mongo import _MongoCore
 from pymongo.errors import OperationFailure
 from pymongo.mongo_client import MongoClient
 from pymongo_inmemory import MongoClient as InMemoryMongoClient
 
+from cachier import cachier
+from cachier.cores.base import RecalculationNeeded
+from cachier.cores.mongo import _MongoCore
+
 # === Enables testing vs a real MongoDB instance ===
 
 
@@ -76,13 +77,13 @@ def _test_mongetter():
 # === Mongo core tests ===
 
 
-@pytest.mark.mongo()
+@pytest.mark.mongo
 def test_information():
     print("\npymongo version: ", end="")
     print(pymongo.__version__)
 
 
-@pytest.mark.mongo()
+@pytest.mark.mongo
 def test_mongo_index_creation():
     """Basic Mongo core functionality."""
 
@@ -99,7 +100,7 @@ def _test_mongo_caching(arg_1, arg_2):
     assert _MongoCore._INDEX_NAME in collection.index_information()
 
 
-@pytest.mark.mongo()
+@pytest.mark.mongo
 def test_mongo_core():
     """Basic Mongo core functionality."""
 
@@ -122,7 +123,7 @@ def _test_mongo_caching(arg_1, arg_2):
     assert val6 == val5
 
 
-@pytest.mark.mongo()
+@pytest.mark.mongo
 def test_mongo_core_keywords():
     """Basic Mongo core functionality with keyword arguments."""
 
@@ -145,7 +146,7 @@ def _test_mongo_caching(arg_1, arg_2):
     assert val6 == val5
 
 
-@pytest.mark.mongo()
+@pytest.mark.mongo
 def test_mongo_stale_after():
     """Testing MongoDB core stale_after functionality."""
 
@@ -178,7 +179,7 @@ def _takes_time(arg_1, arg_2):
     res_queue.put(res)
 
 
-@pytest.mark.mongo()
+@pytest.mark.mongo
 def test_mongo_being_calculated():
     """Testing MongoDB core handling of being calculated scenarios."""
 
@@ -228,7 +229,7 @@ def _bad_mongetter():
     return _BadMongoCollection(_test_mongetter)
 
 
-@pytest.mark.mongo()
+@pytest.mark.mongo
 def test_mongo_write_failure():
     """Testing MongoDB core handling of writing failure scenarios."""
 
@@ -244,7 +245,7 @@ def _func_w_bad_mongo(arg_1, arg_2):
     # assert val1 == val2
 
 
-@pytest.mark.mongo()
+@pytest.mark.mongo
 def test_mongo_clear_being_calculated():
     """Testing MongoDB core clear_being_calculated."""
 
@@ -256,7 +257,7 @@ def _func_w_bad_mongo(arg_1, arg_2):
     _func_w_bad_mongo.clear_being_calculated()
 
 
-@pytest.mark.mongo()
+@pytest.mark.mongo
 def test_stalled_mongo_db_cache():
     @cachier(mongetter=_test_mongetter)
     def _stalled_func():
@@ -269,7 +270,7 @@ def _stalled_func():
         core.wait_on_entry_calc(key=None)
 
 
-@pytest.mark.mongo()
+@pytest.mark.mongo
 def test_stalled_mong_db_core(monkeypatch):
     def mock_get_entry(self, args, kwargs):
         return "key", {"being_calculated": True}
@@ -325,7 +326,7 @@ def _stalled_func_3():
     assert res == 1
 
 
-@pytest.mark.mongo()
+@pytest.mark.mongo
 def test_callable_hash_param():
     def _hash_func(args, kwargs):
         def _hash(obj):