Merge pull request #2771 from cbliard/support-rack-mock-response-with-have-http-status

Support `have_http_status` with `Rack::MockResponse`

Author: JonRowe

lib/rspec/rails/matchers/have_http_status.rb

@@ -33,7 +33,7 @@ def self.matcher_for_status(target)
         # @param obj [Object] object to convert to a response
         # @return [ActionDispatch::TestResponse]
         def as_test_response(obj)
-          if ::ActionDispatch::Response === obj
+          if ::ActionDispatch::Response === obj || ::Rack::MockResponse === obj
             ::ActionDispatch::TestResponse.from_response(obj)
           elsif ::ActionDispatch::TestResponse === obj
             obj

spec/rspec/rails/matchers/have_http_status_spec.rb

@@ -16,6 +16,14 @@ def create_response(opts = {})
       end
     end
 
+    context "given a Rack::MockResponse" do
+      it "returns true for a response with the same code" do
+        response = ::Rack::MockResponse.new(code, {}, "")
+
+        expect(matcher.matches?(response)).to be(true)
+      end
+    end
+
     context "given an ActionDispatch::TestResponse" do
       it "returns true for a response with the same code" do
         response = ::ActionDispatch::TestResponse.new(code).tap { |x|