Merge pull request #643 from JoshCheek/issue642-rack-solution

RackResponse includes the reason phrase

lib/webmock/rack_response.rb

@@ -12,7 +12,7 @@ def evaluate(request)
       Response.new(
         body: body_from_rack_response(response),
         headers: headers,
-        status: status
+        status: [status, Rack::Utils::HTTP_STATUS_CODES[status]]
       )
     end
 

spec/unit/rack_response_spec.rb

@@ -14,6 +14,16 @@
     expect(response.body).to include('This is my root!')
   end
 
+  it "should set the reason phrase based on the status code" do
+    request = WebMock::RequestSignature.new(:get, 'www.example.com')
+    response = @rack_response.evaluate(request)
+    expect(response.status).to eq([200, "OK"])
+
+    request = WebMock::RequestSignature.new(:get, 'www.example.com/error')
+    response = @rack_response.evaluate(request)
+    expect(response.status).to eq([500, "Internal Server Error"])
+  end
+
   it "should behave correctly when the rack response is not a simple array of strings" do
     request = WebMock::RequestSignature.new(:get, 'www.example.com/non_array_response')
     response = @rack_response.evaluate(request)