stripe 1.24.0 → 1.25.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7ecbd18ba821963c6f74fe3f77824f17abc2fe69
-  data.tar.gz: 88b6c051bb7a499cd1b0fe9ca017c56ee6ad3290
+  metadata.gz: 1e35419ace88d4f0aec2f7e5227a02dd5d9c4338
+  data.tar.gz: 223e592ef5558c048bff2825d31ed6842197b874
 SHA512:
-  metadata.gz: 8bcd08d62d22be5862e078ec20c574c2dde8045ebc4a9897853dbf695909e5a62ac9ea8cd417d9385aeba2078ff42ab9221f55e10aa9412f7d43a9fc5ca099d3
-  data.tar.gz: 09b0cb29cba10c861f4453cb648df2dacea2e7d1424ad1a1b141083b6ff5cb3f5cfbd105665aa22c98777e7df1e8d095733e4188c9eb16f911cf1d475453137a
+  metadata.gz: a0056b8bff477a1a2c5e7f2d25876af0aec97002ff4383bd4e4bab53c8e0c2018a18e1f95a50a85465b7baf2309d2cbb26b97abc121e7d90aa4945f6e77df0b3
+  data.tar.gz: e5126b06634a7c2062bbf60f1761eded6773dddb63d45d8a67302bbfcc3b60456f4db40ffbc398d5b6a24cd1f8d00a1b2fe6e37812b94a24717054be7506816f

data/History.txt

@@ -1,3 +1,7 @@
+=== 1.25.0 2015-08-17
+
+* Added support for refund listing and retrieval without an associated charge
+
 === 1.24.0 2015-08-03
 
 * Added support for deleting managed accounts

data/VERSION

@@ -1 +1 @@
-1.24.0
+1.25.0

data/lib/stripe/refund.rb

@@ -1,14 +1,7 @@
 module Stripe
   class Refund < APIResource
-    include Stripe::APIOperations::Update
+    include Stripe::APIOperations::Create
     include Stripe::APIOperations::List
-
-    def url
-      "#{Charge.url}/#{CGI.escape(charge)}/refunds/#{CGI.escape(id)}"
-    end
-
-    def self.retrieve(id, api_key=nil)
-      raise NotImplementedError.new("Refunds cannot be retrieved without a charge ID. Retrieve a refund using charge.refunds.retrieve('refund_id')")
-    end
+    include Stripe::APIOperations::Update
   end
 end

data/lib/stripe/version.rb

@@ -1,3 +1,3 @@
 module Stripe
-  VERSION = '1.24.0'
+  VERSION = '1.25.0'
 end

data/test/stripe/charge_refund_test.rb

@@ -0,0 +1,55 @@
+require File.expand_path('../../test_helper', __FILE__)
+
+module Stripe
+  class ChargeRefundTest < Test::Unit::TestCase
+    should "refunds should be listable" do
+      @mock.expects(:get).once.returns(make_response(make_charge))
+
+      charge = Stripe::Charge.retrieve('test_charge')
+
+      assert charge.refunds.first.kind_of?(Stripe::Refund)
+    end
+
+    should "refunds should be refreshable" do
+      @mock.expects(:get).twice.returns(make_response(make_charge), make_response(make_refund(:id => 'refreshed_refund')))
+
+      charge = Stripe::Charge.retrieve('test_charge')
+      refund = charge.refunds.first
+      refund.refresh
+
+      assert_equal 'refreshed_refund', refund.id
+    end
+
+    should "refunds should be updateable" do
+      @mock.expects(:get).
+        with("#{Stripe.api_base}/v1/charges/test_charge", nil, nil).
+        once.returns(make_response(make_charge))
+      @mock.expects(:post).
+        with("#{Stripe.api_base}/v1/refunds/ref_test_refund", nil, 'metadata[key]=value').
+        once.returns(make_response(make_refund(:metadata => {'key' => 'value'})))
+
+      charge = Stripe::Charge.retrieve('test_charge')
+      refund = charge.refunds.first
+
+      assert_equal nil, refund.metadata['key']
+
+      refund.metadata['key'] = 'value'
+      refund.save
+
+      assert_equal 'value', refund.metadata['key']
+    end
+
+    should "create should return a new refund" do
+      @mock.expects(:get).
+        with("#{Stripe.api_base}/v1/charges/test_charge", nil, nil).
+        once.returns(make_response(make_charge(:id => 'test_charge')))
+      @mock.expects(:post).
+        with("#{Stripe.api_base}/v1/charges/test_charge/refunds", nil, 'amount=20').
+        once.returns(make_response(make_refund(:id => 'test_new_refund')))
+
+      charge = Stripe::Charge.retrieve('test_charge')
+      refund = charge.refunds.create(:amount => 20)
+      assert_equal 'test_new_refund', refund.id
+    end
+  end
+end

data/test/stripe/refund_test.rb

@@ -3,44 +3,52 @@ require File.expand_path('../../test_helper', __FILE__)
 module Stripe
   class RefundTest < Test::Unit::TestCase
     should "refunds should be listable" do
-      @mock.expects(:get).once.returns(make_response(make_charge))
+      @mock.expects(:get).
+        with("#{Stripe.api_base}/v1/refunds", nil, nil).
+        once.returns(make_response(make_refund_array))
 
-      charge = Stripe::Charge.retrieve('test_charge')
+      refunds = Stripe::Refund.all
 
-      assert charge.refunds.first.kind_of?(Stripe::Refund)
+      assert refunds.first.kind_of?(Stripe::Refund)
     end
 
     should "refunds should be refreshable" do
-      @mock.expects(:get).twice.returns(make_response(make_charge), make_response(make_refund(:id => 'refreshed_refund')))
+      @mock.expects(:get).
+        with("#{Stripe.api_base}/v1/refunds/test_refund", nil, nil).
+        twice.returns(make_response(make_refund(:id => 'test_refund')),
+                       make_response(make_refund(:id => 'refreshed_refund')))
 
-      charge = Stripe::Charge.retrieve('test_charge')
-      refund = charge.refunds.first
+      refund = Stripe::Refund.retrieve('test_refund')
       refund.refresh
 
       assert_equal 'refreshed_refund', refund.id
     end
 
     should "refunds should be updateable" do
-      @mock.expects(:get).once.returns(make_response(make_charge))
-      @mock.expects(:post).once.returns(make_response(make_refund(:metadata => {'key' => 'value'})))
+      @mock.expects(:get).
+        with("#{Stripe.api_base}/v1/refunds/get_refund", nil, nil).
+        once.returns(make_response(make_refund(:id => 'save_refund')))
 
-      charge = Stripe::Charge.retrieve('test_charge')
-      refund = charge.refunds.first
+      @mock.expects(:post).
+        with("#{Stripe.api_base}/v1/refunds/save_refund", nil, 'metadata[key]=value').
+        once.returns(make_response(make_refund(:metadata => {'key' => 'value'})))
+
+      refund = Stripe::Refund.retrieve('get_refund')
 
       assert_equal nil, refund.metadata['key']
 
-      refund.metadata['key'] = 'valu'
+      refund.metadata['key'] = 'value'
       refund.save
 
       assert_equal 'value', refund.metadata['key']
     end
 
     should "create should return a new refund" do
-      @mock.expects(:get).once.returns(make_response(make_charge))
-      @mock.expects(:post).once.returns(make_response(make_refund(:id => 'test_new_refund')))
+      @mock.expects(:post).
+        with("#{Stripe.api_base}/v1/refunds", nil, 'charge=test_charge').
+        once.returns(make_response(make_refund(:id => 'test_new_refund')))
 
-      charge = Stripe::Charge.retrieve('test_charge')
-      refund = charge.refunds.create(:amount => 20)
+      refund = Stripe::Refund.create(:charge => 'test_charge')
       assert_equal 'test_new_refund', refund.id
     end
   end

data/test/test_data.rb

@@ -284,11 +284,13 @@ module Stripe
       }
     end
 
-    def make_refund_array(charge_id)
+    def make_refund_array(charge=nil)
+      p = charge ? {:charge => charge} : {}
+
       {
-        :data => [make_refund, make_refund, make_refund],
+        :data => [make_refund(p), make_refund(p), make_refund(p)],
         :object => 'list',
-        :url => '/v1/charges/' + charge_id + '/refunds'
+        :url => charge ? "/v1/charges/#{charge}/refunds" : '/v1/refunds'
       }
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: stripe
 version: !ruby/object:Gem::Version
-  version: 1.24.0
+  version: 1.25.0
 platform: ruby
 authors:
 - Ross Boucher
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-08-03 00:00:00.000000000 Z
+date: 2015-08-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rest-client
@@ -168,6 +168,7 @@ files:
 - test/stripe/application_fee_test.rb
 - test/stripe/balance_test.rb
 - test/stripe/bitcoin_receiver_test.rb
+- test/stripe/charge_refund_test.rb
 - test/stripe/charge_test.rb
 - test/stripe/coupon_test.rb
 - test/stripe/customer_card_test.rb
@@ -217,6 +218,7 @@ test_files:
 - test/stripe/application_fee_test.rb
 - test/stripe/balance_test.rb
 - test/stripe/bitcoin_receiver_test.rb
+- test/stripe/charge_refund_test.rb
 - test/stripe/charge_test.rb
 - test/stripe/coupon_test.rb
 - test/stripe/customer_card_test.rb