fixes

Author: shaypal5

src/cachier/core.py

@@ -13,7 +13,7 @@
 from collections import OrderedDict
 from concurrent.futures import ThreadPoolExecutor
 from datetime import datetime, timedelta
-from functools import partial, wraps
+from functools import wraps
 from typing import Any, Callable, Optional, Union
 from warnings import warn
 
@@ -238,10 +238,9 @@ def _cachier_decorator(func):
         # and 'max_age' (if provided).
         # This ensures that the strictest freshness requirement is enforced.
         #
-        # The main function wrapper is created using
-        # partial(_call, timedelta.max), so that by default, max_age is
-        # effectively infinite (i.e., only 'stale_after' is considered
-        # unless overridden).
+        # The main function wrapper is a standard function that passes
+        # *args and **kwargs to _call. By default, max_age is None,
+        # so only 'stale_after' is considered unless overridden.
         #
         # The user-facing API exposes:
         #   - Per-call: myfunc(..., max_age=timedelta(...))

tests/test_call_with_freshness_threshold.py

@@ -5,8 +5,6 @@
 
 
 def test_call_with_freshness_threshold():
-    from datetime import datetime
-
     @cachier.cachier()
     def test_func(a, b):
         return a + b
@@ -20,7 +18,7 @@ def test_func(a, b):
     # Wait for cache to become stale
     time.sleep(1.0)
     # Should trigger recalculation (stale)
-    val3 = test_func(1, 2, max_age=0.5)
+    val3 = test_func(1, 2, max_age=timedelta(seconds=0.5))
     assert val3 == 3
 
 
@@ -57,7 +55,7 @@ def f(x):
     assert v1 == v2
     time.sleep(1.1)
     v3 = f(1, max_age=timedelta(seconds=5))
-    assert v3 == v1  # max_age looser, cache still valid
+    assert v3 != v1  # max_age looser, but stale_after still applies (stricter)
 
 
 def test_max_age_none_defaults_to_stale_after():
@@ -119,5 +117,6 @@ def f(x):
     v1 = f(1)
     time.sleep(1.1)
     v2 = f(1, max_age=timedelta(seconds=0.5))
-    # With next_time=True, should return stale value (v1), not recalc immediately
+    # With next_time=True, should return stale value (v1) while
+    # triggering a recalculation in the background
     assert v2 == v1