search

taoensso / carmine

Redis client + message queue for Clojure
https://www.taoensso.com/carmine
Eclipse Public License 1.0
1.15k stars 130 forks source link

Fix protocol/get-parsed-reply fn #192

Closed firesofmay closed 7 years ago

firesofmay commented 7 years ago

This bug was introduced in commit - a2bef0cc56d9eb4de9703894ead05c7822ad860e

By mistake the arguments to get function were swapped, hence 1 test in taoensso.carmine.tests.main ns have been failing since then.

This fixes it.

Test output:

$ lein clean; lein expectations taoensso.carmine.tests.main

Benching (this can take some time)
----------------------------------

Lap 1/1...
{:wcar 43, :ping 447, :set 394, :get 399, :roundtrip 496, :ping-pipelined 2012}

Done! (Time for cake?)

Ran 118 tests containing 118 assertions in 8643 msecs
0 failures, 0 errors.
ptaoussanis commented 7 years ago

Good catch, thank you Mayank! Will merge this in a batch with the test PR/s later. Cheers!

ptaoussanis commented 7 years ago

Merged, thanks again!