-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add more tests for the client #20
Comments
vykulakov
pushed a commit
that referenced
this issue
Dec 29, 2019
There are plans to use the test server in other tests of the client so it will be very useful to have it in a separate package.
vykulakov
added a commit
that referenced
this issue
Dec 29, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It will be great to have more client tests that allow going through all client levels together.
Actually they will be some kind of integration tests and should allow avoiding using tests with a real Apache Flume server. So testing the whole project will be faster, simpler and more robust finally.
The text was updated successfully, but these errors were encountered: