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.

Sign up for GitHub

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

Jump to bottom

Add unit tests for AbstractClientStream #25

Closed
ejona86 opened this issue Jan 21, 2015 · 2 comments
Closed

Add unit tests for AbstractClientStream #25

ejona86 opened this issue Jan 21, 2015 · 2 comments
Labels
code health P3
Milestone
1.1

Comments

@ejona86
Copy link
Member

ejona86 commented Jan 21, 2015

Some unit tests from NettyClientStream are probably appropriate to move.
@ejona86 ejona86 added this to the 1.0 milestone Dec 1, 2015
@hsaliak hsaliak added the P3 label Apr 26, 2016
@ejona86 ejona86 modified the milestones: 1.1, 1.0 Jun 9, 2016
@ejona86
Copy link
Member Author

ejona86 commented Sep 26, 2016

This will be fixed by #2140

@carl-mastrangelo
Copy link
Contributor

The PR that was mentioned was submitted, can this be closed?

@ejona86 ejona86 closed this as completed Jan 6, 2017
@lock lock bot locked as resolved and limited conversation to collaborators Sep 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
code health P3
Projects
None yet
Milestone
1.1
Development

No branches or pull requests
3 participants
@hsaliak @ejona86 @carl-mastrangelo