hypertext_client 0.0.7 → 0.0.8

data/CHANGELOG

@@ -1,3 +1,7 @@
+## 0.0.8 (October 11, 2010)
+
+Improved logic for processing id parameter to http methods
+
 ## 0.0.7 (October 11, 2010)
 
 Properly set camel cased parent class name

data/lib/hypertext_client/hypertext_client.rb

@@ -18,7 +18,7 @@
 
 # Hypertext provides an HTTP client based on HTTParty, with a few assumptions baked in.
 module HypertextClient
-	VERSION = '0.0.7'
+	VERSION = '0.0.8'
 
 	extend ActiveSupport::Concern
 	# Include HTTParty
@@ -105,10 +105,11 @@ module HypertextClient
 				end
 
 				path = args.shift
-				if (path.class == String)
-					path = '/' + path unless path =~ %r(http://) || path.chars.first == '/'
+				unless (path.is_a?(Hash) || path.is_a?(Array))
+					path = path.to_s
+					path = '/' + path unless path.nil? || path =~ %r(http://) || path.chars.first == '/'
 				else
-					args.unshift path unless path.nil?
+					args.unshift path
 					path = ''
 				end
 

data/test/test_hypertext_client.rb

@@ -12,7 +12,7 @@ FAKE_ENDPOINT = "http://api.skynetinc.com"
 
 FakeWeb.allow_net_connect = false
 FakeWeb.register_uri(:get,
-										 FAKE_ENDPOINT + '/widgets',
+										 FAKE_ENDPOINT + '/widgets/?',
 										 :body => ([{:foo => 1}] * 5).to_json,
 										 :content_type => 'application/json')
 
@@ -42,7 +42,12 @@ FakeWeb.register_uri(:get,
 										 :body => {:property => 'exists'}.to_json,
 										 :content_type => 'application/json')
 
-FakeWeb.register_uri(:get,
+FakeWeb.register_uri(:any,
+										 FAKE_ENDPOINT + '/camel_cases/example/camel_case_inheritors',
+										 :body => [{:property => 'exists'}].to_json,
+										 :content_type => 'application/json')
+
+FakeWeb.register_uri(:any,
 										 FAKE_ENDPOINT + '/camel_cases/example/camel_case_inheritors/123',
 										 :body => {:property => 'exists'}.to_json,
 										 :content_type => 'application/json')
@@ -146,4 +151,19 @@ class HypertextClientTest < MiniTest::Unit::TestCase
 		example = SkynetClient::CamelCaseInheritor.get('123', :camel_case => 'example')
 		assert example
 	end
+	
+	def test_requires_camelcase_id
+		example = SkynetClient::CamelCaseInheritor.delete('123', :camel_case => 'example')
+		assert_equal '/camel_cases/example/camel_case_inheritors/123', FakeWeb.last_request.path
+	end
+	
+	def test_build_url_non_string_for_sub_model
+		example = SkynetClient::CamelCaseInheritor.delete(123, :camel_case => 'example')
+		assert_equal '/camel_cases/example/camel_case_inheritors/123', FakeWeb.last_request.path
+	end
+	
+	def test_build_collection_url_from_hash_for_sub_model
+		example = SkynetClient::CamelCaseInheritor.get(:camel_case => 'example')
+		assert_equal '/camel_cases/example/camel_case_inheritors', FakeWeb.last_request.path
+	end
 end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 0
-  - 7
-  version: 0.0.7
+  - 8
+  version: 0.0.8
 platform: ruby
 authors: 
 - Quid, Inc.