stripe 1.32.1 → 1.33.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d2d2a6395d80bde7fc0021ce5774016af214c990
-  data.tar.gz: 71f7a568f9e60ae0a696c2e43a31ba963ca05ae6
+  metadata.gz: e065dd2aa5986f505bf2c5497a7c759db6da997c
+  data.tar.gz: ba93ff1df77a7f3d89bc620dbcb0a5e574283c41
 SHA512:
-  metadata.gz: 6e2a2dc5678a605a941e28c2332383c8cebb1be58406881db0d0d1ed47a47f692be87868eb917aedd6ff8f231c4a7b92889c33b65e37bdb459e8fd8f827d8cfe
-  data.tar.gz: 20ce1c2d6be1316ef740ab3ad469fa676e1ada2e5d92cd7bd4587d4b3f0c574284e6b3392d29ed8b8bace956d9ea16c420495de373cebe3b3f00b4b2662a006a
+  metadata.gz: e6a4dc266b606ddff15e281c34da35d61fe16b73bb4d7ffd9d3c1300427bb7353d53b9d2f3ced2e8a3faa6de5beb6b40fdc82089dabc93a5cb93b2e917f52faa
+  data.tar.gz: 8f63334bc9c727b3dca16085019a6525ebf33a253788f74858fb8a5821d3ff162c8a254b493a296d7de03386eda684422aaa7641ea6e8cf15f97dd5194433e19

data/History.txt

@@ -1,3 +1,7 @@
+=== 1.33.0 2016-01-19
+
+* Re-implement `Charge#refund` helper to use the modern endpoint suggested by docs
+
 === 1.32.1 2016-01-07
 
 * Fix bug where ivar left uninitialized in StripeObject could error on serialization

data/VERSION

@@ -1 +1 @@
-1.32.1
+1.33.0

data/lib/stripe/charge.rb

@@ -5,8 +5,12 @@ module Stripe
     include Stripe::APIOperations::Update
 
     def refund(params={}, opts={})
-      response, opts = request(:post, refund_url, params, opts)
-      initialize_from(response, opts)
+      self.refunds.create
+
+      # now that a refund has been created, we expect the state of this object
+      # to change as well (i.e. `refunded` will now be `true`) so refresh it
+      # from the server
+      self.refresh
     end
 
     def capture(params={}, opts={})

data/lib/stripe/version.rb

@@ -1,3 +1,3 @@
 module Stripe
-  VERSION = '1.32.1'
+  VERSION = '1.33.0'
 end

data/test/stripe/api_resource_test.rb

@@ -206,12 +206,12 @@ module Stripe
               opts[:headers][:authorization] == 'Bearer local'
           end.returns(make_response(make_charge))
           Stripe.expects(:execute_request).with do |opts|
-            opts[:url] == "#{Stripe.api_base}/v1/charges/ch_test_charge/refund" &&
+            opts[:url] == "#{Stripe.api_base}/v1/charges/ch_test_charge/refunds" &&
               opts[:headers][:authorization] == 'Bearer local'
-          end.returns(make_response(make_charge))
+          end.returns(make_response(make_refund))
 
           ch = Stripe::Charge.retrieve('ch_test_charge', 'local')
-          ch.refund
+          ch.refunds.create
         end
       end
     end

data/test/stripe/application_fee_test.rb

@@ -14,18 +14,12 @@ module Stripe
     should "application fees should be refundable" do
       fee = Stripe::ApplicationFee.construct_from(make_application_fee)
 
-      # first a post to create a refund
       @mock.expects(:post).once.
         with("#{Stripe.api_base}/v1/application_fees/#{fee.id}/refunds", nil, '').
         returns(make_response(make_application_fee_refund))
 
-      # then a get to refresh the current object
-      @mock.expects(:get).once.
-        with("#{Stripe.api_base}/v1/application_fees/#{fee.id}", nil, nil).
-        returns(make_response({:id => "fee_test_fee", :refunded => true}))
-
-      fee.refund
-      assert fee.refunded
+      refund = fee.refunds.create
+      assert refund.is_a?(Stripe::ApplicationFeeRefund)
     end
   end
 end

data/test/stripe/charge_test.rb

@@ -12,11 +12,11 @@ module Stripe
     end
 
     should "charges should be refundable" do
+      c = Stripe::Charge.construct_from(make_charge)
       @mock.expects(:get).never
-      @mock.expects(:post).once.returns(make_response({:id => "ch_test_charge", :refunded => true}))
-      c = Stripe::Charge.new("test_charge")
-      c.refund
-      assert c.refunded
+      @mock.expects(:post).once.returns(make_response(make_refund(:charge => c)))
+      r = c.refunds.create
+      assert r.is_a?(Stripe::Refund)
     end
 
     should "charges should not be deletable" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: stripe
 version: !ruby/object:Gem::Version
-  version: 1.32.1
+  version: 1.33.0
 platform: ruby
 authors:
 - Ross Boucher
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-01-07 00:00:00.000000000 Z
+date: 2016-01-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rest-client