vangberg/rack-session-redis — pull request #2 by jfairbairn (2012-01-02 23:51:05+00:00)
Title: Fixed broken tests in rack-session-redis master

Hi Harry,

There were some assertions wrongly being made that a Set-Cookie header would be returned for a request where a Cookie header had been provided. They were causing the tests to error. I removed them.

Also (at least for Rack 1.3.5 and 1.4), Rack::Session::Redis#destroy_session needed to be implemented to pass the tests (and presumably to exhibit correct behaviour in reality). I did a simple implementation that just passed through to #set_session.

Do these look right to you?

Thanks,
James