Apply suggestions from code review

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Author: Borda

src/cachier/core.py

@@ -375,10 +375,10 @@ def _cachier_decorator(func):
                     f"@cachier cannot decorate instance method "
                     f"'{func.__qualname__}' because the 'self' parameter is "
                     f"excluded from cache-key computation and all instances "
-                    f"would share a single cache. Pass "
-                    f"allow_non_static_methods=True to the decorator (or "
-                    f"set_global_params) if cross-instance cache sharing is "
-                    f"intentional."
+                    f"would share a single cache. Pass allow_non_static_methods=True "
+                    f"to the decorator or call "
+                    f"set_global_params(allow_non_static_methods=True) if "
+                    f"cross-instance cache sharing is intentional."
                 )
 
         is_coroutine = inspect.iscoroutinefunction(func)

tests/test_smoke.py

@@ -174,9 +174,10 @@ def func():
 def test_classmethod_not_guarded():
     """Test that @classmethod (cls) does not trigger the instance method guard.
 
-    Note: The decorator order must be @classmethod first, then @cachier,
-    so that cachier sees the underlying function (whose first param is
-    ``cls``, not ``self``) and the guard is not triggered.
+    Note: Place ``@classmethod`` above ``@cachier`` so that ``@cachier``
+    decorates the underlying function before it is wrapped as a classmethod.
+    This way cachier sees ``cls`` (not ``self``) as the first parameter and
+    the instance-method guard is not triggered.
 
     """
 
@@ -204,15 +205,18 @@ def method(cls, x):
 @pytest.mark.smoke
 def test_instance_method_global_opt_out_reset():
     """Test that resetting allow_non_static_methods=False re-enables the guard."""
-    set_global_params(allow_non_static_methods=True)
-    set_global_params(allow_non_static_methods=False)
-    with pytest.raises(TypeError, match="instance method"):
-
-        class Foo:
-            @cachier_decorator(backend="memory")
-            def method(self, x):
-                return x
-
+    original = get_global_params().allow_non_static_methods
+    try:
+        set_global_params(allow_non_static_methods=True)
+        set_global_params(allow_non_static_methods=False)
+        with pytest.raises(TypeError, match="instance method"):
+
+            class Foo:
+                @cachier_decorator(backend="memory")
+                def method(self, x):
+                    return x
+    finally:
+        set_global_params(allow_non_static_methods=original)
 
 @pytest.mark.smoke
 def test_instance_method_skip_cache():