1 files changed, 51 insertions, 8 deletions
diff --git a/actionpack/test/dispatch/request/session_test.rb b/actionpack/test/dispatch/request/session_test.rb
index 3f36d4f1a9..10fb04e230 100644
--- a/actionpack/test/dispatch/request/session_test.rb
+++ b/actionpack/test/dispatch/request/session_test.rb
@@ -24,7 +24,7 @@ module ActionDispatch
         s['foo'] = 'bar'
 
         s1 = Session.create(store, env, {})
-        refute_equal s, s1
+        assert_not_equal s, s1
         assert_equal 'bar', s1['foo']
       end
 
@@ -36,29 +36,71 @@ module ActionDispatch
         assert_equal s, Session.find(env)
       end
 
+      def test_destroy
+        s = Session.create(store, {}, {})
+        s['rails'] = 'ftw'
+
+        s.destroy
+
+        assert_empty s
+      end
+
       def test_keys
-        env = {}
-        s = Session.create(store, env, {})
+        s = Session.create(store, {}, {})
         s['rails'] = 'ftw'
         s['adequate'] = 'awesome'
         assert_equal %w[rails adequate], s.keys
       end
 
       def test_values
-        env = {}
-        s = Session.create(store, env, {})
+        s = Session.create(store, {}, {})
         s['rails'] = 'ftw'
         s['adequate'] = 'awesome'
         assert_equal %w[ftw awesome], s.values
       end
 
       def test_clear
-        env = {}
-        s = Session.create(store, env, {})
+        s = Session.create(store, {}, {})
         s['rails'] = 'ftw'
         s['adequate'] = 'awesome'
+
         s.clear
-        assert_equal([], s.values)
+        assert_empty(s.values)
+      end
+
+      def test_update
+        s = Session.create(store, {}, {})
+        s['rails'] = 'ftw'
+
+        s.update(:rails => 'awesome')
+
+        assert_equal(['rails'], s.keys)
+        assert_equal('awesome', s['rails'])
+      end
+
+      def test_delete
+        s = Session.create(store, {}, {})
+        s['rails'] = 'ftw'
+
+        s.delete('rails')
+
+        assert_empty(s.keys)
+      end
+
+      def test_fetch
+        session = Session.create(store, {}, {})
+
+        session['one'] = '1'
+        assert_equal '1', session.fetch(:one)
+
+        assert_equal '2', session.fetch(:two, '2')
+        assert_nil session.fetch(:two, nil)
+
+        assert_equal 'three', session.fetch(:three) {|el| el.to_s }
+
+        assert_raise KeyError do
+          session.fetch(:three)
+        end
       end
 
       private
@@ -66,6 +108,7 @@ module ActionDispatch
         Class.new {
           def load_session(env); [1, {}]; end
           def session_exists?(env); true; end
+          def destroy_session(env, id, options); 123; end
         }.new
       end
     end