msetArgumentTransformer does not correctly handle arrays

[NeilASega]

There appears to be a bug in the msetArgumentTransformer function:

const msetArgumentTransformer = function (args) {
    if (args.length === 1) {
        if (args[0] instanceof Map) {
            return (0, utils_1.convertMapToArray)(args[0]);
        }
        if (typeof args[0] === "object" && args[0] !== null) {  <-- args[0] might be already be an array.
            return (0, utils_1.convertObjectToArray)(args[0]);
        }
    }
    return args;
};

The following update fixes it:

const msetArgumentTransformer = function (args) {
    if (args.length === 1) {
        if (args[0] instanceof Map) {
            return (0, utils_1.convertMapToArray)(args[0]);
        }
        if (typeof args[0] === "object" && args[0] !== null) {
            if (Array.isArray(args[0])) {
                return args[0];
            }
            return (0, utils_1.convertObjectToArray)(args[0]);
        }
    }
    return args;
};

This problem came to light when using the MSETNX command via "ioredis-mock".

Thanks,
-Neil

[NeilASega]

Can I get some traction on this please? Even though the issue only presents itself in "ioredis-mock", the bug is in "ioredis". The reason why "ioredis" works is because you call args.flat() in your Command.js constructor, otherwise it would fail.