Merge pull request #6 from wacko/fix/querystring-params

Send querystring params on GET requests

Author: juanrossi

lib/mango.rb

@@ -35,8 +35,10 @@ def self.request(method, url, api_key=nil, params={}, headers={})
     end
 
     payload = JSON.generate(params) if method == :post || method == :patch
+    params = nil unless method == :get
 
     headers = {
+      :params => params,
       :content_type => 'application/json'
     }.merge(headers)
 

test/mango_test.rb

@@ -19,17 +19,23 @@
 end
 
 test "operation retrieve" do |mock|
-  mock.expects(:get).once.with('https://api.getmango.com/v1/customers/123/').returns(test_response(test_customer))
+  mock.expects(:get).once.with('https://api.getmango.com/v1/customers/123/', {}).returns(test_response(test_customer))
   customer = Mango::Customers.retrieve 123
   assert_equal "customer_hash", customer[:uid]
 end
 
 test "operation list" do |mock|
-  mock.expects(:get).once.with('https://api.getmango.com/v1/customers/').returns(test_response([test_customer]))
+  mock.expects(:get).once.with('https://api.getmango.com/v1/customers/', {}).returns(test_response([test_customer]))
   customers = Mango::Customers.list
   assert_equal "customer_hash", customers.first[:uid]
 end
 
+test "aditional params on get" do |mock|
+  mock.expects(:get).once.with('https://api.getmango.com/v1/customers/', cardtype: 'visa').returns(test_response([test_customer]))
+  customers = Mango::Customers.list cardtype: 'visa'
+  assert_equal "customer_hash", customers.first[:uid]
+end
+
 test "operation create" do |mock|
   mock.expects(:post).once.with('https://api.getmango.com/v1/customers/', anything).returns(test_response(test_customer))
   customer = Mango::Customers.create customer_params

test/test_helper.rb

@@ -8,7 +8,7 @@ def self.mock_rest_client=(mock_client)
   def self.execute_request(options)
     case options[:method]
     when :get
-      @mock_rest_client.get options[:url]
+      @mock_rest_client.get options[:url], options[:headers][:params]
     when :post
       @mock_rest_client.post options[:url], options[:payload]
     when :patch