hypertext_client 0.0.8 → 0.0.9

data/CHANGELOG

@@ -1,3 +1,7 @@
+## 0.0.9 (October 11, 2010)
+
+Prevented caching in base_uri for parent class ids
+
 ## 0.0.8 (October 11, 2010)
 
 Improved logic for processing id parameter to http methods

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.8'
+	VERSION = '0.0.9'
 
 	extend ActiveSupport::Concern
 	# Include HTTParty
@@ -114,13 +114,15 @@ module HypertextClient
 				end
 
 				if self.parents.count > 0 && !(path =~ %r(http://))
+					full_uri = "#{base_uri}"
 					options = args.first
 					raise MissingArgumentError.new("Options hash required; must include parent ids") unless options.is_a? Hash
 					self.parents.each do |parent|
 						key = parent.model_name.demodulize.singularize.underscore.to_sym
 						raise MissingArgumentError.new("Missing required argument: #{key}") unless options.has_key? key
-						base_uri self.base_uri.gsub("|#{key}|", options[key])
+						full_uri.gsub!("|#{key}|", options[key])
 					end
+					path = full_uri + path
 				end
 
 				response = super(path, *args)

data/test/test_hypertext_client.rb

@@ -47,10 +47,20 @@ FakeWeb.register_uri(:any,
 										 :body => [{:property => 'exists'}].to_json,
 										 :content_type => 'application/json')
 
+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')
+										
+FakeWeb.register_uri(:any,
+										 FAKE_ENDPOINT + '/camel_cases/different_example/camel_case_inheritors',
+										 :body => {:property => 'exists'}.to_json,
+										 :content_type => 'application/json')
 
 
 module SkynetClient
@@ -162,8 +172,20 @@ class HypertextClientTest < MiniTest::Unit::TestCase
 		assert_equal '/camel_cases/example/camel_case_inheritors/123', FakeWeb.last_request.path
 	end
 	
+	def test_dont_duplicate_first_param_as_parent_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_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
+	
+	def test_base_uri_should_reset_after_call
+		example = SkynetClient::CamelCaseInheritor.get(:camel_case => 'example')
+		assert_equal '/camel_cases/example/camel_case_inheritors', FakeWeb.last_request.path
+		example = SkynetClient::CamelCaseInheritor.get(:camel_case => 'different_example')
+		assert_equal '/camel_cases/different_example/camel_case_inheritors', FakeWeb.last_request.path
+	end
 end

metadata

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