modify error message when redirected but expecting to render a template

Author: Alex Egan

lib/rspec/rails/matchers/have_rendered.rb

@@ -9,18 +9,32 @@ def initialize(scope, expected, message = nil)
             @expected = Symbol === expected ? expected.to_s : expected
             @message = message
             @scope = scope
+            @redirect_is = nil
           end
 
           # @api private
           def matches?(*)
-            match_unless_raises ActiveSupport::TestCase::Assertion do
+            match_check = match_unless_raises ActiveSupport::TestCase::Assertion do
               @scope.assert_template expected, @message
             end
+            check_redirect unless match_check
+            match_check
+          end
+
+          def check_redirect
+            response = @scope.response
+            return unless response.respond_to?(:redirect?) && response.redirect?
+            @redirect_is = @scope.send(:normalize_argument_to_redirection, response.location)
           end
 
           # @api private
           def failure_message
-            rescued_exception.message
+            if @redirect_is
+              rescued_exception.message[/.* but /] +
+                "was a redirect to <#{@redirect_is}>"
+            else
+              rescued_exception.message
+            end
           end
 
           # @api private

spec/rspec/rails/matchers/have_rendered_spec.rb

@@ -88,6 +88,24 @@ def assert_template(*); raise "oops"; end
           end.to raise_exception("oops")
         end
       end
+
+      context "when fails with a redirect" do
+        let(:response) { ActionController::TestResponse.new(302) }
+        def assert_template(*)
+          message = "expecting <'template_name'> but rendering with <[]>"
+          raise ActiveSupport::TestCase::Assertion.new(message)
+        end
+        def normalize_argument_to_redirection(*)
+          "http://test.host/widgets/1"
+        end
+        it "gives informative error message" do
+          response = ActionController::TestResponse.new(302)
+          response.location = "http://test.host/widgets/1"
+          expect do
+            expect(response).to send(template_expectation, "template_name")
+          end.to raise_exception("expecting <'template_name'> but was a redirect to <http://test.host/widgets/1>")
+        end
+      end
     end
   end
 end