chakrit/sider — pull request #3 by BokononLives (2012-01-17 21:44:14+00:00)
Title: MSetNX should call ReadBool, not ReadOK


The method RedisClient.MSetNX currently calls ReadOK, expecting a SingleLine String reply from the server. This command always fails with an unhandled exception because the server invariably returns an Integer ("1" or "0") for this command. ReadBool parses this response in the intended fashion. I've verified through local tests that the command works correctly after this change. TODO: Perhaps a test suite could be implemented for unit tests of the RedisClient API versus a dummy server (in a private fork, at least, to avoid abuse).