RubyGems - tarquinn - Versions diffs - 0.1.0 → 0.3.0 - Mend

tarquinn 0.1.0 → 0.3.0

Files changed (101) hide show

checksums.yaml +5 -5
data/.circleci/config.yml +85 -0
data/.gitignore +10 -0
data/.rubocop.yml +29 -0
data/.rubocop_todo.yml +13 -0
data/Dockerfile +21 -0
data/Gemfile +16 -2
data/README.md +16 -0
data/Rakefile +8 -3
data/config/check_specs.yml +3 -0
data/config/rubycritc.rb +12 -0
data/config/yardstick.rb +13 -0
data/config/yardstick.yml +33 -0
data/docker-compose.yml +23 -0
data/lib/tarquinn/class_methods.rb +43 -12
data/lib/tarquinn/condition/action_checker.rb +24 -7
data/lib/tarquinn/condition/method_caller.rb +25 -8
data/lib/tarquinn/condition/proc_runner.rb +18 -7
data/lib/tarquinn/condition.rb +57 -4
data/lib/tarquinn/controller.rb +56 -13
data/lib/tarquinn/redirection_config.rb +92 -0
data/lib/tarquinn/redirection_handler.rb +97 -0
data/lib/tarquinn/request_handler.rb +61 -0
data/lib/tarquinn/request_handler_builder.rb +84 -0
data/lib/tarquinn/version.rb +3 -1
data/lib/tarquinn.rb +74 -8
data/spec/dummy/Rakefile +8 -0
data/spec/dummy/app/assets/images/.keep +0 -0
data/spec/dummy/app/assets/stylesheets/application.css +1 -0
data/spec/dummy/app/channels/application_cable/channel.rb +6 -0
data/spec/dummy/app/channels/application_cable/connection.rb +6 -0
data/spec/dummy/app/controllers/application_controller.rb +4 -0
data/spec/dummy/app/controllers/concerns/.keep +0 -0
data/spec/dummy/app/controllers/tarquinn/dummy_controller.rb +43 -0
data/spec/dummy/app/controllers/tarquinn/dummy_route_controller.rb +37 -0
data/spec/dummy/app/helpers/application_helper.rb +4 -0
data/spec/dummy/app/jobs/application_job.rb +9 -0
data/spec/dummy/app/mailers/application_mailer.rb +6 -0
data/spec/dummy/app/models/application_record.rb +5 -0
data/spec/dummy/app/models/concerns/.keep +0 -0
data/spec/dummy/app/views/layouts/application.html.erb +15 -0
data/spec/dummy/app/views/layouts/mailer.html.erb +13 -0
data/spec/dummy/app/views/layouts/mailer.text.erb +1 -0
data/spec/dummy/app/views/tarquinn/dummy_route/index.html +0 -0
data/spec/dummy/app/views/tarquinn/dummy_route/new.html +0 -0
data/spec/dummy/bin/rails +6 -0
data/spec/dummy/bin/rake +6 -0
data/spec/dummy/bin/setup +35 -0
data/spec/dummy/config/application.rb +24 -0
data/spec/dummy/config/boot.rb +7 -0
data/spec/dummy/config/cable.yml +10 -0
data/spec/dummy/config/database.yml +25 -0
data/spec/dummy/config/environment.rb +7 -0
data/spec/dummy/config/environments/development.rb +69 -0
data/spec/dummy/config/environments/production.rb +89 -0
data/spec/dummy/config/environments/test.rb +62 -0
data/spec/dummy/config/initializers/content_security_policy.rb +26 -0
data/spec/dummy/config/initializers/filter_parameter_logging.rb +10 -0
data/spec/dummy/config/initializers/inflections.rb +17 -0
data/spec/dummy/config/initializers/permissions_policy.rb +12 -0
data/spec/dummy/config/locales/en.yml +33 -0
data/spec/dummy/config/puma.rb +45 -0
data/spec/dummy/config/routes.rb +8 -0
data/spec/dummy/config/storage.yml +34 -0
data/spec/dummy/config.ru +8 -0
data/spec/dummy/db/schema.rb +16 -0
data/spec/dummy/lib/assets/.keep +0 -0
data/spec/dummy/log/.keep +0 -0
data/spec/dummy/public/404.html +67 -0
data/spec/dummy/public/422.html +67 -0
data/spec/dummy/public/500.html +66 -0
data/spec/dummy/public/apple-touch-icon-precomposed.png +0 -0
data/spec/dummy/public/apple-touch-icon.png +0 -0
data/spec/dummy/public/favicon.ico +0 -0
data/spec/dummy/storage/.keep +0 -0
data/spec/dummy/tmp/.keep +0 -0
data/spec/dummy/tmp/pids/.keep +0 -0
data/spec/dummy/tmp/storage/.keep +0 -0
data/spec/lib/tarquinn/condition/action_checker_spec.rb +4 -2
data/spec/lib/tarquinn/condition/method_caller_spec.rb +5 -3
data/spec/lib/tarquinn/condition/proc_runner_spec.rb +56 -4
data/spec/lib/tarquinn/condition_spec.rb +53 -0
data/spec/lib/tarquinn/controller_spec.rb +83 -10
data/spec/lib/tarquinn/{config_spec.rb → redirection_config_spec.rb} +3 -1
data/spec/lib/tarquinn/{handler_spec.rb → redirection_handler_spec.rb} +21 -3
data/spec/lib/tarquinn/request_handler_builder_spec.rb +15 -0
data/spec/lib/tarquinn/{engine_spec.rb → request_handler_spec.rb} +5 -3
data/spec/lib/tarquinn_spec.rb +176 -13
data/spec/spec_helper.rb +17 -7
data/spec/support/shared_examples/config.rb +5 -4
data/tarquinn.gemspec +10 -14
data/tarquinn.jpg +0 -0
metadata +83 -107
data/Gemfile.lock +0 -58
data/lib/tarquinn/builder.rb +0 -28
data/lib/tarquinn/concern.rb +0 -17
data/lib/tarquinn/config.rb +0 -39
data/lib/tarquinn/engine.rb +0 -31
data/lib/tarquinn/handler.rb +0 -49
data/spec/lib/tarquinn/builder_spec.rb +0 -13
data/spec/support/models/tarquinn/dummy_controller.rb +0 -43

data/spec/dummy/public/500.html ADDED Viewed

@@ -0,0 +1,66 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>We're sorry, but something went wrong (500)</title>
+  <meta name="viewport" content="width=device-width,initial-scale=1">
+  <style>
+  .rails-default-error-page {
+    background-color: #EFEFEF;
+    color: #2E2F30;
+    text-align: center;
+    font-family: arial, sans-serif;
+    margin: 0;
+  }
+  .rails-default-error-page div.dialog {
+    width: 95%;
+    max-width: 33em;
+    margin: 4em auto 0;
+  }
+  .rails-default-error-page div.dialog > div {
+    border: 1px solid #CCC;
+    border-right-color: #999;
+    border-left-color: #999;
+    border-bottom-color: #BBB;
+    border-top: #B00100 solid 4px;
+    border-top-left-radius: 9px;
+    border-top-right-radius: 9px;
+    background-color: white;
+    padding: 7px 12% 0;
+    box-shadow: 0 3px 8px rgba(50, 50, 50, 0.17);
+  }
+  .rails-default-error-page h1 {
+    font-size: 100%;
+    color: #730E15;
+    line-height: 1.5em;
+  }
+  .rails-default-error-page div.dialog > p {
+    margin: 0 0 1em;
+    padding: 1em;
+    background-color: #F7F7F7;
+    border: 1px solid #CCC;
+    border-right-color: #999;
+    border-left-color: #999;
+    border-bottom-color: #999;
+    border-bottom-left-radius: 4px;
+    border-bottom-right-radius: 4px;
+    border-top-color: #DADADA;
+    color: #666;
+    box-shadow: 0 3px 8px rgba(50, 50, 50, 0.17);
+  }
+  </style>
+</head>
+<body class="rails-default-error-page">
+  <!-- This file lives in public/500.html -->
+  <div class="dialog">
+    <div>
+      <h1>We're sorry, but something went wrong.</h1>
+    </div>
+    <p>If you are the application owner check the logs for more information.</p>
+  </div>
+</body>
+</html>

data/spec/dummy/public/apple-touch-icon-precomposed.png ADDED Viewed

File without changes

data/spec/dummy/public/apple-touch-icon.png ADDED Viewed

File without changes

data/spec/dummy/public/favicon.ico ADDED Viewed

File without changes

data/spec/dummy/storage/.keep ADDED Viewed

File without changes

data/spec/dummy/tmp/.keep ADDED Viewed

File without changes

data/spec/dummy/tmp/pids/.keep ADDED Viewed

File without changes

data/spec/dummy/tmp/storage/.keep ADDED Viewed

File without changes

data/spec/lib/tarquinn/condition/action_checker_spec.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 require 'spec_helper'
 describe Tarquinn::Condition::ActionChecker do
@@ -34,7 +36,7 @@ describe Tarquinn::Condition::ActionChecker do
   end
   context 'when initialized with more routes' do
-    let(:routes) { [ :show, :view ] }
+    let(:routes) { %i[show view] }
     let(:subject) { described_class.new(routes) }
     context 'when receiving a request for one of the given action' do
@@ -44,7 +46,7 @@ describe Tarquinn::Condition::ActionChecker do
     end
     context 'when receiving a request for another action' do
-      let(:routes) { [ :update, :view ] }
+      let(:routes) { %i[update view] }
       it do
         expect(subject.check?(controller)).to be_falsey
       end

data/spec/lib/tarquinn/condition/method_caller_spec.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 require 'spec_helper'
 describe Tarquinn::Condition::MethodCaller do
@@ -24,7 +26,7 @@ describe Tarquinn::Condition::MethodCaller do
   end
   context 'when initialized with more methods' do
-    let(:methods) { [ :true, :false ] }
+    let(:methods) { %i[true false] }
     let(:subject) { described_class.new(methods) }
     context 'when one return true and the other false' do
@@ -34,14 +36,14 @@ describe Tarquinn::Condition::MethodCaller do
     end
     context 'when all return true' do
-      let(:methods) { [ :true, :true ] }
+      let(:methods) { %i[true true] }
       it do
         expect(subject.check?(controller)).to be_truthy
       end
     end
     context 'when all return false' do
-      let(:methods) { [ :false, :false ] }
+      let(:methods) { %i[false false] }
       it do
         expect(subject.check?(controller)).to be_falsey
       end

data/spec/lib/tarquinn/condition/proc_runner_spec.rb CHANGED Viewed

@@ -1,22 +1,74 @@
+# frozen_string_literal: true
 require 'spec_helper'
 describe Tarquinn::Condition::ProcRunner do
-  let(:controller) { double }
-  let(:value) { true }
-  let(:subject) { described_class.new { value } }
+  let(:rails_controller) { rails_controller_class.new }
+  let(:controller) { Tarquinn::Controller.new(rails_controller) }
+  let(:subject) { described_class.new(&block) }
+  let(:rails_controller_class) do
+    Class.new(ActionController::Base)
+  end
   describe '#check?' do
     context 'when block evaluates into true' do
+      let(:block) { proc { true } }
       it do
         expect(subject.check?(controller)).to be_truthy
       end
     end
     context 'when block evaluates into false' do
-      let(:value) { false }
+      let(:block) { proc { false } }
       it do
         expect(subject.check?(controller)).to be_falsey
       end
     end
+    context 'when the block gets calls the controller method' do
+      let(:block) { proc { match? } }
+      context 'when the method returns true' do
+        let(:rails_controller_class) do
+          Class.new(ActionController::Base) do
+            def match?
+              true
+            end
+          end
+        end
+        it do
+          expect(subject.check?(controller)).to be_truthy
+        end
+      end
+      context 'when the method returns false' do
+        let(:rails_controller_class) do
+          Class.new(ActionController::Base) do
+            def match?
+              false
+            end
+          end
+        end
+        it do
+          expect(subject.check?(controller)).to be_falsey
+        end
+      end
+    end
+    context 'when the block references a value defined outside' do
+      let(:block) do
+        val = value
+        proc { val }
+      end
+      let(:value) { [false, true].sample }
+      it do
+        expect(subject.check?(controller)).to eq(value)
+      end
+    end
   end
 end

data/spec/lib/tarquinn/condition_spec.rb ADDED Viewed

@@ -0,0 +1,53 @@
+# frozen_string_literal: true
+require 'spec_helper'
+describe Tarquinn::Condition do
+  subject(:condition) { described_class.new }
+  let(:controller) { instance_double(Tarquinn::Controller) }
+  describe '.method_caller' do
+    context 'when methods are given' do
+      let(:methods) { %i[method1 method2] }
+      it do
+        expect(described_class.method_caller(methods))
+          .to be_a(Tarquinn::Condition::MethodCaller)
+      end
+      it do
+        expect(described_class.method_caller(methods))
+          .to eq(Tarquinn::Condition::MethodCaller.new(methods))
+      end
+    end
+    context 'when no methods are given' do
+      let(:methods) { %i[] }
+      it do
+        expect(described_class.method_caller(methods)).to be_nil
+      end
+    end
+  end
+  describe '.action_checker' do
+    let(:actions) { %i[index show] }
+    it do
+      expect(described_class.action_checker(actions))
+        .to be_a(Tarquinn::Condition::ActionChecker)
+    end
+    it do
+      expect(described_class.action_checker(actions))
+        .to eq(Tarquinn::Condition::ActionChecker.new(actions))
+    end
+  end
+  describe '#check?' do
+    it do
+      expect { condition.check?(controller) }.to raise_error(NotImplementedError)
+    end
+  end
+end

data/spec/lib/tarquinn/controller_spec.rb CHANGED Viewed

@@ -1,38 +1,111 @@
+# frozen_string_literal: true
 require 'spec_helper'
 describe Tarquinn::Controller do
-  let(:controller) { Tarquinn::DummyController.new }
-  let(:subject) { described_class.new(controller) }
+  subject(:controller) { described_class.new(rails_controller) }
+  let(:rails_controller) { Tarquinn::DummyController.new }
   describe '#call' do
-    it 'redirects a method call to the controller' do
-      expect(controller).to receive(:redirect_to)
-      subject.call(:redirect_to)
+    context 'when no arguments are given' do
+      before do
+        allow(rails_controller).to receive(:redirect_to).with(no_args)
+      end
+      it 'delegate a method call to the controller' do
+        subject.call(:redirect_to)
+        expect(rails_controller).to have_received(:redirect_to)
+      end
+    end
+    context 'when arguments are given' do
+      before do
+        allow(rails_controller)
+          .to receive(:redirect_to)
+          .with(1, 2, opt: 3)
+      end
+      it 'delegate a method call to the controller' do
+        subject.call(:redirect_to, 1, 2, opt: 3)
+        expect(rails_controller).to have_received(:redirect_to)
+      end
+    end
+    context 'when a block is given' do
+      let(:block) { proc { 12 } }
+      before do
+        allow(rails_controller)
+          .to receive(:redirect_to)
+          .with(no_args).and_yield
+      end
+      it 'delegate a method call to the controller' do
+        subject.call(:redirect_to, &block)
+        expect(rails_controller).to have_received(:redirect_to)
+      end
+      it 'passes the block to be executed' do
+        expect(subject.call(:redirect_to, &block)).to eq(12)
+      end
+    end
+  end
+  describe '#run' do
+    context 'when block does not call any controller method' do
+      let(:block) { proc { 12 } }
+      it 'returns the value from the block' do
+        expect(controller.run(&block)).to eq(12)
+      end
+    end
+    context 'when block references a value outside' do
+      let(:block) do
+        value = 15
+        proc { value }
+      end
+      it 'returns the value' do
+        expect(controller.run(&block)).to eq(15)
+      end
+    end
+    context 'when block references a method in the controller' do
+      let(:block) { proc { params } }
+      it 'returns the value' do
+        expect(controller.run(&block))
+          .to eq(ActionController::Parameters.new(action: 'show'))
+      end
     end
   end
   describe '#params' do
     it 'returns the instance params call' do
-      expect(subject.params).to eq(action: 'show')
+      expect(subject.params).to eq(ActionController::Parameters.new(action: 'show'))
     end
   end
-  describe '#has_method?' do
+  describe '#method?' do
     context 'when calling for a public method that exists' do
       it do
-        expect(subject.has_method?(:parse_request)).to be_truthy
+        expect(subject.method?(:parse_request)).to be_truthy
       end
     end
     context 'when calling for a private method that exists' do
       it do
-        expect(subject.has_method?(:redirection_path)).to be_truthy
+        expect(subject.method?(:redirection_path)).to be_truthy
       end
     end
     context 'when calling for a non existing method' do
       it do
-        expect(subject.has_method?(:non_existing)).to be_falsey
+        expect(subject.method?(:non_existing)).to be_falsey
       end
     end
   end

data/spec/lib/tarquinn/{config_spec.rb → redirection_config_spec.rb} RENAMED Viewed

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
 require 'spec_helper'
-describe Tarquinn::Config do
+describe Tarquinn::RedirectionConfig do
   let(:subject) { described_class.new(:redirect) }
   describe '#add_redirection_rules' do

data/spec/lib/tarquinn/{handler_spec.rb → redirection_handler_spec.rb} RENAMED Viewed

@@ -1,14 +1,32 @@
+# frozen_string_literal: true
 require 'spec_helper'
-describe Tarquinn::Handler do
+describe Tarquinn::RedirectionHandler do
   let(:redirection_path) { '/path' }
   let(:controller) { Tarquinn::DummyController.new }
-  let(:config) { Tarquinn::Config.new(:redirection_path) }
+  let(:config) { Tarquinn::RedirectionConfig.new(:redirection_path) }
   let(:subject) do
     described_class.new config, Tarquinn::Controller.new(controller)
   end
   describe '#perform_redirect?' do
+    context 'when there are no rules' do
+      it do
+        expect(subject.perform_redirect?).to be_truthy
+      end
+      context 'but some grant a skip' do
+        before do
+          config.add_skip_rules { true }
+        end
+        it do
+          expect(subject.perform_redirect?).to be_falsey
+        end
+      end
+    end
     context 'when rules allow for redirection' do
       before do
         config.add_redirection_rules { true }
@@ -98,7 +116,7 @@ describe Tarquinn::Handler do
       end
       context 'when configured with a method that does not exist in the controller' do
-        let(:config) { Tarquinn::Config.new('/new_path') }
+        let(:config) { Tarquinn::RedirectionConfig.new('/new_path') }
         it 'calls for redirection using static path' do
           expect(controller).to receive(:redirect_to).with('/new_path')

data/spec/lib/tarquinn/request_handler_builder_spec.rb ADDED Viewed

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+require 'spec_helper'
+describe Tarquinn::RequestHandlerBuilder do
+  describe '#build' do
+    let(:controller) do
+      double('controller')
+    end
+    it do
+      expect(subject.build(controller)).to be_a(Tarquinn::RequestHandler)
+    end
+  end
+end

data/spec/lib/tarquinn/{engine_spec.rb → request_handler_spec.rb} RENAMED Viewed

@@ -1,13 +1,15 @@
+# frozen_string_literal: true
 require 'spec_helper'
-describe Tarquinn::Engine do
+describe Tarquinn::RequestHandler do
   let(:redirection_path) { '/path' }
   let(:redirection_path2) { '/path2' }
   let(:controller) do
     double('controller', redirect_path: redirection_path, redirect_path2: redirection_path2)
   end
-  let(:config) { Tarquinn::Config.new(:redirect_path) }
-  let(:config2) { Tarquinn::Config.new(:redirect_path2) }
+  let(:config) { Tarquinn::RedirectionConfig.new(:redirect_path) }
+  let(:config2) { Tarquinn::RedirectionConfig.new(:redirect_path2) }
   let(:configs) { { redirect_path: config, redirect_path2: config2 } }
   let(:subject) do
     described_class.new configs, Tarquinn::Controller.new(controller)

data/spec/lib/tarquinn_spec.rb CHANGED Viewed

@@ -1,24 +1,187 @@
+# frozen_string_literal: true
 require 'spec_helper'
-describe Tarquinn do
-  let(:controller) { Tarquinn::DummyController.new }
+describe Tarquinn, type: :controller do
+  describe '.redirection_rule' do
+    context 'when there is only one condition' do
+      controller(Tarquinn::DummyRouteController) do
+        redirection_rule :redirection, :should_redirect?
+      end
+      before { get :index, params: parameters }
+      context 'when request indicates a redirection' do
+        let(:parameters) { { should_redirect: true } }
+        it 'performs a redirect' do
+          expect(response).to redirect_to('/path')
+        end
+      end
+      context 'when request does not indicate a redirection' do
+        let(:parameters) { {} }
+        it 'does not performs a redirect' do
+          expect(response).not_to redirect_to('/path')
+        end
+      end
+    end
+    # TODO: fix this condition
+    context 'when there is only one block condition' do
+      controller(Tarquinn::DummyRouteController) do
+        redirection_rule :redirection do
+          params[:redirect_block]
+        end
+      end
+      before { get :index, params: parameters }
+      context 'when request indicates a redirection' do
+        let(:parameters) { { redirect_block: true } }
+        it 'performs a redirect' do
+          expect(response).to redirect_to('/path')
+        end
+      end
+      context 'when request does not indicate a redirection' do
+        let(:parameters) { {} }
-  describe 'redirection' do
-    context 'when configuration calls for a method that allows redirection' do
-      it 'redirects to redirection path given by the method' do
-        expect(controller).to receive(:redirect_to).with('/path')
-        controller.parse_request
+        it 'does not performs a redirect' do
+          expect(response).not_to redirect_to('/path')
+        end
       end
     end
-    context 'when redirection skip method returns true' do
-      before do
-        allow(controller).to receive(:should_skip_redirect?) { true }
+    context 'when there are more conditions' do
+      controller(Tarquinn::DummyRouteController) do
+        redirection_rule :redirection, :should_redirect?, :condition2
       end
-      it 'does not redirect to redirection path given by the method' do
-        expect(controller).not_to receive(:redirect_to)
-        controller.parse_request
+      before { get :index, params: parameters }
+      context 'when request indicates a redirection through first condition' do
+        let(:parameters) { { should_redirect: true } }
+        it 'performs a redirect' do
+          expect(response).to redirect_to('/path')
+        end
+      end
+      context 'when request indicates a redirection through second condition' do
+        let(:parameters) { { redirect: true } }
+        it 'performs a redirect' do
+          expect(response).to redirect_to('/path')
+        end
+      end
+      context 'when request does not indicate a redirection' do
+        let(:parameters) { {} }
+        it 'does not performs a redirect' do
+          expect(response).not_to redirect_to('/path')
+        end
+      end
+    end
+    # TODO: fix this condition
+    context 'when there are no conditions' do
+      controller(Tarquinn::DummyRouteController) do
+        redirection_rule :redirection
+      end
+      before { get :index }
+      it 'performs a redirect' do
+        expect(response).to redirect_to('/path')
+      end
+    end
+  end
+  describe '.skip_redirection' do
+    context 'when there is only one skip condition' do
+      controller(Tarquinn::DummyRouteController) do
+        redirection_rule :redirection, :always_redirect
+        skip_redirection_rule :redirection, :should_skip?
+      end
+      before { get :index, params: parameters }
+      context 'when requesting with parameters that do not skip' do
+        let(:parameters) { {} }
+        it 'performs a redirect' do
+          expect(response).to redirect_to('/path')
+        end
+      end
+      context 'when request does not indicate a redirection' do
+        let(:parameters) { { should_skip: true } }
+        it 'does not performs a redirect' do
+          expect(response).not_to redirect_to('/path')
+        end
+      end
+    end
+    context 'when there are more than oneskip condition' do
+      controller(Tarquinn::DummyRouteController) do
+        redirection_rule :redirection, :always_redirect
+        skip_redirection_rule :redirection, :should_skip?, :do_skip?
+      end
+      before { get :index, params: parameters }
+      context 'when requesting with parameters that do not skip' do
+        let(:parameters) { {} }
+        it 'performs a redirect' do
+          expect(response).to redirect_to('/path')
+        end
+      end
+      context 'when request skips redirection using first condition' do
+        let(:parameters) { { should_skip: true } }
+        it 'does not performs a redirect' do
+          expect(response).not_to redirect_to('/path')
+        end
+      end
+      context 'when request skips redirection using second condition' do
+        let(:parameters) { { skip: true } }
+        it 'does not performs a redirect' do
+          expect(response).not_to redirect_to('/path')
+        end
+      end
+    end
+  end
+  describe '.skip_redirection_rule' do
+    controller(Tarquinn::DummyRouteController) do
+      redirection_rule :redirection, :always_redirect
+      skip_redirection :redirection, :index, :delete
+    end
+    before { get action }
+    context 'when requestin a path that redirects' do
+      let(:action) { :new }
+      it 'performs a redirect' do
+        expect(response).to redirect_to('/path')
+      end
+    end
+    context 'when request does not indicate a redirection' do
+      let(:action) { :index }
+      it 'does not performs a redirect' do
+        expect(response).not_to redirect_to('/path')
       end
     end
   end