redis-actionpack 3.1.3.rc2 → 3.1.3.rc3

data/Gemfile

@@ -1,5 +1,8 @@
 source "http://rubygems.org"
 gemspec
 
-gem 'redis-store',      '1.1.0.rc', :path => File.expand_path('../../redis-store',      __FILE__)
+gem 'redis-store',      '1.1.0.rc2', :path => File.expand_path('../../redis-store',      __FILE__)
 gem 'redis-rack-cache', '1.1.rc2',  :path => File.expand_path('../../redis-rack-cache', __FILE__)
+gem 'redis-rack',       '1.3.5.rc', :path => File.expand_path('../../redis-rack', __FILE__)
+gem 'SystemTimer',      :platform => :mri_18
+gem 'minitest_tu_shim', :platform => :ruby_18

data/Rakefile

@@ -3,12 +3,6 @@ Bundler.setup
 require 'rake'
 require 'bundler/gem_tasks'
 
-begin
-  require 'rdoc/task'
-rescue LoadError
-  require 'rake/rdoctask'
-end
-
 load 'tasks/redis.tasks.rb'
 task :default => 'redis:test:suite'
 

data/lib/action_dispatch/middleware/session/redis_store.rb

@@ -1,64 +1,17 @@
 require 'redis-store'
+require 'redis-rack'
 require 'action_dispatch/middleware/session/abstract_store'
 
-# Redis session storage for Rails, and for Rails only. Derived from
-# the MemCacheStore code, simply dropping in Redis instead.
-#
-# Options:
-#  :key          => Same as with the other cookie stores, key name
-#  :secret       => Encryption secret for the key
-#  :host         => Redis host name, default is localhost
-#  :port         => Redis port, default is 6379
-#  :db           => Database number, defaults to 0. Useful to separate your session storage from other data
-#  :key_prefix   => Prefix for keys used in Redis, e.g. myapp-. Useful to separate session storage keys visibly from others
-#  :expire_after => A number in seconds to set the timeout interval for the session. Will map directly to expiry in Redis
 module ActionDispatch
   module Session
-    class RedisSessionStore < AbstractStore
+    class RedisStore < Rack::Session::Redis
+      include Compatibility
+      include StaleSessionCheck
       def initialize(app, options = {})
-        # Support old :expires option
-        options[:expire_after] ||= options[:expires]
-
-        super
-
         options = options.dup
-        servers = [ options.delete(:servers) ].flatten.compact
-        servers = [ "redis://localhost:6379/0" ] if servers.empty?
-        servers.map! do |server|
-          server = Redis::Factory.resolve(server)
-          server.merge(options)
-        end
-
-        @pool = Redis::Factory.create(*servers)
+        options[:redis_server] ||= options[:servers]
+        super
       end
-
-      private
-        def get_session(env, sid)
-          sid ||= generate_sid
-          begin
-            session = @pool.get(sid) || {}
-          rescue Errno::ECONNREFUSED
-            session = {}
-          end
-          [sid, session]
-        end
-
-        def set_session(env, sid, session_data, opts=nil)
-          options = env['rack.session.options']
-          @pool.set(sid, session_data, options)
-          sid
-        rescue Errno::ECONNREFUSED
-          return false
-        end
-
-        def destroy(env)
-          if sid = current_session_id(env)
-            @pool.del(sid)
-            sid
-          end
-        rescue Errno::ECONNREFUSED
-          false
-        end
     end
   end
-end
+end

data/lib/redis/actionpack/version.rb

@@ -1,5 +1,5 @@
 class Redis
   module ActionPack
-    VERSION = '3.1.3.rc2'
+    VERSION = '3.1.3.rc3'
   end
 end

data/redis-actionpack.gemspec

@@ -18,14 +18,15 @@ Gem::Specification.new do |s|
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
 
-  s.add_runtime_dependency 'redis-store',      '1.1.0.rc'
+  s.add_runtime_dependency 'redis-store',      '1.1.0.rc2'
   s.add_runtime_dependency 'actionpack',       '3.1.3'
   s.add_runtime_dependency 'redis-rack-cache', '1.1.rc2'
+  s.add_runtime_dependency 'redis-rack',       '1.3.5.rc'
 
-  s.add_development_dependency 'rake',      '~> 0.9.2.2'
-  s.add_development_dependency 'bundler',   '~> 1.1.rc'
-  s.add_development_dependency 'mocha',     '~> 0.10.0'
-  s.add_development_dependency 'minitest',  '~> 2.8.0'
-  s.add_development_dependency 'purdytest', '~> 1.0.0'
+  s.add_development_dependency 'rake',           '~> 0.9.2.2'
+  s.add_development_dependency 'bundler',        '~> 1.1.rc'
+  s.add_development_dependency 'minitest',       '~> 2.8.0'
+  s.add_development_dependency 'purdytest',      '~> 1.0.0'
+  s.add_development_dependency 'minitest-rails', '~> 0.0.5'
+  s.add_development_dependency 'tzinfo'
 end
-

data/test/dummy/.gitignore

@@ -0,0 +1 @@
+log/*

data/test/dummy/Rakefile

@@ -0,0 +1,7 @@
+#!/usr/bin/env rake
+# Add your own tasks in files placed in lib/tasks ending in .rake,
+# for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
+
+require File.expand_path('../config/application', __FILE__)
+
+Dummy::Application.load_tasks

data/test/dummy/app/controllers/test_controller.rb

@@ -0,0 +1,37 @@
+class TestController < ActionController::Base
+  def no_session_access
+    head :ok
+  end
+
+  def set_session_value
+    session[:foo] = "bar"
+    head :ok
+  end
+
+  def set_session_value_with_expiry
+    request.session_options[:expire_after] = 1.second
+    set_session_value
+  end
+
+  def set_serialized_session_value
+    session[:foo] = SessionAutoloadTest::Foo.new
+    head :ok
+  end
+
+  def get_session_value
+    render :text => "foo: #{session[:foo].inspect}"
+  end
+
+  def get_session_id
+    render :text => "#{request.session_options[:id]}"
+  end
+
+  def call_reset_session
+    session[:bar]
+    reset_session
+    session[:bar] = "baz"
+    head :ok
+  end
+
+  def rescue_action(e) raise end
+end

data/test/dummy/config.ru

@@ -0,0 +1,4 @@
+# This file is used by Rack-based servers to start the application.
+
+require ::File.expand_path('../config/environment',  __FILE__)
+run Dummy::Application

data/test/dummy/config/application.rb

@@ -0,0 +1,11 @@
+require File.expand_path('../boot', __FILE__)
+
+require "action_controller/railtie"
+
+Bundler.require
+
+module Dummy
+  class Application < Rails::Application
+  end
+end
+

data/test/dummy/config/boot.rb

@@ -0,0 +1,10 @@
+require 'rubygems'
+gemfile = File.expand_path('../../../../Gemfile', __FILE__)
+
+if File.exist?(gemfile)
+  ENV['BUNDLE_GEMFILE'] = gemfile
+  require 'bundler'
+  Bundler.setup
+end
+
+$:.unshift File.expand_path('../../../../lib', __FILE__)

data/test/dummy/config/environment.rb

@@ -0,0 +1,5 @@
+# Load the rails application
+require File.expand_path('../application', __FILE__)
+
+# Initialize the rails application
+Dummy::Application.initialize!

data/test/dummy/config/environments/test.rb

@@ -0,0 +1,25 @@
+Dummy::Application.configure do
+  # Settings specified here will take precedence over those in config/application.rb
+
+  # The test environment is used exclusively to run your application's
+  # test suite.  You never need to work with it otherwise.  Remember that
+  # your test database is "scratch space" for the test suite and is wiped
+  # and recreated between test runs.  Don't rely on the data there!
+  config.cache_classes = false
+
+  # Log error messages when you accidentally call methods on nil
+  config.whiny_nils = true
+
+  # Show full error reports and disable caching
+  config.consider_all_requests_local       = true
+  config.action_controller.perform_caching = false
+
+  # Raise exceptions instead of rendering exception templates
+  config.action_dispatch.show_exceptions = false
+
+  # Disable request forgery protection in test environment
+  config.action_controller.allow_forgery_protection    = false
+
+  # Print deprecation notices to the stderr
+  config.active_support.deprecation = :stderr
+end

data/test/dummy/config/initializers/secret_token.rb

@@ -0,0 +1,7 @@
+# Be sure to restart your server when you modify this file.
+
+# Your secret key for verifying the integrity of signed cookies.
+# If you change this key, all old signed cookies will become invalid!
+# Make sure the secret is at least 30 characters and all random,
+# no regular words or you'll be exposed to dictionary attacks.
+Dummy::Application.config.secret_token = 'a7d2ff143cdb3d1f9470a3ce2df7bb220a9c4498cb5c4a35150705de6719114b12ab7512a9618a2d3c25f8d6e62ab22c042445ed856ff674f4ee2faabd9d2041'

data/test/dummy/config/initializers/session_store.rb

@@ -0,0 +1,11 @@
+# Be sure to restart your server when you modify this file.
+
+Dummy::Application.config.session_store :redis_store,
+  :key => '_session_id',
+  :servers => ["redis://127.0.0.1:6380/1/theplaylist",
+    "redis://127.0.0.1:6381/1/theplaylist"]
+
+# Use the database for sessions instead of the cookie-based default,
+# which shouldn't be used to store highly confidential information
+# (create the session table with "rails generate session_migration")
+# Dummy::Application.config.session_store :active_record_store

data/test/dummy/config/routes.rb

@@ -0,0 +1,3 @@
+Dummy::Application.routes.draw do
+  match ':action', :to => TestController
+end

data/test/dummy/script/rails

@@ -0,0 +1,6 @@
+#!/usr/bin/env ruby
+# This command will automatically be run when you run "rails" with Rails 3 gems installed from the root of your application.
+
+APP_PATH = File.expand_path('../../config/application',  __FILE__)
+require File.expand_path('../../config/boot',  __FILE__)
+require 'rails/commands'

data/test/fixtures/session_autoload_test/session_autoload_test/foo.rb

@@ -0,0 +1,10 @@
+module SessionAutoloadTest
+  class Foo
+    def initialize(bar='baz')
+      @bar = bar
+    end
+    def inspect
+      "#<#{self.class} bar:#{@bar.inspect}>"
+    end
+  end
+end

data/test/integration/redis_store_integration_test.rb

@@ -0,0 +1,130 @@
+require 'test_helper'
+
+class RedisStoreIntegrationTest < MiniTest::Rails::IntegrationTest
+  it "reads the data" do
+    get '/set_session_value'
+    response.must_be :success?
+    cookies['_session_id'].wont_be_nil
+
+    get '/get_session_value'
+    response.must_be :success?
+    response.body.must_equal 'foo: "bar"'
+  end
+
+  it "should get nil session value" do
+    get '/get_session_value'
+    response.must_be :success?
+    response.body.must_equal 'foo: nil'
+  end
+
+  it "should delete the data after session reset" do
+    get '/set_session_value'
+    response.must_be :success?
+    cookies['_session_id'].wont_be_nil
+    session_cookie = cookies.send(:hash_for)['_session_id']
+
+    get '/call_reset_session'
+    response.must_be :success?
+    headers['Set-Cookie'].wont_equal []
+
+    cookies << session_cookie
+
+    get '/get_session_value'
+    response.must_be :success?
+    response.body.must_equal 'foo: nil'
+  end
+
+  it "should not send cookies on write, not read" do
+    get '/get_session_value'
+    response.must_be :success?
+    response.body.must_equal 'foo: nil'
+    cookies['_session_id'].must_be_nil
+  end
+
+  it "should set session value after session reset" do
+    get '/set_session_value'
+    response.must_be :success?
+    cookies['_session_id'].wont_be_nil
+    session_id = cookies['_session_id']
+
+    get '/call_reset_session'
+    response.must_be :success?
+    headers['Set-Cookie'].wont_equal []
+
+    get '/get_session_value'
+    response.must_be :success?
+    response.body.must_equal 'foo: nil'
+
+    get '/get_session_id'
+    response.must_be :success?
+    response.body.wont_equal session_id
+  end
+
+  it "should be able to read session id without accessing the session hash" do
+    get '/set_session_value'
+    response.must_be :success?
+    cookies['_session_id'].wont_be_nil
+    session_id = cookies['_session_id']
+
+    get '/get_session_id'
+    response.must_be :success?
+    response.body.must_equal session_id
+  end
+
+  it "should auto-load unloaded class" do
+    with_autoload_path "session_autoload_test" do
+      get '/set_serialized_session_value'
+      response.must_be :success?
+      cookies['_session_id'].wont_be_nil
+    end
+
+    with_autoload_path "session_autoload_test" do
+      get '/get_session_id'
+      assert_response :success
+    end
+
+    with_autoload_path "session_autoload_test" do
+      get '/get_session_value'
+      response.must_be :success?
+      response.body.must_equal 'foo: #<SessionAutoloadTest::Foo bar:"baz">'
+    end
+  end
+
+  it "should not resend the cookie again if session_id cookie is already exists" do
+    get '/set_session_value'
+    response.must_be :success?
+    cookies['_session_id'].wont_be_nil
+
+    get '/get_session_value'
+    response.must_be :success?
+    headers['Set-Cookie'].must_be_nil
+  end
+
+  it "should prevent session fixation" do
+    get '/get_session_value'
+    response.must_be :success?
+    response.body.must_equal 'foo: nil'
+    session_id = cookies['_session_id']
+
+    reset!
+
+    get '/set_session_value', :_session_id => session_id
+    response.must_be :success?
+    cookies['_session_id'].wont_equal session_id
+  end
+
+  it "should write the data with expiration time" do
+    get '/set_session_value_with_expiry'
+    response.must_be :success?
+
+    get '/get_session_value'
+    response.must_be :success?
+    response.body.must_equal 'foo: "bar"'
+
+    sleep 1
+
+    get '/get_session_value'
+    response.must_be :success?
+    response.body.must_equal 'foo: nil'
+  end
+end

data/test/redis/actionpack/version_test.rb

@@ -1,7 +1,7 @@
 require 'test_helper'
 
 describe Redis::ActionPack::VERSION do
-  it "must be equal to 3.1.3.rc2" do
-    Redis::ActionPack::VERSION.must_equal '3.1.3.rc2'
+  it "must be equal to 3.1.3.rc3" do
+    Redis::ActionPack::VERSION.must_equal '3.1.3.rc3'
   end
 end

data/test/test_helper.rb

@@ -1,8 +1,47 @@
-Bundler.setup
-gem 'minitest'
-require 'minitest/spec'
 require 'minitest/autorun'
-require 'mocha'
+require 'minitest/rails'
 require 'active_support/core_ext/numeric/time'
-require 'action_dispatch'
-require 'action_dispatch/middleware/session/redis_store'
+
+ENV["RAILS_ENV"] = "test"
+require File.expand_path("../dummy/config/environment.rb",  __FILE__)
+require "rails/test_help"
+
+Rails.backtrace_cleaner.remove_silencers!
+
+# TODO: remove once https://github.com/blowmage/minitest-rails/issues/12 is resolved
+class MiniTest::Rails::IntegrationTest < MiniTest::Rails::Spec
+  include ActiveSupport::Testing::SetupAndTeardown
+
+  include ActionDispatch::Integration::Runner
+
+  @@app = nil
+
+  def self.app
+    # DEPRECATE Rails application fallback
+    # This should be set by the initializer
+    @@app || (defined?(Rails.application) && Rails.application) || nil
+  end
+
+  def self.app=(app)
+    @@app = app
+  end
+
+  def app
+    super || self.class.app
+  end
+end
+
+def with_autoload_path(path)
+  path = File.join(File.dirname(__FILE__), "fixtures", path)
+  if ActiveSupport::Dependencies.autoload_paths.include?(path)
+    yield
+  else
+    begin
+      ActiveSupport::Dependencies.autoload_paths << path
+      yield
+    ensure
+      ActiveSupport::Dependencies.autoload_paths.reject! {|p| p == path}
+      ActiveSupport::Dependencies.clear
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: redis-actionpack
 version: !ruby/object:Gem::Version
-  version: 3.1.3.rc2
+  version: 3.1.3.rc3
   prerelease: 6
 platform: ruby
 authors:
@@ -9,22 +9,22 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-01-20 00:00:00.000000000 Z
+date: 2012-02-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis-store
-  requirement: &70289499148240 !ruby/object:Gem::Requirement
+  requirement: &70362130694300 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - =
       - !ruby/object:Gem::Version
-        version: 1.1.0.rc
+        version: 1.1.0.rc2
   type: :runtime
   prerelease: false
-  version_requirements: *70289499148240
+  version_requirements: *70362130694300
 - !ruby/object:Gem::Dependency
   name: actionpack
-  requirement: &70289499147740 !ruby/object:Gem::Requirement
+  requirement: &70362130693480 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - =
@@ -32,10 +32,10 @@ dependencies:
         version: 3.1.3
   type: :runtime
   prerelease: false
-  version_requirements: *70289499147740
+  version_requirements: *70362130693480
 - !ruby/object:Gem::Dependency
   name: redis-rack-cache
-  requirement: &70289499147280 !ruby/object:Gem::Requirement
+  requirement: &70362130691920 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - =
@@ -43,10 +43,21 @@ dependencies:
         version: 1.1.rc2
   type: :runtime
   prerelease: false
-  version_requirements: *70289499147280
+  version_requirements: *70362130691920
+- !ruby/object:Gem::Dependency
+  name: redis-rack
+  requirement: &70362130691260 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - =
+      - !ruby/object:Gem::Version
+        version: 1.3.5.rc
+  type: :runtime
+  prerelease: false
+  version_requirements: *70362130691260
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70289499146820 !ruby/object:Gem::Requirement
+  requirement: &70362130690640 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -54,10 +65,10 @@ dependencies:
         version: 0.9.2.2
   type: :development
   prerelease: false
-  version_requirements: *70289499146820
+  version_requirements: *70362130690640
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &70289492991060 !ruby/object:Gem::Requirement
+  requirement: &70362130689740 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -65,40 +76,51 @@ dependencies:
         version: 1.1.rc
   type: :development
   prerelease: false
-  version_requirements: *70289492991060
+  version_requirements: *70362130689740
 - !ruby/object:Gem::Dependency
-  name: mocha
-  requirement: &70289492989200 !ruby/object:Gem::Requirement
+  name: minitest
+  requirement: &70362130688860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.10.0
+        version: 2.8.0
   type: :development
   prerelease: false
-  version_requirements: *70289492989200
+  version_requirements: *70362130688860
 - !ruby/object:Gem::Dependency
-  name: minitest
-  requirement: &70289492986120 !ruby/object:Gem::Requirement
+  name: purdytest
+  requirement: &70362130687820 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 2.8.0
+        version: 1.0.0
   type: :development
   prerelease: false
-  version_requirements: *70289492986120
+  version_requirements: *70362130687820
 - !ruby/object:Gem::Dependency
-  name: purdytest
-  requirement: &70289492984040 !ruby/object:Gem::Requirement
+  name: minitest-rails
+  requirement: &70362130686860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 0.0.5
+  type: :development
+  prerelease: false
+  version_requirements: *70362130686860
+- !ruby/object:Gem::Dependency
+  name: tzinfo
+  requirement: &70362130686180 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70289492984040
+  version_requirements: *70362130686180
 description: Redis session store for ActionPack
 email:
 - guidi.luca@gmail.com
@@ -115,7 +137,20 @@ files:
 - lib/redis-actionpack.rb
 - lib/redis/actionpack/version.rb
 - redis-actionpack.gemspec
-- test/action_dispatch/middleware/session/redis_store_test.rb
+- test/dummy/.gitignore
+- test/dummy/Rakefile
+- test/dummy/app/controllers/test_controller.rb
+- test/dummy/config.ru
+- test/dummy/config/application.rb
+- test/dummy/config/boot.rb
+- test/dummy/config/environment.rb
+- test/dummy/config/environments/test.rb
+- test/dummy/config/initializers/secret_token.rb
+- test/dummy/config/initializers/session_store.rb
+- test/dummy/config/routes.rb
+- test/dummy/script/rails
+- test/fixtures/session_autoload_test/session_autoload_test/foo.rb
+- test/integration/redis_store_integration_test.rb
 - test/redis/actionpack/version_test.rb
 - test/test_helper.rb
 homepage: http://jodosha.github.com/redis-store
@@ -143,6 +178,19 @@ signing_key:
 specification_version: 3
 summary: Redis session store for ActionPack
 test_files:
-- test/action_dispatch/middleware/session/redis_store_test.rb
+- test/dummy/.gitignore
+- test/dummy/Rakefile
+- test/dummy/app/controllers/test_controller.rb
+- test/dummy/config.ru
+- test/dummy/config/application.rb
+- test/dummy/config/boot.rb
+- test/dummy/config/environment.rb
+- test/dummy/config/environments/test.rb
+- test/dummy/config/initializers/secret_token.rb
+- test/dummy/config/initializers/session_store.rb
+- test/dummy/config/routes.rb
+- test/dummy/script/rails
+- test/fixtures/session_autoload_test/session_autoload_test/foo.rb
+- test/integration/redis_store_integration_test.rb
 - test/redis/actionpack/version_test.rb
 - test/test_helper.rb

data/test/action_dispatch/middleware/session/redis_store_test.rb

@@ -1,77 +0,0 @@
-require 'test_helper'
-
-describe ActionDispatch::Session::RedisSessionStore do
-  attr_reader :app
-
-  before do
-    @app = Object.new
-    @store  = ActionDispatch::Session::RedisSessionStore.new(app)
-    @dstore = ActionDispatch::Session::RedisSessionStore.new app, :servers => ["redis://127.0.0.1:6380/1", "redis://127.0.0.1:6381/1"]
-    @rabbit = OpenStruct.new :name => "bunny"
-    @white_rabbit = OpenStruct.new :color => "white"
-    @sid = "rabbit"
-    @env = {'rack.session.options' => {:id => @sid}}
-    with_store_management do |store|
-      class << store
-        attr_reader :pool
-        public :get_session, :set_session, :destroy
-      end
-      store.set_session(@env, @sid, @rabbit)
-      store.pool.del "counter"
-      store.pool.del "rub-a-dub"
-    end
-  end
-
-  it "reads the data" do
-    with_store_management do |store|
-      store.get_session(@env, @sid).must_equal([@sid, @rabbit])
-    end
-  end
-
-  it "should write the data" do
-    with_store_management do |store|
-      store.set_session(@env, @sid, @white_rabbit)
-      store.get_session(@env, @sid).must_equal([@sid, @white_rabbit])
-    end
-  end
-
-  it "should delete the data" do
-    with_store_management do |store|
-      store.destroy(@env)
-      store.get_session(@env, @sid).must_equal([@sid, {}])
-    end
-  end
-
-  it "should write the data with expiration time" do
-    with_store_management do |store|
-      @env['rack.session.options'].merge!(:expires_in => 1.second)
-      store.set_session(@env, @sid, @white_rabbit); sleep 2
-      store.get_session(@env, @sid).must_equal([@sid, {}])
-    end
-  end
-
-  describe "namespace" do
-    before do
-      @namespace = "theplaylist"
-      @store  = ActionDispatch::Session::RedisSessionStore.new(lambda {|| }, :servers => [{ :namespace => @namespace }])
-      @pool   = @store.instance_variable_get(:@pool)
-      @client = @pool.instance_variable_get(:@client)
-    end
-
-    it "should read the data" do
-      @client.expects(:call).with([:get, "#{@namespace}:#{@sid}"])
-      @store.send :get_session, @env, @sid
-    end
-
-    it "should write the data" do
-      @client.expects(:call).with([:set, "#{@namespace}:#{@sid}", Marshal.dump(@white_rabbit)])
-      @store.send :set_session, @env, @sid, @white_rabbit
-    end
-  end
-
-  private
-    def with_store_management
-      yield @store
-      yield @dstore
-    end
-end