stripe 1.58.0 → 2.0.0

data/test/stripe/country_spec_test.rb

@@ -2,42 +2,19 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class CountrySpecTest < Test::Unit::TestCase
+    FIXTURE = API_FIXTURES.fetch(:country_spec)
+
     should "be listable" do
-      @mock.expects(:get).once.
-      returns(make_response(country_spec_array))
       c = Stripe::CountrySpec.list
-
-      assert_equal('/v1/country_specs', c.resource_url)
-      assert_equal('list', c.object)
-      assert(c.data.kind_of?(Array))
-      assert_equal('US', c.data[0].id)
-      assert(c.data[0].kind_of?(Stripe::CountrySpec))
+      assert_requested :get, "#{Stripe.api_base}/v1/country_specs"
+      assert c.data.kind_of?(Array)
+      assert c.data[0].kind_of?(Stripe::CountrySpec)
     end
 
     should "be retrievable" do
-      resp = make_country_spec
-      @mock.expects(:get).once.
-      with('https://api.stripe.com/v1/country_specs/US', nil, nil).
-      returns(make_response(resp))
-      s = Stripe::CountrySpec.retrieve('US')
-
-      assert_equal('/v1/country_specs/US', s.resource_url)
-      assert_equal('country_spec', s.object)
+      s = Stripe::CountrySpec.retrieve(FIXTURE[:id])
+      assert_requested :get, "#{Stripe.api_base}/v1/country_specs/#{FIXTURE[:id]}"
       assert(s.kind_of?(Stripe::CountrySpec))
-
-      s.supported_bank_account_currencies.map{ |k,v|  assert v.kind_of?(Array) }
-      assert_equal(['US'], s.supported_bank_account_currencies['usd'])
-      assert(s.supported_payment_currencies.include?('usd'))
-      assert s.supported_payment_currencies.kind_of?(Array)
-      assert s.supported_payment_methods.kind_of?(Array)
-
-      ['individual', 'company'].map{ |type|
-        item = s.verification_fields[type]
-        assert item.minimum.include?('external_account')
-        assert item.additional.include?('legal_entity.verification.document')
-        assert item.additional.kind_of?(Array)
-        assert item.minimum.kind_of?(Array)
-      }
     end
   end
 end

data/test/stripe/coupon_test.rb

@@ -2,27 +2,43 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class CouponTest < Test::Unit::TestCase
-    should "create should return a new coupon" do
-      @mock.expects(:post).once.returns(make_response(make_coupon))
-      c = Stripe::Coupon.create
-      assert_equal "co_test_coupon", c.id
+    FIXTURE = API_FIXTURES.fetch(:coupon)
+
+    should "be listable" do
+      coupons = Stripe::Coupon.list
+      assert_requested :get, "#{Stripe.api_base}/v1/coupons"
+      assert coupons.data.kind_of?(Array)
+      assert coupons.first.kind_of?(Stripe::Coupon)
+    end
+
+    should "be retrievable" do
+      coupon = Stripe::Coupon.retrieve(FIXTURE[:id])
+      assert_requested :get, "#{Stripe.api_base}/v1/coupons/#{FIXTURE[:id]}"
+      assert coupon.kind_of?(Stripe::Coupon)
+    end
+
+    should "be creatable" do
+      coupon = Stripe::Coupon.create(
+        percent_off: 25,
+        duration: 'repeating',
+        duration_in_months: 3,
+        id: '25OFF'
+      )
+      assert_requested :post, "#{Stripe.api_base}/v1/coupons"
+      assert coupon.kind_of?(Stripe::Coupon)
     end
 
-    should "coupons should be saveable" do
-      @mock.expects(:get).once.returns(make_response(make_coupon))
-      @mock.expects(:post).once.returns(make_response(make_coupon))
-      c = Stripe::Coupon.new("test_coupon")
-      c.refresh
-      c.metadata['foo'] = 'bar'
-      c.save
+    should "be saveable" do
+      coupon = Stripe::Coupon.retrieve(FIXTURE[:id])
+      coupon.metadata['key'] = 'value'
+      coupon.save
+      assert_requested :post, "#{Stripe.api_base}/v1/coupons/#{FIXTURE[:id]}"
     end
 
-    should "coupons should be updateable" do
-      @mock.expects(:post).once.
-        with("https://api.stripe.com/v1/coupons/test_coupon", nil, "metadata[foo]=bar").
-        returns(make_response(make_coupon(metadata: {foo: 'bar'})))
-      c = Stripe::Coupon.update("test_coupon", metadata: {foo: 'bar'})
-      assert_equal('bar', c.metadata['foo'])
+    should "be updateable" do
+      coupon = Stripe::Coupon.update(FIXTURE[:id], metadata: { key: 'value' })
+      assert_requested :post, "#{Stripe.api_base}/v1/coupons/#{FIXTURE[:id]}"
+      assert coupon.kind_of?(Stripe::Coupon)
     end
   end
 end

data/test/stripe/customer_card_test.rb

@@ -2,62 +2,41 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class CustomerCardTest < Test::Unit::TestCase
-    CUSTOMER_CARD_URL = '/v1/customers/test_customer/sources/test_card'
+    FIXTURE = API_FIXTURES.fetch(:source)
 
-    def customer
-      @mock.expects(:get).once.returns(make_response(make_customer))
-      Stripe::Customer.retrieve('test_customer')
+    setup do
+      @customer =
+        Stripe::Customer.retrieve(API_FIXTURES.fetch(:customer)[:id])
     end
 
-    should "customer cards should be listable" do
-      c = customer
-      @mock.expects(:get).once.returns(make_response(make_customer_card_array(customer.id)))
-      cards = c.sources.list(:object => "card").data
-      assert cards.kind_of? Array
-      assert cards[0].kind_of? Stripe::Card
+    should "be listable" do
+      sources = @customer.sources.list
+      assert sources.data.kind_of?(Array)
+      # because of the terrible :wildcard nature of sources, the API stub
+      # cannot currently replace this response with anything meaningful so we
+      # don't assert on the type of individual items like we do in other tests
     end
 
-    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.resource_url
+    should "be creatable" do
+      card = @customer.sources.create(
+        source: API_FIXTURES.fetch(:token)[:id]
+      )
+      assert_requested :post, "#{Stripe.api_base}/v1/customers/#{@customer.id}/sources"
+      assert card.kind_of?(Stripe::BankAccount)
     end
 
-    should "customer cards should be deletable" do
-      c = customer
-      @mock.expects(:get).once.returns(make_response(make_card))
-      @mock.expects(:delete).once.returns(make_response(make_card(:deleted => true)))
-      card = c.sources.retrieve('card')
-      card.delete
-      assert card.deleted
+    should "be deletable" do
+      card = Stripe::Card.construct_from(FIXTURE.merge(customer: @customer.id))
+      card = card.delete
+      assert_requested :delete, "#{Stripe.api_base}/v1/customers/#{@customer.id}/sources/#{FIXTURE[:id]}"
+      assert card.kind_of?(Stripe::Card)
     end
 
-    should "customer cards should be updateable" do
-      c = customer
-      @mock.expects(:get).once.returns(make_response(make_card(:exp_year => "2000")))
-      @mock.expects(:post).once.returns(make_response(make_card(:exp_year => "2100")))
-      card = c.sources.retrieve('card')
-      assert_equal "2000", card.exp_year
-      card.exp_year = "2100"
+    should "be saveable" do
+      card = Stripe::Card.construct_from(FIXTURE.merge(customer: @customer.id))
+      card.metadata['key'] = 'value'
       card.save
-      assert_equal "2100", card.exp_year
-    end
-
-    should "create should return a new customer card" do
-      c = customer
-      @mock.expects(:post).once.returns(make_response(make_card(:id => "test_card")))
-      card = c.sources.create(:source => "tok_41YJ05ijAaWaFS")
-      assert_equal "test_card", card.id
-    end
-
-    should "raise if accessing Stripe::Card.retrieve directly" do
-      assert_raises NotImplementedError do
-        Stripe::Card.retrieve "card_12345"
-      end
+      assert_requested :post, "#{Stripe.api_base}/v1/customers/#{@customer.id}/sources/#{FIXTURE[:id]}"
     end
   end
 end

data/test/stripe/customer_test.rb

@@ -2,109 +2,114 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class CustomerTest < Test::Unit::TestCase
-    should "customers should be listable" do
-      @mock.expects(:get).once.returns(make_response(make_customer_array))
-      c = Stripe::Customer.list.data
-      assert c.kind_of? Array
-      assert c[0].kind_of? Stripe::Customer
-    end
+    FIXTURE = API_FIXTURES.fetch(:customer)
 
-    should "customers should be deletable" do
-      @mock.expects(:delete).once.returns(make_response(make_customer({:deleted => true})))
-      c = Stripe::Customer.new("test_customer")
-      c.delete
-      assert c.deleted
+    should "be listable" do
+      customers = Stripe::Customer.list
+      assert_requested :get, "#{Stripe.api_base}/v1/customers"
+      assert customers.data.kind_of?(Array)
+      assert customers.first.kind_of?(Stripe::Customer)
     end
 
-    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
-      assert_equal "foo", c.mnemonic
-      c.mnemonic = "bar"
-      c.save
-      assert_equal "bar", c.mnemonic
+    should "be retrievable" do
+      customer = Stripe::Customer.retrieve(FIXTURE[:id])
+      assert_requested :get, "#{Stripe.api_base}/v1/customers/#{FIXTURE[:id]}"
+      assert customer.kind_of?(Stripe::Customer)
     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'])
+    should "be creatable" do
+      customer = Stripe::Customer.create
+      assert_requested :post, "#{Stripe.api_base}/v1/customers"
+      assert customer.kind_of?(Stripe::Customer)
     end
 
-    should "create should return a new customer" do
-      @mock.expects(:post).once.returns(make_response(make_customer))
-      c = Stripe::Customer.create
-      assert_equal "c_test_customer", c.id
+    should "be saveable" do
+      customer = Stripe::Customer.retrieve(FIXTURE[:id])
+      customer.metadata['key'] = 'value'
+      customer.save
+      assert_requested :post, "#{Stripe.api_base}/v1/customers/#{FIXTURE[:id]}"
     end
 
-    should "create_upcoming_invoice should create a new invoice" do
-      @mock.expects(:post).once.returns(make_response(make_invoice))
-      i = Stripe::Customer.new("test_customer").create_upcoming_invoice
-      assert_equal "c_test_customer", i.customer
+    should "be updateable" do
+      customer = Stripe::Customer.update(FIXTURE[:id], metadata: { key: 'value' })
+      assert_requested :post, "#{Stripe.api_base}/v1/customers/#{FIXTURE[:id]}"
+      assert customer.kind_of?(Stripe::Customer)
     end
 
-    should "be able to update a customer's subscription" do
-      @mock.expects(:get).once.returns(make_response(make_customer))
-      c = Stripe::Customer.retrieve("test_customer")
-
-      @mock.expects(:post).once.with do |url, api_key, params|
-        url == "#{Stripe.api_base}/v1/customers/c_test_customer/subscription" && api_key.nil? && CGI.parse(params) == {'plan' => ['silver']}
-      end.returns(make_response(make_subscription(:plan => 'silver')))
-      s = c.update_subscription({:plan => 'silver'})
-
-      assert_equal 'subscription', s.object
-      assert_equal 'silver', s.plan.identifier
+    should "be deletable" do
+      customer = Stripe::Customer.retrieve(FIXTURE[:id])
+      customer = customer.delete
+      assert_requested :delete, "#{Stripe.api_base}/v1/customers/#{FIXTURE[:id]}"
+      assert customer.kind_of?(Stripe::Customer)
     end
 
-    should "be able to cancel a customer's subscription" do
-      @mock.expects(:get).once.returns(make_response(make_customer))
-      c = Stripe::Customer.retrieve("test_customer")
-
-      # Not an accurate response, but whatever
-
-      @mock.expects(:delete).once.with("#{Stripe.api_base}/v1/customers/c_test_customer/subscription?at_period_end=true", nil, nil).returns(make_response(make_subscription(:plan => 'silver')))
-      c.cancel_subscription({:at_period_end => 'true'})
-
-      @mock.expects(:delete).once.with("#{Stripe.api_base}/v1/customers/c_test_customer/subscription", nil, nil).returns(make_response(make_subscription(:plan => 'silver')))
-      c.cancel_subscription
+    context "#create_subscription" do
+      should "create a new subscription" do
+        customer = Stripe::Customer.retrieve(FIXTURE[:id])
+        subscription = customer.create_subscription({:plan => 'silver'})
+        assert subscription.kind_of?(Stripe::Subscription)
+      end
     end
 
-    should "be able to create a subscription for a customer" do
-      c = Stripe::Customer.new("test_customer")
-
-      @mock.expects(:post).once.with do |url, api_key, params|
-        url == "#{Stripe.api_base}/v1/customers/test_customer/subscriptions" && api_key.nil? && CGI.parse(params) == {'plan' => ['silver']}
-      end.returns(make_response(make_subscription(:plan => 'silver')))
-      s = c.create_subscription({:plan => 'silver'})
-
-      assert_equal 'subscription', s.object
-      assert_equal 'silver', s.plan.identifier
+    context "#create_upcoming_invoice" do
+      should "create a new invoice" do
+        customer = Stripe::Customer.retrieve(FIXTURE[:id])
+        invoice = customer.create_upcoming_invoice
+        assert invoice.kind_of?(Stripe::Invoice)
+      end
     end
 
-    should "be able to delete a customer's discount" do
-      @mock.expects(:get).once.returns(make_response(make_customer))
-      c = Stripe::Customer.retrieve("test_customer")
+    context "#update_subscription" do
+      should "update a subscription" do
+        customer = Stripe::Customer.retrieve(FIXTURE[:id])
+
+        # deprecated API and not in schema
+        stub_request(:post, "#{Stripe.api_base}/v1/customers/#{customer.id}/subscription").
+          with(body: { plan: "silver" }).
+          to_return(body: JSON.generate(API_FIXTURES[:subscription]))
+        subscription = customer.update_subscription({:plan => 'silver'})
+        assert subscription.kind_of?(Stripe::Subscription)
+      end
+    end
 
-      @mock.expects(:delete).once.with("#{Stripe.api_base}/v1/customers/c_test_customer/discount", nil, nil).returns(make_response(make_delete_discount_response))
-      c.delete_discount
-      assert_equal nil, c.discount
+    context "#cancel_subscription" do
+      should "cancel a subscription" do
+        customer = Stripe::Customer.retrieve(FIXTURE[:id])
+
+        # deprecated API and not in schema
+        stub_request(:delete, "#{Stripe.api_base}/v1/customers/#{customer.id}/subscription").
+          with(query: { at_period_end: "true" }).
+          to_return(body: JSON.generate(API_FIXTURES[:subscription]))
+        subscription = customer.cancel_subscription({:at_period_end => 'true'})
+        assert subscription.kind_of?(Stripe::Subscription)
+      end
     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
+    context "#delete_discount" do
+      should "delete a discount" do
+        customer = Stripe::Customer.retrieve(FIXTURE[:id])
+
+        stub_request(:delete, "#{Stripe.api_base}/v1/customers/#{customer.id}/discount").
+          to_return(body: JSON.generate(API_FIXTURES[:discount]))
+        discount = customer.delete_discount
+        assert discount.kind_of?(Stripe::Customer)
+      end
     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
+    context "source field" do
+      should "allow setting source with token" do
+        c = Stripe::Customer.new("test_customer")
+        c.source = "tok_123"
+        assert_equal "tok_123", c.source
+      end
+
+      should "allow setting source with hash and set flag" do
+        c = Stripe::Customer.new("test_customer")
+        c.source = {
+          :object => 'card'
+        }
+        assert_equal true, c.source.save_with_parent
+      end
     end
   end
 end

data/test/stripe/dispute_test.rb

@@ -2,52 +2,41 @@ require File.expand_path('../../test_helper', __FILE__)
 
 module Stripe
   class DisputeTest < Test::Unit::TestCase
-    should "disputes should be retrievable" do
-      @mock.expects(:get).once.returns(make_response(make_dispute))
-      d = Stripe::Dispute.retrieve('dp_test_dispute')
-      assert d.kind_of?(Stripe::Dispute)
-    end
+    FIXTURE = API_FIXTURES.fetch(:dispute)
 
-    should "disputes should be listable" do
-      @mock.expects(:get).once.returns(make_response(make_dispute_array))
-      d = Stripe::Dispute.list
-      assert d.data.kind_of? Array
-      d.each do |dispute|
-        assert dispute.kind_of?(Stripe::Dispute)
-      end
+    should "be listable" do
+      disputes = Stripe::Dispute.list
+      assert_requested :get, "#{Stripe.api_base}/v1/disputes"
+      assert disputes.data.kind_of?(Array)
+      assert disputes.first.kind_of?(Stripe::Dispute)
     end
 
-    should "disputes should be closeable" do
-      @mock.expects(:get).never
-      @mock.expects(:post).with(
-        "#{Stripe.api_base}/v1/disputes/test_dispute/close",
-        nil,
-        ''
-      ).once.returns(make_response({:id => 'dp_test_dispute', :status => 'lost'}))
-      d = Stripe::Dispute.new('test_dispute')
-      d.close
+    should "be retrievable" do
+      dispute = Stripe::Dispute.retrieve(FIXTURE[:id])
+      assert_requested :get, "#{Stripe.api_base}/v1/disputes/#{FIXTURE[:id]}"
+      assert dispute.kind_of?(Stripe::Dispute)
     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'])
+    should "be saveable" do
+      dispute = Stripe::Dispute.retrieve(FIXTURE[:id])
+      dispute.metadata['key'] = 'value'
+      dispute.save
+      assert_requested :post, "#{Stripe.api_base}/v1/disputes/#{FIXTURE[:id]}"
     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",
-        nil,
-        'evidence[customer_name]=customer'
-      ).once.returns(make_response(make_dispute))
+    should "be updateable" do
+      dispute = Stripe::Dispute.update(FIXTURE[:id], metadata: { key: 'value' })
+      assert_requested :post, "#{Stripe.api_base}/v1/disputes/#{FIXTURE[:id]}"
+      assert dispute.kind_of?(Stripe::Dispute)
+    end
 
-      d = Stripe::Dispute.new('test_dispute')
-      d.refresh
-      d.evidence['customer_name'] = 'customer'
-      d.save
+    context "#close" do
+      should "be closeable" do
+        dispute = Stripe::Dispute.retrieve(FIXTURE[:id])
+        dispute.close
+        assert_requested :post,
+          "#{Stripe.api_base}/v1/disputes/#{FIXTURE[:id]}/close"
+      end
     end
   end
 end