Make ActiveResource#exists? work [#3020 state:resolved]

Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
rails · Aug 9, 2009 · 323f58f · 323f58f
1 parent 29a5549
commit 323f58f
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/activeresource/lib/active_resource/connection.rb b/activeresource/lib/active_resource/connection.rb
@@ -75,7 +75,8 @@ class Connection
     HTTP_FORMAT_HEADER_NAMES = {  :get => 'Accept',
       :put => 'Content-Type',
       :post => 'Content-Type',
-      :delete => 'Accept'
+      :delete => 'Accept',
+      :head => 'Accept'
     }
 
     attr_reader :site, :user, :password, :timeout, :proxy, :ssl_options
@@ -155,7 +156,7 @@ def post(path, body = '', headers = {})
     # Execute a HEAD request.
     # Used to obtain meta-information about resources, such as whether they exist and their size (via response headers).
     def head(path, headers = {})
-      request(:head, path, build_request_headers(headers))
+      request(:head, path, build_request_headers(headers, :head))
     end
 
 

diff --git a/activeresource/test/base_test.rb b/activeresource/test/base_test.rb
@@ -965,6 +965,14 @@ def to_param
     end
   end
 
+  def test_exists_without_http_mock
+    http = Net::HTTP.new(Person.site.host, Person.site.port)
+    ActiveResource::Connection.any_instance.expects(:http).returns(http)
+    http.expects(:request).returns(ActiveResource::Response.new(""))
+
+    assert Person.exists?('not-mocked')
+  end
+
   def test_to_xml
     matz = Person.find(1)
     xml = matz.encode