fix: rpush error message synced with redis

stipsan · Jan 30, 2022 · c58eda0 · c58eda0
1 parent 8fb0fa4
commit c58eda0
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/jest.config.redis.js b/jest.config.redis.js
@@ -45,7 +45,6 @@ module.exports = {
     'test/integration/commands/publish.js',
     'test/integration/commands/rpop.js',
     'test/integration/commands/rpoplpush.js',
-    'test/integration/commands/rpush.js',
     'test/integration/commands/scan.js',
     'test/integration/commands/scanStream.js',
     'test/integration/commands/scard.js',

diff --git a/src/commands/rpush.js b/src/commands/rpush.js
@@ -1,6 +1,6 @@
 export function rpush(key, ...values) {
   if (this.data.has(key) && !(this.data.get(key) instanceof Array)) {
-    throw new Error(`Key ${key} does not contain a list`)
+    throw new Error('WRONGTYPE Operation against a key holding the wrong kind of value')
   }
   const list = this.data.get(key) || []
   const length = list.push(...values)

diff --git a/test/integration/commands/rpush.js b/test/integration/commands/rpush.js
@@ -26,7 +26,7 @@ runTwinSuite('rpush', command => {
       try {
         await redis[command]('foo', 1)
       } catch (err) {
-        expect(err.message).toBe('Key foo does not contain a list')
+        expect(err.message).toBe('WRONGTYPE Operation against a key holding the wrong kind of value')
       }
     })
   })