Inconsistent {DECR}: Error message for out-of-range OR non-number values differs from Redis error message

[mghildiy]

Steps to reproduce

SET mykey "234293482390480948029348230948"
DECR mykey
SET mykey "Hello"
DECR mykey

Expected output

The expected output when the above set of commands when run on Redis

(error) value is not an integer or out of range

Observed output

The observed output when the above set of commands when run on DiceDB

(error) ERROR WRONGTYPE Operation against a key holding the wrong kind of value

The steps to run the test cases are mentioned in the README of the dice-tests repository.

Expectations for resolution

This issue will be considered resolved when the following things are done

1. changes in the dice code to meet the expected behavior
2. Successful run of the tcl test behavior

You can find the tests under the tests directory of the dice repository and the steps to run are in the README file. Refer to the following links to set up DiceDB and Redis 7.2.5 locally

• setup DiceDB locally
• setup Redis 7.2.5 locally
• setup DiceDB CLI

[mghildiy]

This issue has been raised based on findings for issue.

[ashutoshdhande]

@mghildiy , are your expected and observed cases referring to the same scenario? I ran the same scenario as you, but encountered the same error for both (which is expected).

[mghildiy]

@mghildiy , are your expected and observed cases referring to the same scenario? I ran the same scenario as you, but encountered the same error for both (which is expected).

Error message needs to be same as redis.

[adityaa-p]

@mghildiy The current error message is the same as redis only. Can you please recheck?

[mghildiy]

For Redis:

For dice:

I am running latest docker versions for both.

[apoorvyadav1111]

Hi @mghildiy , I tried running latest dice db code using go and I am getting following:

[mghildiy]

Master branch exhibits same bheaviour as redis while my findings were based on latest dokcer image.
I am closing this ticket.