weavejester/ring-mock — pull request #3 by aredington (2012-07-27 20:17:32+00:00)
Title: (request :post "url" params) doesn't include the params

While doing some Clojure web development, Tim Ewald and I have had some complications using ring-mock through kerodon. After tracing the problem, we think this patch will resolve our issues. The changes in this request:

- Only feed params into query-string if the verb is get
- Feed params into the body if the verb is post
- Move some fns earlier in the file so request can use them
- Switch body from a multimethod to a protocol

We could remove the protocol impl if that's important, but felt it would be a good overall change and would include it since we were messing with body already. We updated tests to reflect the new return values of request when the verb is post.