Bug#1014101: python-fakeredis: autopkgtest needs update for new version of redis: error message changed

[Paul Gevers]

Source: python-fakeredis
Version: 1.6.1-1
Severity: serious
X-Debbugs-CC: re...@packages.debian.org
Tags: sid bookworm
User: debian...@lists.debian.org
Usertags: needs-update
Control: affects -1 src:redis

Dear maintainer(s),



                       pass            fail
redis                  from testing    5:7.0.2-2
python-fakeredis       from testing    1.6.1-1
all others             from testing    from testing

I copied some of the output at the bottom of this report.





More information about this bug and the reason for filing it can be found on
https://wiki.debian.org/ContinuousIntegration/RegressionEmailInformation

Paul

[1] https://qa.debian.org/excuses.php?package=redis

https://ci.debian.net/data/autopkgtest/testing/amd64/p/python-fakeredis/23195767/log.gz




r = <Redis <ConnectionsPool [db:0, size:[1:10], free:1]>>

    async def test_failed_script_error(r):
        await r.set('foo', 'bar')

With a recent upload of redis the autopkgtest of python-fakeredis fails in testing when that autopkgtest is run with the binary packages of redis from unstable. It passes when run with only packages from testing. In tabular form: Currently this regression is blocking the migration of redis to testing [1]. Of course, redis shouldn't just break your autopkgtest (or even worse, your package), but it seems to me that the change in redis was intended and your package needs to update to the new situation. It seems the failure is just an error message that got improved. If this is a real problem in your package (and not only in your autopkgtest), the right binary package(s) from redis should really add a versioned Breaks on the unfixed version of (one of your) package(s). Note: the Breaks is nice even if the issue is only in the autopkgtest as it helps the migration software to figure out the right versions to combine in the tests. =================================== FAILURES =================================== ________________________ test_failed_script_error[real] ________________________ with pytest.raises(aioredis.ReplyError, match='^ERR Error running script'):

          await r.eval('return redis.call("ZCOUNT", KEYS[1])', ['foo'])

E aioredis.errors.ReplyError: ERR Wrong number of args calling Redis command from script script: 2558dbbbe1424be2d1e9af6eece6309035f8696f, on @user_script:1.

test/test_aioredis1.py:158: ReplyError

During handling of the above exception, another exception occurred:

r = <Redis <ConnectionsPool [db:0, size:[1:10], free:1]>>

    async def test_failed_script_error(r):
        await r.set('foo', 'bar')

      with pytest.raises(aioredis.ReplyError, match='^ERR Error running 
script'):

E AssertionError: Regex pattern '^ERR Error running script' does not match 'ERR Wrong number of args calling Redis command from script script: 2558dbbbe1424be2d1e9af6eece6309035f8696f, on @user_script:1.'.

test/test_aioredis1.py:157: AssertionError

OpenPGP_signature
Description: OpenPGP digital signature