From 53493e81e5acd7331e36a8822c38f910e9d80667 Mon Sep 17 00:00:00 2001 From: mstruve Date: Wed, 26 Sep 2018 13:52:07 -0500 Subject: [PATCH] add spec to validate blank sessions are not stored in redis --- test/rack/session/redis_test.rb | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/test/rack/session/redis_test.rb b/test/rack/session/redis_test.rb index 6fbe928..94a8bfa 100644 --- a/test/rack/session/redis_test.rb +++ b/test/rack/session/redis_test.rb @@ -90,6 +90,12 @@ sesion_store.threadsafe?.must_equal(true) end + it "does not store a blank session" do + session_store = Rack::Session::Redis.new(incrementor) + sid = session_store.generate_unique_sid({}) + session_store.with { |c| c.get(sid).must_be_nil } + end + it "locks the store mutex" do mutex = Mutex.new mutex.expects(:lock).once