-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Lack of MockJoinGroupResponse in mockresponses.go #1577
Comments
I have supplement some kind of mock responses to make group consumer work, but not enough to submit a commit. Will try and continue working on it. |
@liuxin-reg any luck with this? Happy to help out if you want. |
I submitted a PR with a few mocks, but I think consumer group mocking will take a bit more than just that. |
@carldunham Thanks for your response and PR. I had add some basic mocks for consumer group responses too, it can works now, but without enough detailed considerations contained. Now, I removed this modification from folder vendor, since I don't think this kind of mock is necessary for normal consumer client. |
Any updates on this one @carldunham @liuxin-reg ? |
Thank you for taking the time to raise this issue. However, it has not had any activity on it in the past 90 days and will be closed in 30 days if no updates occur. |
Fixed by #1750 |
master
I haven't found struct MockJoinGroupResponse in mockresponses.go, will we support that?
The text was updated successfully, but these errors were encountered: