redis-lua-py 0.5.0__tar.gz → 0.5.1__tar.gz

{redis_lua_py-0.5.0 → redis_lua_py-0.5.1}/.github/workflows/ci.yml

@@ -37,6 +37,17 @@ jobs:
   oldest:
     name: Tests (oldest supported Python and redis-py)
     runs-on: ubuntu-latest
+    # A real server as well, since what fakeredis cannot run -- Redis Functions
+    # and script flags -- is exactly what old redis-py spells differently.
+    services:
+      redis:
+        image: redis:7
+        ports: ["6379:6379"]
+        options: >-
+          --health-cmd "redis-cli ping"
+          --health-interval 10s
+          --health-timeout 5s
+          --health-retries 5
     steps:
       - uses: actions/checkout@v4
       - uses: astral-sh/setup-uv@v5
@@ -46,8 +57,11 @@ jobs:
       # The floor in pyproject.toml is only a claim until something runs on it.
       # fakeredis declares redis>=4.3 but works on 4.2, the first release with
       # redis.asyncio, so the pin goes in after the resolver is done.
-      - run: uv pip install --no-deps "redis==4.2.0" async-timeout deprecated
+      - run: uv pip install --no-deps "redis==4.2.0" async-timeout deprecated packaging
       - run: uv run --no-sync pytest
+      - run: uv run --no-sync pytest
+        env:
+          REDIS_URL: redis://localhost:6379/0
 
   live:
     name: Tests against a real Redis

redis_lua_py-0.5.1/.release-please-manifest.json

@@ -0,0 +1,3 @@
+{
+  ".": "0.5.1"
+}

{redis_lua_py-0.5.0 → redis_lua_py-0.5.1}/CHANGELOG.md

@@ -7,6 +7,13 @@ are written by [release-please](https://github.com/googleapis/release-please)
 from the conventional commit subjects on `main`; see
 [CONTRIBUTING.md](CONTRIBUTING.md) for how a release is cut.
 
+## [0.5.1](https://github.com/IgnaceMaes/redis-lua-py/compare/v0.5.0...v0.5.1) (2026-09-12)
+
+
+### Fixed
+
+* load redis functions libraries on redis-py 4.2 ([#16](https://github.com/IgnaceMaes/redis-lua-py/issues/16)) ([85c2ba2](https://github.com/IgnaceMaes/redis-lua-py/commit/85c2ba20056224af7b2c9720176f799cf54b163f))
+
 ## [0.5.0](https://github.com/IgnaceMaes/redis-lua-py/compare/v0.4.0...v0.5.0) (2026-09-12)
 
 

{redis_lua_py-0.5.0 → redis_lua_py-0.5.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: redis-lua-py
-Version: 0.5.0
+Version: 0.5.1
 Summary: Write Redis Lua scripts as real Python functions, not strings.
 Project-URL: Homepage, https://ignacemaes.com/redis-lua-py/
 Project-URL: Documentation, https://ignacemaes.com/redis-lua-py/

{redis_lua_py-0.5.0 → redis_lua_py-0.5.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "redis-lua-py"
-version = "0.5.0"
+version = "0.5.1"
 description = "Write Redis Lua scripts as real Python functions, not strings."
 readme = "README.md"
 requires-python = ">=3.10"

{redis_lua_py-0.5.0 → redis_lua_py-0.5.1}/src/redis_lua_py/__init__.py

@@ -67,7 +67,7 @@ __all__ = [
     "script",
 ]
 
-__version__ = "0.5.0"  # x-release-please-version
+__version__ = "0.5.1"  # x-release-please-version
 
 
 @overload

{redis_lua_py-0.5.0 → redis_lua_py-0.5.1}/src/redis_lua_py/_library.py

@@ -43,6 +43,23 @@ def _function_missing(error: BaseException) -> bool:
     return "Function not found" in str(error)
 
 
+def _modern_function_load(client: object) -> Any:
+    """The client's function_load, if it takes the library code first.
+
+    Early redis-py releases, 4.2.0 among them, still have the Redis 7
+    release-candidate signature, function_load(engine, library, code), which
+    no released Redis accepts.
+    """
+    function_load = getattr(client, "function_load", None)
+    if function_load is None:
+        return None
+    try:
+        parameters = list(inspect.signature(function_load).parameters)
+    except (TypeError, ValueError):  # a callable without an inspectable signature
+        return None
+    return function_load if parameters[:1] == ["code"] else None
+
+
 class Library:
     """A Redis Functions library, built from Python functions.
 
@@ -130,8 +147,10 @@ class Library:
         is only needed ahead of a pipeline, or to load at deploy time. Returns
         what the client returns: an awaitable for an async client.
         """
-        function_load = getattr(client, "function_load", None)
+        function_load = _modern_function_load(client)
         if function_load is not None:
+            # Preferred where it exists, since redis-py routes it to every
+            # primary of a cluster.
             return function_load(self.lua, replace=True)
         return client.execute_command("FUNCTION", "LOAD", "REPLACE", self.lua)
 

{redis_lua_py-0.5.0 → redis_lua_py-0.5.1}/tests/test_functions.py

@@ -163,6 +163,28 @@ class TestLoadingOnDemand:
 
         assert bump_all(client, keys=["a", "b"]) == ("ran", "FCALL", "bump_all", 2, "a", "b")
 
+    def test_an_old_function_load_signature_is_not_used(self) -> None:
+        """redis-py 4.2.0 has function_load(engine, library, code), from a Redis 7 RC."""
+
+        class OldRedisPy(FakeClient):
+            def execute_command(self, *args: Any) -> Any:
+                if args[:3] == ("FUNCTION", "LOAD", "REPLACE"):
+                    self.calls.append(args[:3])
+                    self.loaded = args[3]
+                    return "rlp_counters"
+                return FakeClient.execute_command(self, *args)
+
+            def function_load(  # type: ignore[override]
+                self, engine: str, library: str, code: str, replace: bool = False
+            ) -> str:
+                raise AssertionError("the release-candidate signature must not be called")
+
+        client = OldRedisPy()
+
+        assert bump(client, key="k", by=2) == ("ran", "FCALL", "bump", 1, "k", "2")
+        assert client.loaded == counters.lua
+        assert ("FUNCTION", "LOAD", "REPLACE") in client.calls
+
     def test_another_error_is_not_swallowed(self) -> None:
         class Failing(FakeClient):
             def execute_command(self, *args: Any) -> Any:

redis_lua_py-0.5.0/.release-please-manifest.json

@@ -1,3 +0,0 @@
-{
-  ".": "0.5.0"
-}