stripe 1.31.0 → 1.58.0

data/test/stripe/customer_card_test.rb

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

data/test/stripe/customer_test.rb

@@ -16,7 +16,7 @@ module Stripe
       assert c.deleted
     end
 
-    should "customers should be updateable" do
+    should "customers should be saveable" do
       @mock.expects(:get).once.returns(make_response(make_customer({:mnemonic => "foo"})))
       @mock.expects(:post).once.returns(make_response(make_customer({:mnemonic => "bar"})))
       c = Stripe::Customer.new("test_customer").refresh
@@ -26,6 +26,14 @@ module Stripe
       assert_equal "bar", c.mnemonic
     end
 
+    should "customers should be updateable" do
+      @mock.expects(:post).once.
+        with("https://api.stripe.com/v1/customers/test_customer", nil, "metadata[foo]=bar").
+        returns(make_response(make_customer(metadata: {foo: 'bar'})))
+      c = Stripe::Customer.update("test_customer", metadata: {foo: 'bar'})
+      assert_equal('bar', c.metadata['foo'])
+    end
+
     should "create should return a new customer" do
       @mock.expects(:post).once.returns(make_response(make_customer))
       c = Stripe::Customer.create
@@ -84,5 +92,20 @@ module Stripe
       c.delete_discount
       assert_equal nil, c.discount
     end
+
+    should "can have a token source set" do
+      c = Stripe::Customer.new("test_customer")
+      c.source = "tok_123"
+      assert_equal "tok_123", c.source
+    end
+
+    should "set a flag if given an object source" do
+      c = Stripe::Customer.new("test_customer")
+      c.source = {
+        :object => 'card'
+      }
+      assert_equal true, c.source.save_with_parent
+    end
   end
 end
+

data/test/stripe/dispute_test.rb

@@ -29,6 +29,14 @@ module Stripe
     end
 
     should "disputes should be updateable" do
+      @mock.expects(:post).once.
+        with("https://api.stripe.com/v1/disputes/test_dispute", nil, "metadata[foo]=bar").
+        returns(make_response(make_dispute(metadata: {foo: 'bar'})))
+      d = Stripe::Dispute.update("test_dispute", metadata: {foo: 'bar'})
+      assert_equal('bar', d.metadata['foo'])
+    end
+
+    should "disputes should be saveable" do
       @mock.expects(:get).once.returns(make_response(make_dispute))
       @mock.expects(:post).with(
         "#{Stripe.api_base}/v1/disputes/dp_test_dispute",

data/test/stripe/errors_test.rb

@@ -0,0 +1,18 @@
+require File.expand_path('../../test_helper', __FILE__)
+
+module Stripe
+  class StripeErrorTest < Test::Unit::TestCase
+    context "#to_s" do
+      should "convert to string" do
+        e = StripeError.new("message")
+        assert_equal "message", e.to_s
+
+        e = StripeError.new("message", 200)
+        assert_equal "(Status 200) message", e.to_s
+
+        e = StripeError.new("message", nil, nil, nil, { :request_id => "request-id" })
+        assert_equal "(Request request-id) message", e.to_s
+      end
+    end
+  end
+end

data/test/stripe/invoice_item_test.rb

@@ -0,0 +1,19 @@
+require File.expand_path('../../test_helper', __FILE__)
+
+module Stripe
+  class InvoiceItemTest < Test::Unit::TestCase
+    should "retrieve should retrieve invoice items" do
+      @mock.expects(:get).once.returns(make_response(make_invoice_item))
+      ii = Stripe::InvoiceItem.retrieve('in_test_invoice_item')
+      assert_equal 'ii_test_invoice_item', ii.id
+    end
+
+    should "invoice items should be updateable" do
+      @mock.expects(:post).once.
+        with('https://api.stripe.com/v1/invoiceitems/test_invoice_item', nil, 'metadata[foo]=bar').
+        returns(make_response(make_charge(metadata: {'foo' => 'bar'})))
+      ii = Stripe::InvoiceItem.update("test_invoice_item", metadata: {foo: 'bar'})
+      assert_equal('bar', ii.metadata['foo'])
+    end
+  end
+end

data/test/stripe/invoice_test.rb

@@ -23,6 +23,14 @@ module Stripe
       assert_equal nil, i.next_payment_attempt
     end
 
+    should "invoices should be updateable" do
+      @mock.expects(:post).once.
+        with("https://api.stripe.com/v1/invoices/test_invoice", nil, "metadata[foo]=bar").
+        returns(make_response(make_invoice(metadata: {foo: 'bar'})))
+      i = Stripe::Invoice.update("test_invoice", metadata: {foo: 'bar'})
+      assert_equal('bar', i.metadata['foo'])
+    end
+
     should "pay with extra opts should pay an invoice" do
       @mock.expects(:get).once.returns(make_response(make_invoice))
       i = Stripe::Invoice.retrieve('in_test_invoice', {:api_key => 'foobar'})
@@ -30,11 +38,29 @@ module Stripe
       Stripe.expects(:execute_request).with do |opts|
         opts[:url] == "#{Stripe.api_base}/v1/invoices/in_test_invoice/pay" &&
           opts[:method] == :post &&
-          opts[:headers][:authorization] == 'Bearer foobar'
+          opts[:headers]['Authorization'] == 'Bearer foobar'
       end.returns(make_response(make_paid_invoice))
 
       i.pay
       assert_equal nil, i.next_payment_attempt
     end
+
+    should "be able to retrieve upcoming invoices" do
+      base = "#{Stripe.api_base}/v1/invoices/upcoming?"
+      cus_sub = "customer=c_test_customer&subscription=s_test_subscription&"
+      item0 = "subscription_items[][plan]=gold&subscription_items[][quantity]=1&"
+      item1 = "subscription_items[][plan]=silver&subscription_items[][quantity]=2"
+      @mock.expects(:get).once.with(base + cus_sub + item0 + item1, nil, nil).
+        returns(make_response(make_invoice(:customer => 'c_test_customer', :subscription => 's_test_subscription')))
+
+      i = Stripe::Invoice.upcoming(
+        :customer => 'c_test_customer',
+        :subscription => 's_test_subscription',
+        :subscription_items => [{:plan => 'gold', :quantity =>1}, {:plan => 'silver', :quantity =>2}]
+      )
+
+      assert_equal 'c_test_customer', i.customer
+      assert_equal 's_test_subscription', i.subscription
+    end
   end
 end

data/test/stripe/list_object_test.rb

@@ -31,7 +31,11 @@ module Stripe
       ]
       expected = Util.convert_to_stripe_object(arr, {})
 
-      list = TestListObject.construct_from({ :data => [{ :id => 1 }], :has_more => true })
+      list = TestListObject.construct_from({
+        :data => [{ :id => 1 }],
+        :has_more => true,
+        :url => "/things",
+      })
       @mock.expects(:get).once.with("#{Stripe.api_base}/things?starting_after=1", nil, nil).
         returns(make_response({ :data => [{ :id => 2 }, { :id => 3}], :has_more => false }))
 
@@ -46,7 +50,11 @@ module Stripe
       ]
       expected = Util.convert_to_stripe_object(arr, {})
 
-      list = TestListObject.construct_from({ :data => [{ :id => 1 }], :has_more => true })
+      list = TestListObject.construct_from({
+        :data => [{ :id => 1 }],
+        :has_more => true,
+        :url => "/things",
+      })
       @mock.expects(:get).once.with("#{Stripe.api_base}/things?starting_after=1", nil, nil).
         returns(make_response({ :data => [{ :id => 2 }, { :id => 3}], :has_more => false }))
 
@@ -70,7 +78,11 @@ module Stripe
     #
 
     should "fetch a next page through #next_page" do
-      list = TestListObject.construct_from({ :data => [{ :id => 1 }], :has_more => true })
+      list = TestListObject.construct_from({
+        :data => [{ :id => 1 }],
+        :has_more => true,
+        :url => "/things",
+      })
       @mock.expects(:get).once.with("#{Stripe.api_base}/things?starting_after=1", nil, nil).
         returns(make_response({ :data => [{ :id => 2 }], :has_more => false }))
       next_list = list.next_page
@@ -78,13 +90,15 @@ module Stripe
     end
 
     should "fetch a next page through #next_page and respect limit" do
-      list = TestListObject.construct_from({ :data => [{ :id => 1 }], :has_more => true })
+      list = TestListObject.construct_from({
+        :data => [{ :id => 1 }],
+        :has_more => true,
+        :url => "/things",
+      })
       list.filters = { :expand => ['data.source'], :limit => 3 }
       @mock.expects(:get).with do |url, _, _|
-        # apparently URI.parse in 1.8.7 doesn't support query parameters ...
-        url, query = url.split("?")
         u = URI.parse(url)
-        params = CGI.parse(query)
+        params = CGI.parse(u.query)
         u.host == URI.parse(Stripe.api_base).host && u.path == "/things" && params == {
           "expand[]"       => ["data.source"],
           "limit"          => ["3"],
@@ -96,7 +110,11 @@ module Stripe
     end
 
     should "fetch an empty page through #next_page" do
-      list = TestListObject.construct_from({ :data => [{ :id => 1 }], :has_more => false })
+      list = TestListObject.construct_from({
+        :data => [{ :id => 1 }],
+        :has_more => false,
+        :url => "/things",
+      })
       next_list = list.next_page
       assert_equal Stripe::ListObject.empty_list, next_list
     end
@@ -106,7 +124,10 @@ module Stripe
     #
 
     should "fetch a next page through #previous_page" do
-      list = TestListObject.construct_from({ :data => [{ :id => 2 }] })
+      list = TestListObject.construct_from({
+        :data => [{ :id => 2 }],
+        :url => "/things",
+      })
       @mock.expects(:get).once.with("#{Stripe.api_base}/things?ending_before=2", nil, nil).
         returns(make_response({ :data => [{ :id => 1 }] }))
       next_list = list.previous_page
@@ -114,7 +135,10 @@ module Stripe
     end
 
     should "fetch a next page through #previous_page and respect limit" do
-      list = TestListObject.construct_from({ :data => [{ :id => 2 }] })
+      list = TestListObject.construct_from({
+        :data => [{ :id => 2 }],
+        :url => "/things",
+      })
       list.filters = { :expand => ['data.source'], :limit => 3 }
       @mock.expects(:get).with do |url, _, _|
         # apparently URI.parse in 1.8.7 doesn't support query parameters ...
@@ -141,10 +165,10 @@ module Stripe
       @mock.expects(:get).twice.returns(make_response(make_charge_array))
       c = Stripe::Charge.all
       assert c.kind_of?(Stripe::ListObject)
-      assert_equal('/v1/charges', c.url)
+      assert_equal('/v1/charges', c.resource_url)
       all = c.all
       assert all.kind_of?(Stripe::ListObject)
-      assert_equal('/v1/charges', all.url)
+      assert_equal('/v1/charges', all.resource_url)
       assert all.data.kind_of?(Array)
     end
   end
@@ -152,7 +176,4 @@ end
 
 # A helper class with a URL that allows us to try out pagination.
 class TestListObject < Stripe::ListObject
-  def url
-    "/things"
-  end
 end

data/test/stripe/order_return_test.rb

@@ -0,0 +1,25 @@
+require File.expand_path('../../test_helper', __FILE__)
+
+module Stripe
+  class OrderReturnTest < Test::Unit::TestCase
+    should "returns should be listable" do
+      @mock.expects(:get).once.returns(make_response(make_order_return_array))
+      returns = Stripe::OrderReturn.list
+      assert returns.data.kind_of?(Array)
+      returns.each do |ret|
+        assert ret.kind_of?(Stripe::OrderReturn)
+      end
+    end
+
+    should "returns should not be deletable" do
+      p = Stripe::OrderReturn.new("test_order")
+      assert_raises(NoMethodError) { p.delete }
+    end
+
+    should "returns should be immutable" do
+      p = Stripe::OrderReturn.new("test_order")
+      p.items = []
+      assert_raises(NoMethodError) { p.save }
+    end
+  end
+end

data/test/stripe/order_test.rb

@@ -19,7 +19,7 @@ module Stripe
       end
     end
 
-    should "orders should be updateable" do
+    should "orders should be saveable" do
       @mock.expects(:get).once.returns(make_response(make_order))
       @mock.expects(:post).once.returns(make_response(make_order))
       p = Stripe::Order.new("test_order")
@@ -28,6 +28,14 @@ module Stripe
       p.save
     end
 
+    should "orders should be updateable" do
+      @mock.expects(:post).once.
+        with('https://api.stripe.com/v1/orders/test_order', nil, 'status=fulfilled').
+        returns(make_response(make_order(status: 'fulfilled')))
+      ii = Stripe::Order.update("test_order", status: 'fulfilled')
+      assert_equal('fulfilled', ii.status)
+    end
+
     should "orders should allow metadata updates" do
       @mock.expects(:get).once.returns(make_response(make_order))
       @mock.expects(:post).once.returns(make_response(make_order))
@@ -48,5 +56,17 @@ module Stripe
       order.pay(:token => 'test_token')
       assert_equal "paid", order.status
     end
+
+    should "return an order" do
+      @mock.expects(:get).once.
+        returns(make_response(make_order(:id => 'or_test_order')))
+      order = Stripe::Order.retrieve('or_test_order')
+
+      @mock.expects(:post).once.
+        with('https://api.stripe.com/v1/orders/or_test_order/returns', nil, 'items[][parent]=sku_foo').
+        returns(make_response(make_order_return({:order => order.id})))
+      order_return = order.return_order(:items => [{:parent => 'sku_foo'}])
+      assert_equal order.id, order_return.order
+    end
   end
 end

data/test/stripe/plan_test.rb

@@ -0,0 +1,31 @@
+require File.expand_path('../../test_helper', __FILE__)
+
+module Stripe
+  class PlanTest < Test::Unit::TestCase
+    should "plans should be listable" do
+      @mock.expects(:get).once.returns(make_response(make_plan_array))
+      plans = Stripe::Plan.list
+      assert plans.data.kind_of?(Array)
+      plans.each do |plan|
+        assert plan.kind_of?(Stripe::Plan)
+      end
+    end
+
+    should "plans should be saveable" do
+      @mock.expects(:get).once.returns(make_response(make_plan))
+      @mock.expects(:post).once.returns(make_response(make_plan))
+      p = Stripe::Plan.new("test_plan")
+      p.refresh
+      p.metadata['foo'] = 'bar'
+      p.save
+    end
+
+    should "plans should be updateable" do
+      @mock.expects(:post).once.
+        with('https://api.stripe.com/v1/plans/test_plan', nil, 'metadata[foo]=bar').
+        returns(make_response(make_plan(metadata: {foo: 'bar'})))
+      p = Stripe::Plan.update("test_plan", metadata: {foo: 'bar'})
+      assert_equal('bar', p.metadata['foo'])
+    end
+  end
+end

data/test/stripe/product_test.rb

@@ -11,15 +11,17 @@ module Stripe
       end
     end
 
-    should "products should not be deletable" do
-      assert_raises NoMethodError do
-        @mock.expects(:get).once.returns(make_response(make_product))
-        p = Stripe::Product.retrieve("test_product")
-        p.delete
-      end
+    should "products should be deletable" do
+      @mock.expects(:get).once.returns(make_response(make_product))
+      @mock.expects(:delete).once.
+        returns(make_response(make_product(:deleted => true)))
+
+      p = Stripe::Product.retrieve("test_product")
+      p.delete
+      assert p.deleted
     end
 
-    should "products should be updateable" do
+    should "products should be saveable" do
       @mock.expects(:get).once.returns(make_response(make_product))
       @mock.expects(:post).once.returns(make_response(make_product))
       p = Stripe::Product.new("test_product")
@@ -28,6 +30,14 @@ module Stripe
       p.save
     end
 
+    should "products should be updateable" do
+      @mock.expects(:post).once.
+        with('https://api.stripe.com/v1/products/test_product', nil, 'description=update').
+        returns(make_response(make_product(description: 'update')))
+      p = Stripe::Product.update("test_product", description: 'update')
+      assert_equal('update', p.description)
+    end
+
     should "products should allow metadata updates" do
       @mock.expects(:get).once.returns(make_response(make_product))
       @mock.expects(:post).once.returns(make_response(make_product))

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/recipient_test.rb

@@ -0,0 +1,21 @@
+require File.expand_path('../../test_helper', __FILE__)
+
+module Stripe
+  class RecipientTest < Test::Unit::TestCase
+
+    should "recipient should be retrievable" do
+      @mock.expects(:get).once.returns(make_response(make_recipient))
+      r = Stripe::Recipient.retrieve('test_recipient')
+      assert_equal 'rp_test_recipient', r.id
+    end
+
+    should "recipient should be updateable" do
+      @mock.expects(:post).once.
+        with("https://api.stripe.com/v1/recipients/test_recipient", nil, "metadata[foo]=bar").
+        returns(make_response(make_recipient(metadata: {foo: 'bar'})))
+      r = Stripe::Recipient.update('test_recipient', metadata: {foo: 'bar'})
+      assert_equal 'bar', r.metadata['foo']
+    end
+  end
+end
+

data/test/stripe/refund_test.rb

@@ -24,7 +24,7 @@ module Stripe
       assert_equal 'refreshed_refund', refund.id
     end
 
-    should "refunds should be updateable" do
+    should "refunds should be saveable" do
       @mock.expects(:get).
         with("#{Stripe.api_base}/v1/refunds/get_refund", nil, nil).
         once.returns(make_response(make_refund(:id => 'save_refund')))
@@ -43,6 +43,15 @@ module Stripe
       assert_equal 'value', refund.metadata['key']
     end
 
+    should "refunds should be updateable" do
+      @mock.expects(:post).
+        with("#{Stripe.api_base}/v1/refunds/update_refund", nil, 'metadata[key]=value').
+        once.returns(make_response(make_refund(:metadata => {'key' => 'value'})))
+
+      refund = Stripe::Refund.update('update_refund', metadata: {key: 'value'})
+      assert_equal 'value', refund.metadata['key']
+    end
+
     should "create should return a new refund" do
       @mock.expects(:post).
         with("#{Stripe.api_base}/v1/refunds", nil, 'charge=test_charge').

data/test/stripe/sku_test.rb

@@ -12,12 +12,21 @@ module Stripe
       end
     end
 
-    should "SKUs should not be deletable" do
-      assert_raises NoMethodError do
-        @mock.expects(:get).once.returns(make_response(make_sku))
-        p = Stripe::SKU.retrieve("test_product")
-        p.delete
-      end
+    should "SKUs should be updateable" do
+      @mock.expects(:post).once.
+        with("#{Stripe.api_base}/v1/skus/test_sku", nil, 'metadata[foo]=bar').
+        returns(make_response(make_sku(:metadata => {foo: 'bar'})))
+      s = Stripe::SKU.update("test_sku", metadata: {foo: 'bar'})
+      assert_equal 'bar', s.metadata['foo']
+    end
+
+    should "SKUs should be deletable" do
+      @mock.expects(:get).once.returns(make_response(make_sku))
+      @mock.expects(:delete).once.returns(make_response(make_sku(:deleted => true)))
+
+      s = Stripe::SKU.retrieve("test_sku")
+      s.delete
+      assert s.deleted
     end
 
   end

data/test/stripe/source_test.rb

@@ -0,0 +1,83 @@
+require File.expand_path('../../test_helper', __FILE__)
+
+module Stripe
+  class SourceTest < Test::Unit::TestCase
+    should 'be creatable' do
+      @mock.expects(:post).once.returns(make_response(make_source_card))
+      src = Stripe::Source.create(
+        type: 'card',
+        token: 'tok_test',
+      )
+      assert_equal 'src_test_card', src.id
+    end
+
+    should 'be retrievable' do
+      @mock.expects(:get).once.returns(make_response(make_source_card))
+      src = Stripe::Source.retrieve('src_test_card')
+      assert_equal 'src_test_card', src.id
+    end
+
+    should 'be updatable' do
+      @mock.expects(:post).once
+        .with(
+          "#{Stripe.api_base}/v1/sources/src_test_card",
+          nil,
+          'metadata[foo]=bar'
+        )
+        .returns(make_response(make_source_card(metadata: {foo: 'bar'})))
+      src = Stripe::Source.update('src_test_card', metadata: {foo: 'bar'})
+      assert_equal 'bar', src.metadata['foo']
+    end
+
+    should 'be saveable' do
+      @mock.expects(:get).once.returns(make_response(make_source_card))
+      src = Stripe::Source.retrieve('src_test_card')
+
+      @mock.expects(:post).once
+        .with(
+          "#{Stripe.api_base}/v1/sources/src_test_card",
+          nil,
+          'metadata[foo]=bar'
+        )
+        .returns(make_response(make_source_card(metadata: {foo: 'bar'})))
+      src.metadata['foo'] = 'bar'
+      src.save
+      assert_equal 'bar', src.metadata['foo']
+    end
+
+    should 'not be deletable' do
+      @mock.expects(:get).once.returns(make_response(make_source_card))
+      src = Stripe::Source.retrieve('src_test_card')
+
+      assert_raises NoMethodError do
+        src.delete
+      end
+    end
+
+    should 'not be listable' do
+      assert_raises NoMethodError do
+        Stripe::Source.list
+      end
+    end
+
+    should 'be verifiable' do
+      @mock.expects(:get).once.returns(make_response(make_source_ach_debit))
+      src = Stripe::Source.retrieve('src_test_ach_debit')
+
+      @mock.expects(:post).once
+        .with(
+          'https://api.stripe.com/v1/sources/src_test_ach_debit/verify',
+          nil,
+          'values[]=32&values[]=45'
+        )
+        .returns(make_response(make_source_ach_debit(
+          verification: {
+            attempts_remaining: 0,
+            status: 'succeeded',
+          }
+        )))
+      src.verify(values: [32, 45])
+      assert_equal 'succeeded', src.verification.status
+    end
+  end
+end