follow_direct: Include rack.session.options (#233)

* Include session options in request

* Use consistent testing pattern

* Increase allowed ClassLength

* Add more description to the body output

.rubocop_todo.yml

@@ -28,7 +28,7 @@ Metrics/AbcSize:
 # Offense count: 2
 # Configuration parameters: CountComments.
 Metrics/ClassLength:
-  Max: 172
+  Max: 173
 
 # Offense count: 5
 Metrics/CyclomaticComplexity:

lib/rack/test.rb

@@ -197,7 +197,8 @@ def follow_redirect!
         send(
           request_method, last_response['Location'], params,
           'HTTP_REFERER' => last_request.url,
-          'rack.session' => last_request.session
+          'rack.session' => last_request.session,
+          'rack.session.options' => last_request.session_options
         )
       end
 

spec/fixtures/fake_app.rb

@@ -30,7 +30,7 @@ class FakeApp < Sinatra::Base
 
       %i[get put post delete].each do |meth|
         send(meth, '/redirected') do
-          additional_info = meth == :get ? ", session #{session}" : " using #{meth} with #{params}"
+          additional_info = meth == :get ? ", session #{session} with options #{request.session_options}" : " using #{meth} with #{params}"
           "You've been redirected" + additional_info
         end
       end

spec/rack/test_spec.rb

@@ -362,7 +362,7 @@ def close
       follow_redirect!
 
       expect(last_response).not_to be_redirect
-      expect(last_response.body).to eq("You've been redirected, session {}")
+      expect(last_response.body).to eq("You've been redirected, session {} with options {}")
       expect(last_request.env['HTTP_REFERER']).to eql('http://example.org/redirect')
     end
 
@@ -380,6 +380,13 @@ def close
       expect(last_response.body).to include('session {"foo"=>"bar"}')
     end
 
+    it 'includes session options when following the redirect' do
+      get '/redirect', {}, 'rack.session.options' => { 'foo' => 'bar' }
+      follow_redirect!
+
+      expect(last_response.body).to include('session {} with options {"foo"=>"bar"}')
+    end
+
     it 'raises an error if the last_response is not set' do
       expect do
         follow_redirect!