Merge pull request #13 from dmargery/follow_redirect

Follow redirect : fix for issue #12

lib/restfully/session.rb

@@ -190,6 +190,9 @@ def process(response, request)
         logger.debug "Building response..."
         Resource.new(self, response, request).build
       when 201,202
+        logger.debug "Getting description of newly created resource at: #{response.head['Location'].inspect}"
+        get response.head['Location'], :head => request.head
+      when 301,302
         logger.debug "Following redirection to: #{response.head['Location'].inspect}"
         get response.head['Location'], :head => request.head
       when 204

spec/restfully/session_spec.rb

@@ -247,6 +247,19 @@
       end
     end
 
+    [301, 302].each do |status|
+      it "should fetch the resource specified in the Location header if status = #{status}" do
+        @response.stub!(:code).and_return(status)
+        @response.head['Location'] = @uri+"/path"
+
+        @session.should_receive(:get).
+          with(@uri+"/path", :head => @request.head).
+          and_return(resource=mock("resource"))
+        @session.process(@response, @request).
+          should == resource
+      end
+    end
+
     it "should return a Restfully::Resource if successful" do
       Restfully::MediaType.register Restfully::MediaType::ApplicationJson
       body = {