stripe 1.13.0 → 1.14.0

data/History.txt

@@ -1,3 +1,8 @@
+=== 1.14.0 2014-06-17
+
+* 1 major enhancement:
+  * Add metadata for refunds and disputes
+
 === 1.13.0 2014-05-28
 
 * 1 major enhancement:

data/VERSION

@@ -1 +1 @@
-1.13.0
+1.14.0

data/lib/stripe.rb

@@ -38,6 +38,7 @@ require 'stripe/recipient'
 require 'stripe/card'
 require 'stripe/subscription'
 require 'stripe/application_fee'
+require 'stripe/refund'
 
 # Errors
 require 'stripe/errors/stripe_error'
@@ -87,7 +88,7 @@ module Stripe
                           :ssl_ca_file => @ssl_bundle_path)
     end
 
-    unless @CERTIFICATE_VERIFIED
+    if @verify_ssl_certs and !@CERTIFICATE_VERIFIED
       @CERTIFICATE_VERIFIED = CertificateBlacklist.check_ssl_cert(@api_base, @ssl_bundle_path)
     end
 

data/lib/stripe/refund.rb

@@ -0,0 +1,13 @@
+module Stripe
+  class Refund < APIResource
+    include Stripe::APIOperations::Update
+
+    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
+  end
+end

data/lib/stripe/util.rb

@@ -31,6 +31,7 @@ module Stripe
         'card' => Card,
         'subscription' => Subscription,
         'list' => ListObject,
+        'refund' => Refund,
         'application_fee' => ApplicationFee
       }
     end

data/lib/stripe/version.rb

@@ -1,3 +1,3 @@
 module Stripe
-  VERSION = '1.13.0'
+  VERSION = '1.14.0'
 end

data/test/stripe/metadata_test.rb

@@ -111,4 +111,4 @@ module Stripe
       Gem::Version.new(version) >= Gem::Version.new('1.9')
     end
   end
-end
+end

data/test/stripe/refund_test.rb

@@ -0,0 +1,47 @@
+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(test_response(test_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(test_response(test_charge), test_response(test_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).once.returns(test_response(test_charge))
+      @mock.expects(:post).once.returns(test_response(test_refund(:metadata => {'key' => 'value'})))
+
+      charge = Stripe::Charge.retrieve('test_charge')
+      refund = charge.refunds.first
+
+      assert_equal nil, refund.metadata['key']
+
+      refund.metadata['key'] = 'valu'
+      refund.save
+
+      assert_equal 'value', refund.metadata['key']
+    end
+
+    should "create should return a new refund" do
+      @mock.expects(:get).once.returns(test_response(test_charge))
+      @mock.expects(:post).once.returns(test_response(test_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/test_helper.rb

@@ -119,6 +119,7 @@ def test_customer_array
 end
 
 def test_charge(params={})
+  id = params[:id] || 'ch_test_charge'
   {
     :refunded => false,
     :paid => true,
@@ -132,12 +133,13 @@ def test_charge(params={})
       :id => "cc_test_card",
       :object => "card"
     },
-    :id => "ch_test_charge",
+    :id => id,
     :reason => "execute_charge",
     :livemode => false,
     :currency => "usd",
     :object => "charge",
     :created => 1304114826,
+    :refunds => test_refund_array(id),
     :metadata => {}
   }.merge(params)
 end
@@ -204,6 +206,18 @@ def test_subscription(params = {})
   }.merge(params)
 end
 
+def test_refund(params = {})
+  {
+    :object => 'refund',
+    :amount => 30,
+    :currency => "usd",
+    :created => 1308595038,
+    :id => "ref_test_refund",
+    :charge => "ch_test_charge",
+    :metadata => {}
+  }.merge(params)
+end
+
 def test_subscription_array(customer_id)
   {
     :data => [test_subscription, test_subscription, test_subscription],
@@ -212,6 +226,15 @@ def test_subscription_array(customer_id)
   }
 end
 
+def test_refund_array(charge_id)
+  {
+    :data => [test_refund, test_refund, test_refund],
+    :object => 'list',
+    :url => '/v1/charges/' + charge_id + '/refunds'
+  }
+end
+
+
 def test_invoice
   {
     :id => 'in_test_invoice',

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: stripe
 version: !ruby/object:Gem::Version
-  version: 1.13.0
+  version: 1.14.0
   prerelease: 
 platform: ruby
 authors:
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-05-28 00:00:00.000000000 Z
+date: 2014-06-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rest-client
@@ -175,6 +175,7 @@ files:
 - lib/stripe/list_object.rb
 - lib/stripe/plan.rb
 - lib/stripe/recipient.rb
+- lib/stripe/refund.rb
 - lib/stripe/singleton_api_resource.rb
 - lib/stripe/stripe_object.rb
 - lib/stripe/subscription.rb
@@ -195,6 +196,7 @@ files:
 - test/stripe/list_object_test.rb
 - test/stripe/metadata_test.rb
 - test/stripe/recipient_card_test.rb
+- test/stripe/refund_test.rb
 - test/stripe/stripe_object_test.rb
 - test/stripe/subscription_test.rb
 - test/stripe/transfer_test.rb
@@ -238,6 +240,7 @@ test_files:
 - test/stripe/list_object_test.rb
 - test/stripe/metadata_test.rb
 - test/stripe/recipient_card_test.rb
+- test/stripe/refund_test.rb
 - test/stripe/stripe_object_test.rb
 - test/stripe/subscription_test.rb
 - test/stripe/transfer_test.rb