fixup! fixup! Adapt to redis 3.*

MinoMino · Aug 10, 2024 · 0914d7e · 0914d7e
1 parent 32c483e
commit 0914d7e
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -20,6 +20,6 @@ jobs:
       - name: Start Redis
         uses: superchargejs/[email protected]
       - name: Install redis package
-        run: sudo python3 -m pip install redis==${{ matrix.redis-package-version }}
+        run: PIP_BREAK_SYSTEM_PACKAGES=1 sudo python3 -m pip install redis==${{ matrix.redis-package-version }}
       - name: Run test
         run: python3 -m unittest discover python/tests/
diff --git a/python/minqlx/database.py b/python/minqlx/database.py
@@ -319,8 +319,8 @@ def close(self):
                 Redis._pool = None
 
     if redis.VERSION > (3,):
-        def lrem(self, name, value, count):
-            return self.r.lrem(name, count, value)
+        def lrem(self, name, count, value):
+            return self.r.lrem(name, value, count)
 
         def setex(self, name, value, time):
             return self.r.setex(name, time, value)

diff --git a/python/tests/test_database.py b/python/tests/test_database.py
@@ -70,7 +70,7 @@ def test_lrem_01(self):
             "Dog",
             "Buffalo",
         )
-        c.lrem(key, "Cat", 2)
+        c.lrem(key, 2, "Cat")
         result = c.lrange(key, 0, -1)
         self.assertEqual(
             result,