Add #url and make #to_hash return Hash of which values are empty

Author: yykamei

lib/faraday/response.rb

@@ -33,6 +33,10 @@ def body
       finished? ? env.body : nil
     end
 
+     def url
+      finished? ? env.url : nil
+    end
+
     def finished?
       !!env
     end
@@ -59,12 +63,10 @@ def success?
     end
 
     def to_hash
-      return {} unless finished?
-
       {
-        status: env.status, body: env.body,
-        response_headers: env.response_headers,
-        url: env.url
+        status: status, body: body,
+        response_headers: headers,
+        url: url
       }
     end
 

spec/faraday/response_spec.rb

@@ -13,6 +13,7 @@
   it { expect(subject.success?).to be_falsey }
   it { expect(subject.status).to eq(404) }
   it { expect(subject.body).to eq('yikes') }
+  it { expect(subject.url).to eq(URI('https://lostisland.github.io/faraday')) }
   it { expect(subject.headers['Content-Type']).to eq('text/plain') }
   it { expect(subject['content-type']).to eq('text/plain') }
 
@@ -35,7 +36,7 @@
     context 'when response is not finished' do
       subject { Faraday::Response.new.to_hash }
 
-      it { is_expected.to eq({}) }
+      it { is_expected.to eq({ status: nil, body: nil, response_headers: {}, url: nil }) }
     end
   end