Redis.Cluster can't mocked by Jest

[icebob]

Hello,

I struggle with an issue in my unit tests (where Jest mock the ioredis classes) with the latest v4.24.3 version.
In the previous versions the following unit tests pass:

jest.mock("ioredis");

const Redis = require("ioredis");

describe("Test Redis", () => {
	it("create Redis instance", () => {
		const client = new Redis();
		expect(client).toBeInstanceOf(Redis);
		expect(Redis).toBeCalledTimes(1);
	});

	it("create Redis Cluster instance", () => {
		const client = new Redis.Cluster();
		expect(client).toBeInstanceOf(Redis.Cluster);
		expect(Redis.Cluster).toBeCalledTimes(1);
	});
});

Result

But with v4.24.3 I've got errors for the same tests:

Do you have any idea what is the problem and how can I fix it? Thanks in advance.

[luin]

Hi @icebob, what was the ioredis version in the first screenshot?

[icebob]

v4.24.2, the previous version.

[icebob]

In the meantime, I'm trying debugging the issue and I think the problem comes from this changes: bf3fe29#diff-d5a4b3a0309f2337144861084c1014523cced561eb722cb9684c63f5e41e0567
Because in the compiled JS the Cluster changed to a getter and it seems Jest can't traverse the getters.

[ioredis-robot]

🎉 This issue has been resolved in version 4.24.4 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[icebob]

It was quick, thanks @luin, I really appreciate it. Now it works fine.

[tomijais]

I have the same problem with the Redis constructor in the new version