Added some new API's to simplify usage#8
Conversation
|
Hi, Thanks for you contribution! Couple things before I'm ready to merge:
Thanks! |
|
Ah, right, I didn't notice that my computer changed line-endings. Will resolve that. The only things I changed in TestRequest was uncomment the "safari" request. I also agree with separating the binary check in, and will do that. Though I'm not entirely sure what you mean by the last point. I figured that the tests should be the same, given that what I wanted to check was that my handler produced the (exact) same output as expected by your library already. The only other way of doing that (as I can think of up front) would be to replicate the functionality of the http-machine inside the test-runner (except you don't actually need to do the parsing, only calling the handler with the appropriate data), however a good reason for leaving the tests as they are is that by doing it the way I am now, I discovered wrongdoings in how I handled the data already. For instance, I didn't know that the OnBody would be called multiple times with parts of the body, so I only saved the ArraySegment given me there initially. However, that ofcause failed a test, and I had to go read your tests to figure out why it was failing. Also, on a completely different sidenote, I'm the author and host of the NuDoc project, which hosts documentation (at the NuDoc Website). I was wondering if you would allow me to provide documentation for this project. This will require the documentation be added as source-code comments, but will at the same time show the documentation in Visual Studio. |
|
There you go. Nice and clean commits. |
Also, I've checked in the "packages" directory to simplify developement for peoples who want
to extend/fork/fix etc. IMHO a cloned directory is expected to work AS IS, (maybe
having to run a "git submodules init"). When I opened the project it
simply would not compile because of missing references.