Favor RSpec.describe over describe.

Add `.rspec` file and remove require statements that are no longer
needed.

.rspec

@@ -0,0 +1,3 @@
+--require spec_helper
+--format documentation
+--color

spec/warden/authenticated_data_store_spec.rb

@@ -1,8 +1,6 @@
 # encoding: utf-8
 # frozen_string_literal: true
-require 'spec_helper'
-
-describe "authenticated data store" do
+RSpec.describe "authenticated data store" do
 
   before(:each) do
     @env = env_with_params

spec/warden/config_spec.rb

@@ -1,8 +1,6 @@
 # encoding: utf-8
 # frozen_string_literal: true
-require 'spec_helper'
-
-describe Warden::Config do
+RSpec.describe Warden::Config do
 
   before(:each) do
     @config = Warden::Config.new

spec/warden/errors_spec.rb

@@ -1,8 +1,6 @@
 # encoding: utf-8
 # frozen_string_literal: true
-require 'spec_helper'
-
-describe Warden::Proxy::Errors do
+RSpec.describe Warden::Proxy::Errors do
 
   before(:each) do
     @errors = Warden::Proxy::Errors.new

spec/warden/hooks_spec.rb

@@ -1,8 +1,6 @@
 # encoding: utf-8
 # frozen_string_literal: true
-require 'spec_helper'
-
-describe "standard authentication hooks" do
+RSpec.describe "standard authentication hooks" do
 
   before(:all) do
     load_strategies

spec/warden/manager_spec.rb

@@ -1,8 +1,6 @@
 # encoding: utf-8
 # frozen_string_literal: true
-require 'spec_helper'
-
-describe Warden::Manager do
+RSpec.describe Warden::Manager do
 
   before(:all) do
     load_strategies

spec/warden/proxy_spec.rb

@@ -1,14 +1,12 @@
 # encoding: utf-8
 # frozen_string_literal: true
-require 'spec_helper'
-
-describe Warden::Proxy do
+RSpec.describe Warden::Proxy do
   before(:all) do
     load_strategies
   end
 
   before(:each) do
-    @basic_app = lambda{|env| [200,{'Content-Type' => 'text/plain'},'OK']}
+    @basic_app = lambda{|_env| [200,{'Content-Type' => 'text/plain'},'OK']}
     @authd_app = lambda do |e|
       e['warden'].authenticate
       if e['warden'].authenticated?
@@ -48,21 +46,21 @@
 
     it "should allow authentication in my application" do
       env = env_with_params('/', :username => "fred", :password => "sekrit")
-      app = lambda do |_env|
-        _env['warden'].authenticate
-        expect(_env['warden']).to be_authenticated
-        expect(_env['warden.spec.strategies']).to eq([:password])
+      app = lambda do |env|
+        env['warden'].authenticate
+        expect(env['warden']).to be_authenticated
+        expect(env['warden.spec.strategies']).to eq([:password])
         valid_response
       end
       setup_rack(app).call(env)
     end
 
     it "should allow me to select which strategies I use in my application" do
       env = env_with_params("/", :foo => "bar")
-      app = lambda do |_env|
-        _env['warden'].authenticate(:failz)
-        expect(_env['warden']).not_to be_authenticated
-        expect(_env['warden.spec.strategies']).to eq([:failz])
+      app = lambda do |env|
+        env['warden'].authenticate(:failz)
+        expect(env['warden']).not_to be_authenticated
+        expect(env['warden.spec.strategies']).to eq([:failz])
         valid_response
       end
       setup_rack(app).call(env)
@@ -730,7 +728,7 @@
 
       it "should return false if scope cannot be retrieved from session" do
         begin
-          Warden::Manager.serialize_from_session { |k| nil }
+          Warden::Manager.serialize_from_session { |_k| nil }
           app = lambda do |env|
             env['rack.session']['warden.user.foo_scope.key'] = "a foo user"
             env['warden'].authenticated?(:foo_scope)

spec/warden/scoped_session_serializer.rb

@@ -1,8 +1,6 @@
 # encoding: utf-8
 # frozen_string_literal: true
-require 'spec_helper'
-
-describe Warden::Manager do
+RSpec.describe Warden::Manager do
   before(:each) do
     @env = env_with_params
     @env['rack.session'] ||= {}
@@ -17,6 +15,7 @@
     rescue
     end
   end
+
   after(:each) do
     Warden::Manager.serialize_from_session { |k| k }
     Warden::Manager.serialize_into_session { |u| u }
@@ -40,14 +39,14 @@ def serializer_respond_to?(name)
   end
 
   it "should respond to {scope}_deserialize if Manager.serialize_from_session is called with scope" do
-    Rack::Builder.new do 
-      Warden::Manager.serialize_from_session ( :admin ) { |n| n }
+    Rack::Builder.new do
+      Warden::Manager.serialize_from_session(:admin) { |n| n }
     end
     serializer_respond_to?(:admin_deserialize).should be_true
   end
 
   it "should respond to {scope}_serialize if Manager.serialize_into_session is called with scope" do
-    Rack::Builder.new do 
+    Rack::Builder.new do
       Warden::Manager.serialize_into_session(:admin) { |n| n }
     end
     serializer_respond_to?(:admin_serialize).should be_true

spec/warden/session_serializer_spec.rb

@@ -1,8 +1,6 @@
 # encoding: utf-8
 # frozen_string_literal: true
-require 'spec_helper'
-
-describe Warden::SessionSerializer do
+RSpec.describe Warden::SessionSerializer do
   before(:each) do
     @env = env_with_params
     @env['rack.session'] ||= {}

spec/warden/strategies_spec.rb

@@ -1,8 +1,6 @@
 # encoding: utf-8
 # frozen_string_literal: true
-require 'spec_helper'
-
-describe Warden::Strategies do
+RSpec.describe Warden::Strategies do
   it "should let me add a strategy via a block" do
     Warden::Strategies.add(:strategy1) do
       def authenticate!

spec/warden/test/helpers_spec.rb

@@ -1,8 +1,6 @@
 # encoding: utf-8
 # frozen_string_literal: true
-require 'spec_helper'
-
-describe Warden::Test::Helpers do
+RSpec.describe Warden::Test::Helpers do
   before{ $captures = [] }
   after{ Warden.test_reset! }
 

spec/warden/test/mock_spec.rb

@@ -1,8 +1,6 @@
 # encoding: utf-8
 # frozen_string_literal: true
-require 'spec_helper'
-
-describe Warden::Test::Mock do
+RSpec.describe Warden::Test::Mock do
   before{ $captures = [] }
   after{ Warden.test_reset! }
 

spec/warden/test/test_mode_spec.rb

@@ -1,24 +1,22 @@
 # encoding: utf-8
 # frozen_string_literal: true
-require 'spec_helper'
-
-describe Warden::Test::WardenHelpers do
+RSpec.describe Warden::Test::WardenHelpers do
   before :all do
     Warden.test_mode!
   end
 
   before do
     $captures = []
-    @app = lambda{|e| valid_response }
+    @app = lambda{|_e| valid_response }
   end
 
   after do
     Warden.test_reset!
   end
 
-  it{ expect(Warden).to respond_to(:test_mode!)       }
-  it{ expect(Warden).to respond_to(:on_next_request)  }
-  it{ expect(Warden).to respond_to(:test_reset!)      }
+  it{ expect(Warden).to respond_to(:test_mode!) }
+  it{ expect(Warden).to respond_to(:on_next_request) }
+  it{ expect(Warden).to respond_to(:test_reset!) }
 
   it "should execute the on_next_request block on the next request" do
     Warden.on_next_request do |warden|
@@ -31,15 +29,15 @@
   end
 
   it "should execute many on_next_request blocks on the next request" do
-    Warden.on_next_request{|w| $captures << :first    }
-    Warden.on_next_request{|w| $captures << :second   }
+    Warden.on_next_request{|_w| $captures << :first }
+    Warden.on_next_request{|_w| $captures << :second }
     setup_rack(@app).call(env_with_params)
     expect($captures).to eq([:first, :second])
   end
 
   it "should not execute on_next_request blocks on subsequent requests" do
     app = setup_rack(@app)
-    Warden.on_next_request{|w| $captures << :first }
+    Warden.on_next_request{|_w| $captures << :first }
     app.call(env_with_params)
     expect($captures).to eq([:first])
     $captures.clear
@@ -49,17 +47,17 @@
 
   it "should allow me to set new_on_next_request items to execute in the same test" do
     app = setup_rack(@app)
-    Warden.on_next_request{|w| $captures << :first }
+    Warden.on_next_request{|_w| $captures << :first }
     app.call(env_with_params)
     expect($captures).to eq([:first])
-    Warden.on_next_request{|w| $captures << :second }
+    Warden.on_next_request{|_w| $captures << :second }
     app.call(env_with_params)
     expect($captures).to eq([:first, :second])
   end
 
   it "should remove the on_next_request items when test is reset" do
     app = setup_rack(@app)
-    Warden.on_next_request{|w| $captures << :first }
+    Warden.on_next_request{|_w| $captures << :first }
     Warden.test_reset!
     app.call(env_with_params)
     expect($captures).to eq([])
@@ -68,7 +66,7 @@
   context "asset requests" do
     it "should not execute on_next_request blocks if this is an asset request" do
       app = setup_rack(@app)
-      Warden.on_next_request{|w| $captures << :first }
+      Warden.on_next_request{|_w| $captures << :first }
       app.call(env_with_params("/assets/fun.gif"))
       expect($captures).to eq([])
     end