feat: Redis.Cluster constructor support

[onichandame]

Problem

I'm using the following practice:

const isUnittest = process.env.NODE_ENV === `unittest`
const Redis = require(isUnittest ? `ioredis-mock` : `ioredis`)

const cluster = new Redis.Cluster([`redis://xxxxx:6379`])
 // more code here

But this lib does not seem to support cluster creation yet.

Solution

The Redis.Cluster constructor is all I need, hence I didn't dig deeply into the internal mechanisms of the redis cluster. I know this is far from complete, so feel free to give advice or simply reject for good :D

Related to #359

[codeclimate]

Code Climate has analyzed commit 80db0e7 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (80% is the threshold).

This pull request will bring the total coverage in the repository to 98.3% (0.0% change).

View more on Code Climate.

[stipsan]

Great starting point! When I started this library I knew little about redis or ioredis, and you could call this whole library a "poor man's ioredis mock" 😂

After applying the review suggestions run these commands, commit and push: npm run lint -- --fix && npx prettier --write . 😄

[onichandame]

It seems that pre-commit or pre-push hooks could help enforce the linting and formating of PRs. http://npmjs.com/package/husky

[stipsan]

@onichandame yup it's set up but it appears to have stopped working after v5 or something 🤔 Haven't had time to look into what's up...

[stipsan]

🎉 This PR is included in version 5.5.0 🎉

The release is available on:

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

Your semantic-release bot 📦🚀