Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pipeline.length and Pipeline._queue.length are unavailable #1046

Closed
venku122 opened this issue Feb 16, 2021 · 1 comment
Closed

Pipeline.length and Pipeline._queue.length are unavailable #1046

venku122 opened this issue Feb 16, 2021 · 1 comment
Labels

Comments

@venku122
Copy link
Contributor

ioredis exposes the number of commands in the pipeline queue via Pipeline.length which is a proxy for Pipeline._queue.length
https://github.com/luin/ioredis/blob/master/lib/pipeline.ts#L57

ioredis-mock does not have a _queue but appears to use this.batch to track its queued commands.
https://github.com/stipsan/ioredis-mock/blob/master/src/pipeline.js#L8
It would be nice to map this.batch.length to Pipeline.length or Pipeline._queue.length so that implementations of ioredis that rely on tracking the size of the queue can be mocked appropriately.

@github-actions
Copy link
Contributor

🎉 This issue has been resolved in version 8.2.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant