fix some more

docs/v4-to-v5.md

@@ -109,7 +109,13 @@ Some command arguments/replies have changed to align more closely to data types
 - `GEORADIUSBYMEMBERSTORE` -> `GEORADIUSBYMEMBER_STORE`
 - `XACK`: `boolean` -> `number` [^boolean-to-number]
 - `XADD`: the `INCR` option has been removed, use `XADD_INCR` instead
+- `LASTSAVE`: `Date` -> `number` (unix timestamp)
+- `HELLO`: `protover` moved from the options object to it's own argument, `auth` -> `AUTH`, `clientName` -> `SETNAME`
+- `MODULE LIST`: `version` -> `ver` [^map-keys]
+- `MEMORY STATS`: [^map-keys]
 
 [^enum-to-constants]: TODO
 
 [^boolean-to-number]: TODO
+
+[^map-keys]: [TODO](https://github.com/redis/node-redis/discussions/2506)

packages/client/lib/commands/COPY.spec.ts

@@ -45,7 +45,7 @@ describe('COPY', () => {
   testUtils.testAll('copy', async client => {
     assert.equal(
       await client.copy('{tag}source', '{tag}destination'),
-      false
+      0
     );
   }, {
     client: GLOBAL.SERVERS.OPEN,

packages/client/lib/commands/DBSIZE.spec.ts

@@ -1,19 +1,19 @@
 import { strict as assert } from 'assert';
 import testUtils, { GLOBAL } from '../test-utils';
-import { transformArguments } from './DBSIZE';
+import DBSIZE from './DBSIZE';
 
 describe('DBSIZE', () => {
-    it('transformArguments', () => {
+  it('transformArguments', () => {
-        assert.deepEqual(
+    assert.deepEqual(
-            transformArguments(),
+      DBSIZE.transformArguments(),
-            ['DBSIZE']
+      ['DBSIZE']
-        );
+    );
-    });
+  });
 
-    testUtils.testWithClient('client.dbSize', async client => {
+  testUtils.testWithClient('client.dbSize', async client => {
-        assert.equal(
+    assert.equal(
-            await client.dbSize(),
+      await client.dbSize(),
-            0
+      0
-        );
+    );
-    }, GLOBAL.SERVERS.OPEN);
+  }, GLOBAL.SERVERS.OPEN);
 });

packages/client/lib/commands/EXPIRE.spec.ts

@@ -22,7 +22,7 @@ describe('EXPIRE', () => {
   testUtils.testAll('expire', async client => {
     assert.equal(
       await client.expire('key', 0),
-      false
+      0
     );
   }, {
     client: GLOBAL.SERVERS.OPEN,

packages/client/lib/commands/EXPIREAT.spec.ts

@@ -30,7 +30,7 @@ describe('EXPIREAT', () => {
   testUtils.testAll('expireAt', async client => {
     assert.equal(
       await client.expireAt('key', 1),
-      false
+      0
     );
   }, {
     client: GLOBAL.SERVERS.OPEN,