add spec to validate blank sessions are not stored in redis

redis-store · Sep 26, 2018 · 53493e8 · 53493e8
1 parent e735cea
commit 53493e8
Showing 1 changed file with 6 additions and 0 deletions.
diff --git 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