Merge pull request #2800 from rspec/revert-2799-main

Revert "Improve consistency with official controller spec style"

Author: JonRowe

features/matchers/redirect_to_matcher.feature

@@ -13,30 +13,25 @@ Feature: `redirect_to` matcher
       require "rails_helper"
 
       RSpec.describe WidgetsController do
+
         describe "#create" do
-          it "redirects to widget_url(@widget)" do
-            post :create, :params => { :widget => { :name => "Foo" } }
+          subject { post :create, :params => { :widget => { :name => "Foo" } } }
 
-            expect(response).to redirect_to(widget_url(assigns(:widget)))
+          it "redirects to widget_url(@widget)" do
+            expect(subject).to redirect_to(widget_url(assigns(:widget)))
           end
 
           it "redirects_to :action => :show" do
-            post :create, :params => { :widget => { :name => "Foo" } }
-
-            expect(response).to redirect_to :action => :show,
+            expect(subject).to redirect_to :action => :show,
                                            :id => assigns(:widget).id
           end
 
           it "redirects_to(@widget)" do
-            post :create, :params => { :widget => { :name => "Foo" } }
-
-            expect(response).to redirect_to(assigns(:widget))
+            expect(subject).to redirect_to(assigns(:widget))
           end
 
           it "redirects_to /widgets/:id" do
-            post :create, :params => { :widget => { :name => "Foo" } }
-
-            expect(response).to redirect_to("/widgets/#{assigns(:widget).id}")
+            expect(subject).to redirect_to("/widgets/#{assigns(:widget).id}")
           end
         end
       end

features/matchers/render_template_matcher.feature

@@ -16,18 +16,16 @@ Feature: `render_template` matcher
 
       RSpec.describe GadgetsController do
         describe "GET #index" do
-          it "renders the index template" do
-            get :index
+          subject { get :index }
 
-            expect(response).to render_template(:index)
-            expect(response).to render_template("index")
-            expect(response).to render_template("gadgets/index")
+          it "renders the index template" do
+            expect(subject).to render_template(:index)
+            expect(subject).to render_template("index")
+            expect(subject).to render_template("gadgets/index")
           end
 
           it "does not render a different template" do
-            get :index
-
-            expect(response).to_not render_template("gadgets/show")
+            expect(subject).to_not render_template("gadgets/show")
           end
         end
       end
@@ -42,16 +40,14 @@ Feature: `render_template` matcher
 
       RSpec.describe GadgetsController do
         describe "GET #index" do
-          it "renders the application layout" do
-            get :index
+          subject { get :index }
 
-            expect(response).to render_template("layouts/application")
+          it "renders the application layout" do
+            expect(subject).to render_template("layouts/application")
           end
 
           it "does not render a different layout" do
-            get :index
-
-            expect(response).to_not render_template("layouts/admin")
+            expect(subject).to_not render_template("layouts/admin")
           end
         end
       end