stripe 3.3.2 → 3.4.1

data/test/stripe/apple_pay_domain_test.rb

@@ -1,31 +1,31 @@
-require File.expand_path('../../test_helper', __FILE__)
+require File.expand_path("../../test_helper", __FILE__)
 
 module Stripe
   class ApplePayDomainTest < Test::Unit::TestCase
     should "be listable" do
       domains = Stripe::ApplePayDomain.list
       assert_requested :get, "#{Stripe.api_base}/v1/apple_pay/domains"
-      assert domains.data.kind_of?(Array)
-      assert domains.data[0].kind_of?(Stripe::ApplePayDomain)
+      assert domains.data.is_a?(Array)
+      assert domains.data[0].is_a?(Stripe::ApplePayDomain)
     end
 
     should "be retrievable" do
       domain = Stripe::ApplePayDomain.retrieve("apwc_123")
-    assert_requested :get, "#{Stripe.api_base}/v1/apple_pay/domains/apwc_123"
-      assert domain.kind_of?(Stripe::ApplePayDomain)
+      assert_requested :get, "#{Stripe.api_base}/v1/apple_pay/domains/apwc_123"
+      assert domain.is_a?(Stripe::ApplePayDomain)
     end
 
     should "be creatable" do
-      domain = Stripe::ApplePayDomain.create(:domain_name => "example.com")
+      domain = Stripe::ApplePayDomain.create(domain_name: "example.com")
       assert_requested :post, "#{Stripe.api_base}/v1/apple_pay/domains"
-      assert domain.kind_of?(Stripe::ApplePayDomain)
+      assert domain.is_a?(Stripe::ApplePayDomain)
     end
 
     should "be deletable" do
       domain = Stripe::ApplePayDomain.retrieve("apwc_123")
       domain = domain.delete
       assert_requested :delete, "#{Stripe.api_base}/v1/apple_pay/domains/#{domain.id}"
-      assert domain.kind_of?(Stripe::ApplePayDomain)
+      assert domain.is_a?(Stripe::ApplePayDomain)
     end
   end
 end

data/test/stripe/application_fee_refund_test.rb

@@ -1,4 +1,4 @@
-require File.expand_path('../../test_helper', __FILE__)
+require File.expand_path("../../test_helper", __FILE__)
 
 module Stripe
   class ApplicationFeeRefundTest < Test::Unit::TestCase
@@ -11,25 +11,25 @@ module Stripe
 
       # notably this *doesn't* make an API call
       assert_not_requested :get,
-        "#{Stripe.api_base}/v1/application_fees/#{@fee.id}/refunds"
+                           "#{Stripe.api_base}/v1/application_fees/#{@fee.id}/refunds"
 
-      assert refunds.data.kind_of?(Array)
-      assert refunds.first.kind_of?(Stripe::ApplicationFeeRefund)
+      assert refunds.data.is_a?(Array)
+      assert refunds.first.is_a?(Stripe::ApplicationFeeRefund)
     end
 
     should "be creatable" do
       refund = @fee.refunds.create
       assert_requested :post,
-        "#{Stripe.api_base}/v1/application_fees/#{@fee.id}/refunds"
-      assert refund.kind_of?(Stripe::ApplicationFeeRefund)
+                       "#{Stripe.api_base}/v1/application_fees/#{@fee.id}/refunds"
+      assert refund.is_a?(Stripe::ApplicationFeeRefund)
     end
 
     should "be saveable" do
       refund = @fee.refunds.first
-      refund.metadata['key'] = 'value'
+      refund.metadata["key"] = "value"
       refund.save
       assert_requested :post,
-        "#{Stripe.api_base}/v1/application_fees/#{@fee.id}/refunds/#{refund.id}"
+                       "#{Stripe.api_base}/v1/application_fees/#{@fee.id}/refunds/#{refund.id}"
     end
   end
 end

data/test/stripe/application_fee_test.rb

@@ -1,12 +1,12 @@
-require File.expand_path('../../test_helper', __FILE__)
+require File.expand_path("../../test_helper", __FILE__)
 
 module Stripe
   class ApplicationFeeTest < Test::Unit::TestCase
     should "be listable" do
       fees = Stripe::ApplicationFee.list
       assert_requested :get, "#{Stripe.api_base}/v1/application_fees"
-      assert fees.data.kind_of?(Array)
-      assert fees.data[0].kind_of?(Stripe::ApplicationFee)
+      assert fees.data.is_a?(Array)
+      assert fees.data[0].is_a?(Stripe::ApplicationFee)
     end
   end
 end

data/test/stripe/balance_test.rb

@@ -1,11 +1,11 @@
-require File.expand_path('../../test_helper', __FILE__)
+require File.expand_path("../../test_helper", __FILE__)
 
 module Stripe
   class BalanceTest < Test::Unit::TestCase
     should "be retrievable" do
       balance = Stripe::Balance.retrieve
       assert_requested :get, "#{Stripe.api_base}/v1/balance"
-      assert balance.kind_of?(Stripe::Balance)
+      assert balance.is_a?(Stripe::Balance)
     end
   end
 end

data/test/stripe/bank_account_test.rb

@@ -1,4 +1,4 @@
-require File.expand_path('../../test_helper', __FILE__)
+require File.expand_path("../../test_helper", __FILE__)
 
 module Stripe
   class BankAccountTest < Test::Unit::TestCase
@@ -9,27 +9,25 @@ module Stripe
           id: "ba_123"
         )
         assert_equal "/v1/accounts/acct_123/external_accounts/ba_123",
-          bank_account.resource_url
+                     bank_account.resource_url
       end
 
       should "return a customer URL" do
         bank_account = Stripe::BankAccount.construct_from(
           customer: "cus_123",
-          id: "ba_123",
+          id: "ba_123"
         )
         assert_equal "/v1/customers/cus_123/sources/ba_123",
-          bank_account.resource_url
+                     bank_account.resource_url
       end
     end
 
     context "#verify" do
-      should 'verify the account' do
-        bank_account = Stripe::BankAccount.construct_from({
-          customer: "cus_123",
-          id: "ba_123",
-        })
-        bank_account = bank_account.verify(amounts: [1,2])
-        assert bank_account.kind_of?(Stripe::BankAccount)
+      should "verify the account" do
+        bank_account = Stripe::BankAccount.construct_from(customer: "cus_123",
+                                                          id: "ba_123")
+        bank_account = bank_account.verify(amounts: [1, 2])
+        assert bank_account.is_a?(Stripe::BankAccount)
       end
     end
   end

data/test/stripe/charge_test.rb

@@ -1,48 +1,48 @@
-require File.expand_path('../../test_helper', __FILE__)
+require File.expand_path("../../test_helper", __FILE__)
 
 module Stripe
   class ChargeTest < Test::Unit::TestCase
     should "be listable" do
       charges = Stripe::Charge.list
       assert_requested :get, "#{Stripe.api_base}/v1/charges"
-      assert charges.data.kind_of?(Array)
-      assert charges.data[0].kind_of?(Stripe::Charge)
+      assert charges.data.is_a?(Array)
+      assert charges.data[0].is_a?(Stripe::Charge)
     end
 
     should "be retrievable" do
       charge = Stripe::Charge.retrieve("ch_123")
       assert_requested :get, "#{Stripe.api_base}/v1/charges/ch_123"
-      assert charge.kind_of?(Stripe::Charge)
+      assert charge.is_a?(Stripe::Charge)
     end
 
     should "be creatable" do
       charge = Stripe::Charge.create(
         amount: 100,
         currency: "USD",
-        source: "src_123",
+        source: "src_123"
       )
       assert_requested :post, "#{Stripe.api_base}/v1/charges"
-      assert charge.kind_of?(Stripe::Charge)
+      assert charge.is_a?(Stripe::Charge)
     end
 
     should "be saveable" do
       charge = Stripe::Charge.retrieve("ch_123")
-      charge.metadata['key'] = 'value'
+      charge.metadata["key"] = "value"
       charge.save
       assert_requested :post, "#{Stripe.api_base}/v1/charges/#{charge.id}"
     end
 
     should "be updateable" do
-      charge = Stripe::Charge.update("ch_123", metadata: {foo: 'bar'})
+      charge = Stripe::Charge.update("ch_123", metadata: { foo: "bar" })
       assert_requested :post, "#{Stripe.api_base}/v1/charges/ch_123"
-      assert charge.kind_of?(Stripe::Charge)
+      assert charge.is_a?(Stripe::Charge)
     end
 
     context "#mark_as_fraudulent" do
       should "charges should be able to be marked as fraudulent" do
         charge = Stripe::Charge.retrieve("ch_123")
         charge = charge.mark_as_fraudulent
-        assert charge.kind_of?(Stripe::Charge)
+        assert charge.is_a?(Stripe::Charge)
       end
     end
 
@@ -50,7 +50,7 @@ module Stripe
       should "charges should be able to be marked as safe" do
         charge = Stripe::Charge.retrieve("ch_123")
         charge = charge.mark_as_safe
-        assert charge.kind_of?(Stripe::Charge)
+        assert charge.is_a?(Stripe::Charge)
       end
     end
   end

data/test/stripe/country_spec_test.rb

@@ -1,18 +1,18 @@
-require File.expand_path('../../test_helper', __FILE__)
+require File.expand_path("../../test_helper", __FILE__)
 
 module Stripe
   class CountrySpecTest < Test::Unit::TestCase
     should "be listable" do
       c = Stripe::CountrySpec.list
       assert_requested :get, "#{Stripe.api_base}/v1/country_specs"
-      assert c.data.kind_of?(Array)
-      assert c.data[0].kind_of?(Stripe::CountrySpec)
+      assert c.data.is_a?(Array)
+      assert c.data[0].is_a?(Stripe::CountrySpec)
     end
 
     should "be retrievable" do
       s = Stripe::CountrySpec.retrieve("US")
       assert_requested :get, "#{Stripe.api_base}/v1/country_specs/US"
-      assert(s.kind_of?(Stripe::CountrySpec))
+      assert(s.is_a?(Stripe::CountrySpec))
     end
   end
 end

data/test/stripe/coupon_test.rb

@@ -1,42 +1,42 @@
-require File.expand_path('../../test_helper', __FILE__)
+require File.expand_path("../../test_helper", __FILE__)
 
 module Stripe
   class CouponTest < Test::Unit::TestCase
     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)
+      assert coupons.data.is_a?(Array)
+      assert coupons.first.is_a?(Stripe::Coupon)
     end
 
     should "be retrievable" do
       coupon = Stripe::Coupon.retrieve("25OFF")
       assert_requested :get, "#{Stripe.api_base}/v1/coupons/25OFF"
-      assert coupon.kind_of?(Stripe::Coupon)
+      assert coupon.is_a?(Stripe::Coupon)
     end
 
     should "be creatable" do
       coupon = Stripe::Coupon.create(
         percent_off: 25,
-        duration: 'repeating',
+        duration: "repeating",
         duration_in_months: 3,
-        id: '25OFF'
+        id: "25OFF"
       )
       assert_requested :post, "#{Stripe.api_base}/v1/coupons"
-      assert coupon.kind_of?(Stripe::Coupon)
+      assert coupon.is_a?(Stripe::Coupon)
     end
 
     should "be saveable" do
       coupon = Stripe::Coupon.retrieve("25OFF")
-      coupon.metadata['key'] = 'value'
+      coupon.metadata["key"] = "value"
       coupon.save
       assert_requested :post, "#{Stripe.api_base}/v1/coupons/#{coupon.id}"
     end
 
     should "be updateable" do
-      coupon = Stripe::Coupon.update("25OFF", metadata: { key: 'value' })
+      coupon = Stripe::Coupon.update("25OFF", metadata: { key: "value" })
       assert_requested :post, "#{Stripe.api_base}/v1/coupons/25OFF"
-      assert coupon.kind_of?(Stripe::Coupon)
+      assert coupon.is_a?(Stripe::Coupon)
     end
   end
 end

data/test/stripe/customer_card_test.rb

@@ -1,4 +1,4 @@
-require File.expand_path('../../test_helper', __FILE__)
+require File.expand_path("../../test_helper", __FILE__)
 
 module Stripe
   class CustomerCardTest < Test::Unit::TestCase
@@ -8,7 +8,7 @@ module Stripe
 
     should "be listable" do
       sources = @customer.sources.list
-      assert sources.data.kind_of?(Array)
+      assert sources.data.is_a?(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
@@ -16,29 +16,25 @@ module Stripe
 
     should "be creatable" do
       card = @customer.sources.create(
-        source: "tok_123",
+        source: "tok_123"
       )
       assert_requested :post, "#{Stripe.api_base}/v1/customers/#{@customer.id}/sources"
-      assert card.kind_of?(Stripe::BankAccount)
+      assert card.is_a?(Stripe::BankAccount)
     end
 
     should "be deletable" do
-      card = Stripe::Card.construct_from({
-        customer: @customer.id,
-        id: "card_123"
-      })
+      card = Stripe::Card.construct_from(customer: @customer.id,
+                                         id: "card_123")
       card = card.delete
       assert_requested :delete, "#{Stripe.api_base}/v1/customers/#{@customer.id}/sources/card_123"
-      assert card.kind_of?(Stripe::Card)
+      assert card.is_a?(Stripe::Card)
     end
 
     should "be saveable" do
-      card = Stripe::Card.construct_from({
-        customer: @customer.id,
-        id: "card_123",
-        metadata: {},
-      })
-      card.metadata['key'] = 'value'
+      card = Stripe::Card.construct_from(customer: @customer.id,
+                                         id: "card_123",
+                                         metadata: {})
+      card.metadata["key"] = "value"
       card.save
       assert_requested :post, "#{Stripe.api_base}/v1/customers/#{@customer.id}/sources/card_123"
     end

data/test/stripe/customer_test.rb

@@ -1,51 +1,51 @@
-require File.expand_path('../../test_helper', __FILE__)
+require File.expand_path("../../test_helper", __FILE__)
 
 module Stripe
   class CustomerTest < Test::Unit::TestCase
     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)
+      assert customers.data.is_a?(Array)
+      assert customers.first.is_a?(Stripe::Customer)
     end
 
     should "be retrievable" do
       customer = Stripe::Customer.retrieve("cus_123")
       assert_requested :get, "#{Stripe.api_base}/v1/customers/cus_123"
-      assert customer.kind_of?(Stripe::Customer)
+      assert customer.is_a?(Stripe::Customer)
     end
 
     should "be creatable" do
       customer = Stripe::Customer.create
       assert_requested :post, "#{Stripe.api_base}/v1/customers"
-      assert customer.kind_of?(Stripe::Customer)
+      assert customer.is_a?(Stripe::Customer)
     end
 
     should "be saveable" do
       customer = Stripe::Customer.retrieve("cus_123")
-      customer.metadata['key'] = 'value'
+      customer.metadata["key"] = "value"
       customer.save
       assert_requested :post, "#{Stripe.api_base}/v1/customers/#{customer.id}"
     end
 
     should "be updateable" do
-      customer = Stripe::Customer.update("cus_123", metadata: { key: 'value' })
+      customer = Stripe::Customer.update("cus_123", metadata: { key: "value" })
       assert_requested :post, "#{Stripe.api_base}/v1/customers/cus_123"
-      assert customer.kind_of?(Stripe::Customer)
+      assert customer.is_a?(Stripe::Customer)
     end
 
     should "be deletable" do
       customer = Stripe::Customer.retrieve("cus_123")
       customer = customer.delete
       assert_requested :delete, "#{Stripe.api_base}/v1/customers/#{customer.id}"
-      assert customer.kind_of?(Stripe::Customer)
+      assert customer.is_a?(Stripe::Customer)
     end
 
     context "#create_subscription" do
       should "create a new subscription" do
         customer = Stripe::Customer.retrieve("cus_123")
-        subscription = customer.create_subscription({:plan => 'silver'})
-        assert subscription.kind_of?(Stripe::Subscription)
+        subscription = customer.create_subscription(plan: "silver")
+        assert subscription.is_a?(Stripe::Subscription)
       end
     end
 
@@ -53,7 +53,7 @@ module Stripe
       should "create a new invoice" do
         customer = Stripe::Customer.retrieve("cus_123")
         invoice = customer.create_upcoming_invoice
-        assert invoice.kind_of?(Stripe::Invoice)
+        assert invoice.is_a?(Stripe::Invoice)
       end
     end
 
@@ -62,11 +62,11 @@ module Stripe
         customer = Stripe::Customer.retrieve("cus_123")
 
         # 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({ object: "subscription" }))
-        subscription = customer.update_subscription({:plan => 'silver'})
-        assert subscription.kind_of?(Stripe::Subscription)
+        stub_request(:post, "#{Stripe.api_base}/v1/customers/#{customer.id}/subscription")
+          .with(body: { plan: "silver" })
+          .to_return(body: JSON.generate(object: "subscription"))
+        subscription = customer.update_subscription(plan: "silver")
+        assert subscription.is_a?(Stripe::Subscription)
       end
     end
 
@@ -75,11 +75,11 @@ module Stripe
         customer = Stripe::Customer.retrieve("cus_123")
 
         # 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({ object: "subscription" }))
-        subscription = customer.cancel_subscription({:at_period_end => 'true'})
-        assert subscription.kind_of?(Stripe::Subscription)
+        stub_request(:delete, "#{Stripe.api_base}/v1/customers/#{customer.id}/subscription")
+          .with(query: { at_period_end: "true" })
+          .to_return(body: JSON.generate(object: "subscription"))
+        subscription = customer.cancel_subscription(at_period_end: "true")
+        assert subscription.is_a?(Stripe::Subscription)
       end
     end
 
@@ -87,10 +87,10 @@ module Stripe
       should "delete a discount" do
         customer = Stripe::Customer.retrieve("cus_123")
 
-        stub_request(:delete, "#{Stripe.api_base}/v1/customers/#{customer.id}/discount").
-          to_return(body: JSON.generate({ object: "customer" }))
+        stub_request(:delete, "#{Stripe.api_base}/v1/customers/#{customer.id}/discount")
+          .to_return(body: JSON.generate(object: "customer"))
         discount = customer.delete_discount
-        assert discount.kind_of?(Stripe::Customer)
+        assert discount.is_a?(Stripe::Customer)
       end
     end
 
@@ -104,11 +104,10 @@ module Stripe
       should "allow setting source with hash and set flag" do
         c = Stripe::Customer.new("test_customer")
         c.source = {
-          :object => 'card'
+          object: "card",
         }
         assert_equal true, c.source.save_with_parent
       end
     end
   end
 end
-