-
Notifications
You must be signed in to change notification settings - Fork 9
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
Unit tests for the HyperTransport #16
Comments
A bit OT, but: did you consider moving the transport into a separate crate, like a micro framework for API clients? The way it abstracts hyper from the actions is really cool (and I'm going to write quite a few clients in the future, that's why I'm asking 😉). |
@jgillich No, I haven't considered it, but I like the idea. What do you have in mind? The main hurdle I see is that the Chill transport is tailored for CouchDB and doesn't extend to other applications. For example, see the |
I haven't entirely thought this through yet, but revision is a query parameter, right? You could assume standard HTTP types like query parameters and headers, which the transport could then use those to and construct a HTTP/WS/HTCPCP request or parse them right away for testing. But I have to think about that for while, there probably is a better way. |
@jgillich I'm on board with the general idea. Any code in Chill that isn't fundamental to CouchDB is better off existing in another crate. However, I've got higher priorities and don't foresee working on this anytime soon. Feel free to take ownership of the idea and make it happen! |
The
HyperTransport
is tested indirectly, via the integration tests. Consider writing unit tests for it to ensure all cases are covered.The text was updated successfully, but these errors were encountered: