kvrocks2redis test scripts fail about kvrocks HOT 6 CLOSED

I guess the return value OK should also be treat as passed.

from kvrocks.

Thank you! @Zakelly
It seems like some commands return value has some wrong.

These codes were introduced in #1963.

Is it possible that the issue is due to different versions of Python Redis client?

from kvrocks.

@jihuayu
My redis-cli and redis-server version are both 7.2.4, when I run setex command in redis-cli, it returns:

127.0.0.1:6379> setex foo_ex 7200 2
OK

Redis python module version:

pip3 show redis   
Name: redis
Version: 2.10.3
Summary: Python client for Redis key-value store
Home-page: http://github.com/andymccurdy/redis-py
Author: Andy McCurdy
Author-email: [email protected]
License: MIT
Location: /Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages
Requires: 
Required-by: 

from kvrocks.

@Zakelly
Thank you for discovering this issue. The test for kvrocks2redis has not been run in CI, which may be a hidden bug.

After the fix it, do you want to add this test to our CI? refer: #1976

from kvrocks.

@jihuayu Yes I'd like to. Please assign to me, Thanks

from kvrocks.

It seems the README is providing the wrong redis-py version. When I changed to 5.0.3, the problem solved.

from kvrocks.