stripe 1.36.1 → 1.36.2

data/test/stripe/recipient_card_test.rb

@@ -17,14 +17,14 @@ module Stripe
       assert cards[0].kind_of? Stripe::Card
     end
 
-    should "recipient cards should have the correct url" do
+    should "recipient cards should have the correct resource url" do
       c = recipient
       @mock.expects(:get).once.returns(make_response(make_card(
         :id => 'test_card',
         :recipient => 'test_recipient'
       )))
       card = c.cards.retrieve('card')
-      assert_equal RECIPIENT_CARD_URL, card.url
+      assert_equal RECIPIENT_CARD_URL, card.resource_url
     end
 
     should "recipient cards should be deletable" do

data/test/stripe/stripe_object_test.rb

@@ -121,20 +121,20 @@ module Stripe
 
     should "#serialize_params on an empty object" do
       obj = Stripe::StripeObject.construct_from({})
-      assert_equal({}, Stripe::StripeObject.serialize_params(obj))
+      assert_equal({}, obj.serialize_params)
     end
 
     should "#serialize_params on a new object with a subobject" do
       obj = Stripe::StripeObject.new
       obj.metadata = { :foo => "bar" }
       assert_equal({ :metadata => { :foo => "bar" } },
-        Stripe::StripeObject.serialize_params(obj))
+        obj.serialize_params)
     end
 
     should "#serialize_params on a basic object" do
       obj = Stripe::StripeObject.construct_from({ :foo => nil })
       obj.update_attributes(:foo => "bar")
-      assert_equal({ :foo => "bar" }, Stripe::StripeObject.serialize_params(obj))
+      assert_equal({ :foo => "bar" }, obj.serialize_params)
     end
 
     should "#serialize_params on a more complex object" do
@@ -146,7 +146,7 @@ module Stripe
       })
       obj.foo.bar = "newbar"
       assert_equal({ :foo => { :bar => "newbar" } },
-        Stripe::StripeObject.serialize_params(obj))
+        obj.serialize_params)
     end
 
     should "#serialize_params on an array" do
@@ -155,7 +155,7 @@ module Stripe
       })
       obj.foo = ["new-value"]
       assert_equal({ :foo => ["new-value"] },
-        Stripe::StripeObject.serialize_params(obj))
+        obj.serialize_params)
     end
 
     should "#serialize_params on an array that shortens" do
@@ -164,7 +164,7 @@ module Stripe
       })
       obj.foo = ["new-value"]
       assert_equal({ :foo => ["new-value"] },
-        Stripe::StripeObject.serialize_params(obj))
+        obj.serialize_params)
     end
 
     should "#serialize_params on an array that lengthens" do
@@ -173,7 +173,7 @@ module Stripe
       })
       obj.foo = ["new-value"] * 4
       assert_equal({ :foo => ["new-value"] * 4 },
-        Stripe::StripeObject.serialize_params(obj))
+        obj.serialize_params)
     end
 
     should "#serialize_params on an array of hashes" do
@@ -187,12 +187,12 @@ module Stripe
       ]
       obj.foo[0].bar = "baz"
       assert_equal({ :foo => [{ :bar => "baz" }] },
-        Stripe::StripeObject.serialize_params(obj))
+        obj.serialize_params)
     end
 
     should "#serialize_params doesn't include unchanged values" do
       obj = Stripe::StripeObject.construct_from({ :foo => nil })
-      assert_equal({}, Stripe::StripeObject.serialize_params(obj))
+      assert_equal({}, obj.serialize_params)
     end
 
     should "#serialize_params on an array that is unchanged" do
@@ -200,7 +200,7 @@ module Stripe
         :foo => ["0-index", "1-index", "2-index"],
       })
       obj.foo = ["0-index", "1-index", "2-index"]
-      assert_equal({}, Stripe::StripeObject.serialize_params(obj))
+      assert_equal({}, obj.serialize_params)
     end
 
     should "#serialize_params with a StripeObject" do
@@ -211,8 +211,80 @@ module Stripe
       obj.metadata =
         Stripe::StripeObject.construct_from({ :foo => 'bar' })
 
-      serialized = Stripe::StripeObject.serialize_params(obj)
+      serialized = obj.serialize_params
       assert_equal({ :foo => "bar" }, serialized[:metadata])
     end
+
+    should "#serialize_params with a StripeObject that's been replaced" do
+      obj = Stripe::StripeObject.construct_from({
+        :metadata => Stripe::StripeObject.construct_from({ :bar => 'foo' })
+      })
+
+      # Here we replace the object wholesale which means that the client must
+      # be able to blank out the values that were in the old object, but which
+      # are no longer present in the new one.
+      obj.metadata =
+        Stripe::StripeObject.construct_from({ :baz => 'foo' })
+
+      serialized = obj.serialize_params
+      assert_equal({ :bar => "", :baz => 'foo' }, serialized[:metadata])
+    end
+
+    should "#serialize_params with an array of StripeObjects" do
+      obj = Stripe::StripeObject.construct_from({})
+      obj.metadata = [
+        Stripe::StripeObject.construct_from({ :foo => 'bar' })
+      ]
+
+      serialized = obj.serialize_params
+      assert_equal([{ :foo => "bar" }], serialized[:metadata])
+    end
+
+    should "#serialize_params and remove embedded APIResources" do
+      obj = Stripe::StripeObject.construct_from({
+        :customer => Customer.construct_from({})
+      })
+
+      serialized = obj.serialize_params
+      assert_equal({}, serialized)
+    end
+
+    should "#serialize_params takes a force option" do
+      obj = Stripe::StripeObject.construct_from({
+        :id => 'id',
+        :metadata => Stripe::StripeObject.construct_from({ :foo => 'bar' })
+      })
+
+      serialized = obj.serialize_params(:force => true)
+      assert_equal({ :id => 'id', :metadata => { :foo => 'bar' } }, serialized)
+    end
+
+    should "#dirty! forces an object and its subobjects to be saved" do
+      obj = Stripe::StripeObject.construct_from({
+        :id => 'id',
+        :metadata => Stripe::StripeObject.construct_from({ :foo => 'bar' })
+      })
+
+      # note that `force` and `dirty!` are for different things, but are
+      # functionally equivalent
+      obj.dirty!
+
+      serialized = obj.serialize_params
+      assert_equal({ :id => 'id', :metadata => { :foo => 'bar' } }, serialized)
+    end
+
+    should "warn that .serialize_params is deprecated" do
+      old_stderr = $stderr
+      $stderr = StringIO.new
+      begin
+        obj = Stripe::StripeObject.construct_from({})
+        Stripe::StripeObject.serialize_params(obj)
+        message = "NOTE: Stripe::StripeObject.serialize_params is " +
+          "deprecated; use #serialize_params instead"
+        assert_match Regexp.new(message), $stderr.string
+      ensure
+        $stderr = old_stderr
+      end
+    end
   end
 end

data/test/test_data.rb

@@ -56,7 +56,7 @@ module Stripe
       {
         :data => [make_balance_transaction, make_balance_transaction, make_balance_transaction],
         :object => "list",
-        :url => "/v1/balance/history"
+        :resource_url => "/v1/balance/history"
       }
     end
 
@@ -93,7 +93,7 @@ module Stripe
       {
         :data => [make_application_fee, make_application_fee, make_application_fee],
         :object => 'list',
-        :url => '/v1/application_fees'
+        :resource_url => '/v1/application_fees'
       }
     end
 
@@ -101,7 +101,7 @@ module Stripe
       {
         :data => [make_application_fee_refund, make_application_fee_refund, make_application_fee_refund],
         :object => 'list',
-        :url => '/v1/application_fees/' + fee_id + '/refunds'
+        :resource_url => '/v1/application_fees/' + fee_id + '/refunds'
       }
     end
 
@@ -126,7 +126,7 @@ module Stripe
       {
         :data => [make_customer, make_customer, make_customer],
         :object => 'list',
-        :url => '/v1/customers'
+        :resource_url => '/v1/customers'
       }
     end
 
@@ -160,7 +160,7 @@ module Stripe
       {
         :data => [make_charge, make_charge, make_charge],
         :object => 'list',
-        :url => '/v1/charges'
+        :resource_url => '/v1/charges'
       }
     end
 
@@ -184,7 +184,7 @@ module Stripe
       {
         :data => [make_dispute, make_dispute, make_dispute],
         :object => 'list',
-        :url => '/v1/disputes'
+        :resource_url => '/v1/disputes'
       }
     end
 
@@ -192,7 +192,7 @@ module Stripe
       {
         :data => [make_card, make_card, make_card],
         :object => 'list',
-        :url => '/v1/recipients/' + recipient_id + '/cards'
+        :resource_url => '/v1/recipients/' + recipient_id + '/cards'
       }
     end
 
@@ -200,7 +200,7 @@ module Stripe
       {
         :data => [make_card, make_card, make_card],
         :object => 'list',
-        :url => '/v1/customers/' + customer_id + '/sources'
+        :resource_url => '/v1/customers/' + customer_id + '/sources'
       }
     end
 
@@ -235,7 +235,7 @@ module Stripe
         :created => 1403047735,
         :size => 4908,
         :purpose => params[:purpose] || "dispute_evidence",
-        :url => nil,
+        :resource_url => nil,
         :type => nil,
       }
     end
@@ -244,7 +244,7 @@ module Stripe
       {
         :data => [make_file, make_file, make_file],
         :object => 'list',
-        :url => '/v1/files'
+        :resource_url => '/v1/files'
       }
     end
 
@@ -287,7 +287,7 @@ module Stripe
       {
         :data => [make_subscription, make_subscription, make_subscription],
         :object => 'list',
-        :url => '/v1/customers/' + customer_id + '/subscriptions'
+        :resource_url => '/v1/customers/' + customer_id + '/subscriptions'
       }
     end
 
@@ -297,7 +297,7 @@ module Stripe
       {
         :data => [make_refund(p), make_refund(p), make_refund(p)],
         :object => 'list',
-        :url => charge ? "/v1/charges/#{charge}/refunds" : '/v1/refunds'
+        :resource_url => charge ? "/v1/charges/#{charge}/refunds" : '/v1/refunds'
       }
     end
 
@@ -305,7 +305,7 @@ module Stripe
       {
         :data => [make_reversal, make_reversal, make_reversal],
         :object => 'list',
-        :url => '/v1/transfers/' + transfer_id + '/reversals'
+        :resource_url => '/v1/transfers/' + transfer_id + '/reversals'
       }
     end
 
@@ -366,7 +366,7 @@ module Stripe
       {
         :data => [make_invoice],
         :object => 'list',
-        :url => '/v1/invoices?customer=test_customer'
+        :resource_url => '/v1/invoices?customer=test_customer'
       }
     end
 
@@ -396,7 +396,7 @@ module Stripe
       {
         :data => [make_recipient, make_recipient, make_recipient],
         :object => 'list',
-        :url => '/v1/recipients'
+        :resource_url => '/v1/recipients'
       }
     end
 
@@ -427,7 +427,7 @@ module Stripe
       {
         :data => [make_transfer, make_transfer, make_transfer],
         :object => 'list',
-        :url => '/v1/transfers'
+        :resource_url => '/v1/transfers'
       }
     end
 
@@ -466,7 +466,7 @@ module Stripe
       {
         :data => [make_bitcoin_receiver, make_bitcoin_receiver, make_bitcoin_receiver],
         :object => 'list',
-        :url => '/v1/bitcoin/receivers'
+        :resource_url => '/v1/bitcoin/receivers'
       }
     end
 
@@ -485,7 +485,7 @@ module Stripe
       {
         :data => [make_bitcoin_transaction, make_bitcoin_transaction, make_bitcoin_transaction],
         :object => 'list',
-        :url => "/v1/bitcoin/receivers/btcrcv_test_receiver/transactions"
+        :resource_url => "/v1/bitcoin/receivers/btcrcv_test_receiver/transactions"
       }
     end
 
@@ -567,7 +567,7 @@ module Stripe
     def make_product_array
       {
         :object => "list",
-        :url => "/v1/products",
+        :resource_url => "/v1/products",
         :data => [
           make_product,
           make_product,
@@ -602,7 +602,7 @@ module Stripe
     def make_sku_array(product_id, params={})
       {
         :object => "list",
-        :url => "/v1/skus",
+        :resource_url => "/v1/skus",
         :data => [
           make_sku(:product => product_id),
           make_sku(:product => product_id),
@@ -656,7 +656,7 @@ module Stripe
     def make_order_array(params={})
       {
         :object => "list",
-        :url => "/v1/orders",
+        :resource_url => "/v1/orders",
         :data => [
           make_order,
           make_order,
@@ -675,7 +675,7 @@ module Stripe
     def country_spec_array
       {
         :object => "list",
-        :url => "/v1/country_specs",
+        :resource_url => "/v1/country_specs",
         :data => [
           make_country_spec,
           make_country_spec,
@@ -709,7 +709,7 @@ module Stripe
               "tos_acceptance.ip"
             ],
             :additional => [
-              "legal_entity.personal_id_number", 
+              "legal_entity.personal_id_number",
               "legal_entity.verification.document"
             ]
           },

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: stripe
 version: !ruby/object:Gem::Version
-  version: 1.36.1
+  version: 1.36.2
 platform: ruby
 authors:
 - Ross Boucher
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-03-04 00:00:00.000000000 Z
+date: 2016-03-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rest-client