stripe 4.12.0 → 4.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a30c5b27bb9161913905e0ada304efcef00519446eb64cbf2e7c3d0d625a2810
-  data.tar.gz: d8503b19941c67e68c75b7b4e1c24cae2bd7a4c80a5b91e5c4e81058d23b658d
+  metadata.gz: 8fefd607d30d32ab6b4b1a3062c031fec8366abd331c311770136721354b89a4
+  data.tar.gz: c537164f6d8871f1508c4a2b82d894eec944169fbf4143c50c274df37f78170e
 SHA512:
-  metadata.gz: a3a0ecd3172c802dc16bacab0be7f3d34373ed2230d18e5a89b84814d3705568e6f0ca261fb576492c1ec3ef39132b17a9b9126ba69f3941e45a4b3f851a3f07
-  data.tar.gz: 7a6d70ebe915abe5774a4f5e55815e43ea24d6db4f4553ce77f18394b93aca8eeae051f9fd557d74bdda4d3e4d77112b821dca22a62c7284fc942392fc797638
+  metadata.gz: 1c7ab0496bb4f3359ca20399f7fd2dc72ce62d193ac18083974bf4f164b6204e9a77c01227814023e1ce91b589acfb505849dbe4793e748b25eb15c0183944c4
+  data.tar.gz: 3b3bfae890cb4c99a4a9005f9aac45bfd92c270035fab4df4f92b8fe17af3a520475c390cbdab59e9eabde7e76b4a66da816a39d6a0f2502fda0b1f5d1d9125f

data/.travis.yml

@@ -17,7 +17,7 @@ sudo: false
 env:
   global:
     # If changing this number, please also change it in `test/test_helper.rb`.
-    - STRIPE_MOCK_VERSION=0.49.0
+    - STRIPE_MOCK_VERSION=0.52.0
 
 cache:
   directories:

data/CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## 4.13.0 - 2019-04-16
+* [#766](https://github.com/stripe/stripe-ruby/pull/766) Relax constraints on objects that we'll accept as a file (now they just need to respond to `#read`)
+
 ## 4.12.0 - 2019-04-02
 * [#752](https://github.com/stripe/stripe-ruby/pull/752) Add `.delete` class method on deletable API resources
 * [#754](https://github.com/stripe/stripe-ruby/pull/754) Add class methods for all custom API requests (e.g. `Charge.capture`)

data/README.md

@@ -35,7 +35,7 @@ If you want to build the gem from source:
 
 ### Requirements
 
-* Ruby 2.0+.
+* Ruby 2.1+.
 
 ### Bundler
 

data/VERSION

@@ -1 +1 @@
-4.12.0
+4.13.0

data/lib/stripe/file.rb

@@ -20,7 +20,11 @@ module Stripe
       # rest-client would accept a vanilla `File` for upload, but Faraday does
       # not. Support the old API by wrapping a `File`-like object with an
       # `UploadIO` object if we're given one.
-      if params[:file] && params[:file].respond_to?(:path) && params[:file].respond_to?(:read)
+      if params[:file] && !params[:file].is_a?(String)
+        unless params[:file].respond_to?(:read)
+          raise ArgumentError, "file must respond to `#read`"
+        end
+
         params[:file] = Faraday::UploadIO.new(params[:file], nil)
       end
 

data/lib/stripe/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Stripe
-  VERSION = "4.12.0".freeze
+  VERSION = "4.13.0".freeze
 end

data/test/stripe/account_test.rb

@@ -265,5 +265,130 @@ module Stripe
         assert_equal(expected, obj.serialize_params)
       end
     end
+
+    context "#create_external_account" do
+      should "create an external account" do
+        external_account = Stripe::Account.create_external_account(
+          "acct_123",
+          external_account: "btok_123"
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/accounts/acct_123/external_accounts"
+        assert external_account.is_a?(Stripe::BankAccount)
+      end
+    end
+
+    context "#retrieve_external_account" do
+      should "retrieve an external account" do
+        external_account = Stripe::Account.retrieve_external_account(
+          "acct_123",
+          "ba_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/accounts/acct_123/external_accounts/ba_123"
+        assert external_account.is_a?(Stripe::BankAccount)
+      end
+    end
+
+    context "#update_external_account" do
+      should "update an external account" do
+        external_account = Stripe::Account.update_external_account(
+          "acct_123",
+          "ba_123",
+          metadata: { foo: "bar" }
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/accounts/acct_123/external_accounts/ba_123"
+        assert external_account.is_a?(Stripe::BankAccount)
+      end
+    end
+
+    context "#delete_external_account" do
+      should "delete an external_account" do
+        external_account = Stripe::Account.delete_external_account(
+          "acct_123",
+          "ba_123"
+        )
+        assert_requested :delete, "#{Stripe.api_base}/v1/accounts/acct_123/external_accounts/ba_123"
+        assert external_account.deleted
+        assert_equal "ba_123", external_account.id
+      end
+    end
+
+    context "#list_external_accounts" do
+      should "list the account's external accounts" do
+        external_accounts = Stripe::Account.list_external_accounts(
+          "acct_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/accounts/acct_123/external_accounts"
+        assert external_accounts.is_a?(Stripe::ListObject)
+        assert external_accounts.data.is_a?(Array)
+      end
+    end
+
+    context "#create_login_link" do
+      should "create a login link" do
+        login_link = Stripe::Account.create_login_link(
+          "acct_123"
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/accounts/acct_123/login_links"
+        assert login_link.is_a?(Stripe::LoginLink)
+      end
+    end
+
+    context "#create_person" do
+      should "create a person" do
+        person = Stripe::Account.create_person(
+          "acct_123",
+          first_name: "John",
+          last_name: "Doe"
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/accounts/acct_123/persons"
+        assert person.is_a?(Stripe::Person)
+      end
+    end
+
+    context "#retrieve_person" do
+      should "retrieve a person" do
+        person = Stripe::Account.retrieve_person(
+          "acct_123",
+          "person_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/accounts/acct_123/persons/person_123"
+        assert person.is_a?(Stripe::Person)
+      end
+    end
+
+    context "#update_person" do
+      should "update a person" do
+        person = Stripe::Account.update_person(
+          "acct_123",
+          "person_123",
+          first_name: "John"
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/accounts/acct_123/persons/person_123"
+        assert person.is_a?(Stripe::Person)
+      end
+    end
+
+    context "#delete_person" do
+      should "delete an person" do
+        person = Stripe::Account.delete_person(
+          "acct_123",
+          "person_123"
+        )
+        assert_requested :delete, "#{Stripe.api_base}/v1/accounts/acct_123/persons/person_123"
+        assert person.deleted
+        assert_equal "person_123", person.id
+      end
+    end
+
+    context "#list_persons" do
+      should "list the account's external accounts" do
+        persons = Stripe::Account.list_persons(
+          "acct_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/accounts/acct_123/persons"
+        assert persons.is_a?(Stripe::ListObject)
+        assert persons.data.is_a?(Array)
+      end
+    end
   end
 end

data/test/stripe/application_fee_test.rb

@@ -10,5 +10,49 @@ module Stripe
       assert fees.data.is_a?(Array)
       assert fees.data[0].is_a?(Stripe::ApplicationFee)
     end
+
+    context "#create_refund" do
+      should "create a refund" do
+        refund = Stripe::ApplicationFee.create_refund(
+          "fee_123"
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/application_fees/fee_123/refunds"
+        assert refund.is_a?(Stripe::ApplicationFeeRefund)
+      end
+    end
+
+    context "#retrieve_refund" do
+      should "retrieve a refund" do
+        refund = Stripe::ApplicationFee.retrieve_refund(
+          "fee_123",
+          "fr_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/application_fees/fee_123/refunds/fr_123"
+        assert refund.is_a?(Stripe::ApplicationFeeRefund)
+      end
+    end
+
+    context "#update_refund" do
+      should "update a refund" do
+        refund = Stripe::ApplicationFee.update_refund(
+          "fee_123",
+          "fr_123",
+          metadata: { foo: "bar" }
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/application_fees/fee_123/refunds/fr_123"
+        assert refund.is_a?(Stripe::ApplicationFeeRefund)
+      end
+    end
+
+    context "#list_refunds" do
+      should "list the application fee's refuns" do
+        refunds = Stripe::ApplicationFee.list_refunds(
+          "fee_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/application_fees/fee_123/refunds"
+        assert refunds.is_a?(Stripe::ListObject)
+        assert refunds.data.is_a?(Array)
+      end
+    end
   end
 end

data/test/stripe/customer_test.rb

@@ -113,6 +113,57 @@ module Stripe
         assert discount.is_a?(Stripe::Discount)
       end
     end
+    context "#create_source" do
+      should "create a source" do
+        Stripe::Customer.create_source(
+          "cus_123",
+          source: "tok_123"
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/customers/cus_123/sources"
+      end
+    end
+
+    context "#retrieve_source" do
+      should "retrieve a source" do
+        Stripe::Customer.retrieve_source(
+          "cus_123",
+          "ba_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/customers/cus_123/sources/ba_123"
+      end
+    end
+
+    context "#update_source" do
+      should "update a source" do
+        Stripe::Customer.update_source(
+          "cus_123",
+          "ba_123",
+          metadata: { foo: "bar" }
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/customers/cus_123/sources/ba_123"
+      end
+    end
+
+    context "#delete_source" do
+      should "delete a source" do
+        Stripe::Customer.delete_source(
+          "cus_123",
+          "ba_123"
+        )
+        assert_requested :delete, "#{Stripe.api_base}/v1/customers/cus_123/sources/ba_123"
+      end
+    end
+
+    context "#list_sources" do
+      should "list the customer's sources" do
+        sources = Stripe::Customer.list_sources(
+          "cus_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/customers/cus_123/sources"
+        assert sources.is_a?(Stripe::ListObject)
+        assert sources.data.is_a?(Array)
+      end
+    end
 
     context "source field" do
       should "allow setting source with token" do

data/test/stripe/file_test.rb

@@ -31,7 +31,8 @@ module Stripe
       should "be creatable with a File" do
         file = Stripe::File.create(
           purpose: "dispute_evidence",
-          file: ::File.new(__FILE__)
+          file: ::File.new(__FILE__),
+          file_link_data: { create: true }
         )
         assert_requested :post, "#{Stripe.uploads_base}/v1/files"
         assert file.is_a?(Stripe::File)
@@ -44,7 +45,8 @@ module Stripe
 
         file = Stripe::File.create(
           purpose: "dispute_evidence",
-          file: tempfile
+          file: tempfile,
+          file_link_data: { create: true }
         )
         assert_requested :post, "#{Stripe.uploads_base}/v1/files"
         assert file.is_a?(Stripe::File)
@@ -53,11 +55,33 @@ module Stripe
       should "be creatable with Faraday::UploadIO" do
         file = Stripe::File.create(
           purpose: "dispute_evidence",
-          file: Faraday::UploadIO.new(::File.new(__FILE__), nil)
+          file: Faraday::UploadIO.new(::File.new(__FILE__), nil),
+          file_link_data: { create: true }
         )
         assert_requested :post, "#{Stripe.uploads_base}/v1/files"
         assert file.is_a?(Stripe::File)
       end
+
+      should "be creatable with a string" do
+        file = Stripe::File.create(
+          purpose: "dispute_evidence",
+          file: "my-file-contents",
+          file_link_data: { create: true }
+        )
+        assert_requested :post, "#{Stripe.uploads_base}/v1/files"
+        assert file.is_a?(Stripe::File)
+      end
+
+      should "raise given a file object that doesn't respond to #read" do
+        e = assert_raises(ArgumentError) do
+          Stripe::File.create(
+            purpose: "dispute_evidence",
+            file: Object.new,
+            file_link_data: { create: true }
+          )
+        end
+        assert_equal "file must respond to `#read`", e.message
+      end
     end
 
     should "be deserializable when `object=file`" do

data/test/stripe/file_upload_test.rb

@@ -34,7 +34,8 @@ module Stripe
       should "be creatable with a File" do
         file = Stripe::FileUpload.create(
           purpose: "dispute_evidence",
-          file: ::File.new(__FILE__)
+          file: ::File.new(__FILE__),
+          file_link_data: { create: true }
         )
         assert_requested :post, "#{Stripe.uploads_base}/v1/files"
         assert file.is_a?(Stripe::FileUpload)
@@ -47,7 +48,8 @@ module Stripe
 
         file = Stripe::FileUpload.create(
           purpose: "dispute_evidence",
-          file: tempfile
+          file: tempfile,
+          file_link_data: { create: true }
         )
         assert_requested :post, "#{Stripe.uploads_base}/v1/files"
         assert file.is_a?(Stripe::FileUpload)
@@ -56,7 +58,8 @@ module Stripe
       should "be creatable with Faraday::UploadIO" do
         file = Stripe::FileUpload.create(
           purpose: "dispute_evidence",
-          file: Faraday::UploadIO.new(::File.new(__FILE__), nil)
+          file: Faraday::UploadIO.new(::File.new(__FILE__), nil),
+          file_link_data: { create: true }
         )
         assert_requested :post, "#{Stripe.uploads_base}/v1/files"
         assert file.is_a?(Stripe::FileUpload)

data/test/stripe/invoice_test.rb

@@ -38,8 +38,6 @@ module Stripe
       assert invoice.is_a?(Stripe::Invoice)
     end
 
-    should "be deletable" do
-    end
     context "#delete" do
       should "be deletable" do
         invoice = Stripe::Invoice.retrieve("in_123")

data/test/stripe/plan_test.rb

@@ -21,7 +21,7 @@ module Stripe
       plan = Stripe::Plan.create(
         amount: 5000,
         interval: "month",
-        name: "Sapphire elite",
+        nickname: "Sapphire elite",
         currency: "usd"
       )
       assert_requested :post, "#{Stripe.api_base}/v1/plans"
@@ -32,7 +32,7 @@ module Stripe
       plan = Stripe::Plan.create(
         amount: 5000,
         interval: "month",
-        name: "Sapphire elite",
+        nickname: "Sapphire elite",
         currency: "usd",
         usage_type: "metered"
       )
@@ -43,7 +43,7 @@ module Stripe
     should "be creatable with tiered configuration" do
       plan = Stripe::Plan.create(
         interval: "month",
-        name: "Sapphire elite",
+        nickname: "Sapphire elite",
         currency: "usd",
         billing_scheme: "tiered",
         tiers_mode: "volume",
@@ -56,7 +56,7 @@ module Stripe
     should "be creatable with transform_usage" do
       plan = Stripe::Plan.create(
         interval: "month",
-        name: "Sapphire elite",
+        nickname: "Sapphire elite",
         currency: "usd",
         amount: 5000,
         transform_usage: { round: "up", divide_by: 50 }

data/test/stripe/subscription_schedule_test.rb

@@ -89,5 +89,28 @@ module Stripe
         assert revisions.data[0].is_a?(Stripe::SubscriptionScheduleRevision)
       end
     end
+
+    context "#retrieve_revision" do
+      should "retrieve a subscription schedule revision" do
+        revision = Stripe::SubscriptionSchedule.retrieve_revision(
+          "sub_sched_123",
+          "sub_sched_rev_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/subscription_schedules/sub_sched_123/revisions/sub_sched_rev_123"
+        assert revision.is_a?(Stripe::SubscriptionScheduleRevision)
+      end
+    end
+
+    context "#list_revisions" do
+      should "list a subscription schedule's revisions" do
+        revisions = Stripe::SubscriptionSchedule.list_revisions(
+          "sub_sched_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/subscription_schedules/sub_sched_123/revisions"
+        assert revisions.is_a?(Stripe::ListObject)
+        assert revisions.data.is_a?(Array)
+        assert revisions.data[0].is_a?(Stripe::SubscriptionScheduleRevision)
+      end
+    end
   end
 end

data/test/stripe/transfer_test.rb

@@ -39,5 +39,50 @@ module Stripe
       assert_requested :post, "#{Stripe.api_base}/v1/transfers/tr_123"
       assert transfer.is_a?(Stripe::Transfer)
     end
+
+    context "#create_reversal" do
+      should "create a reversal" do
+        reversal = Stripe::Transfer.create_reversal(
+          "tr_123",
+          amount: 100
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/transfers/tr_123/reversals"
+        assert reversal.is_a?(Stripe::Reversal)
+      end
+    end
+
+    context "#retrieve_reversal" do
+      should "retrieve a reversal" do
+        reversal = Stripe::Transfer.retrieve_reversal(
+          "tr_123",
+          "trr_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/transfers/tr_123/reversals/trr_123"
+        assert reversal.is_a?(Stripe::Reversal)
+      end
+    end
+
+    context "#update_reversal" do
+      should "update a reversal" do
+        reversal = Stripe::Transfer.update_reversal(
+          "tr_123",
+          "trr_123",
+          metadata: { foo: "bar" }
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/transfers/tr_123/reversals/trr_123"
+        assert reversal.is_a?(Stripe::Reversal)
+      end
+    end
+
+    context "#list_reversals" do
+      should "list the transfer's reversals" do
+        reversals = Stripe::Transfer.list_reversals(
+          "tr_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/transfers/tr_123/reversals"
+        assert reversals.is_a?(Stripe::ListObject)
+        assert reversals.data.is_a?(Array)
+      end
+    end
   end
 end

data/test/test_helper.rb

@@ -17,7 +17,7 @@ require ::File.expand_path("../test_data", __FILE__)
 require ::File.expand_path("../stripe_mock", __FILE__)
 
 # If changing this number, please also change it in `.travis.yml`.
-MOCK_MINIMUM_VERSION = "0.49.0".freeze
+MOCK_MINIMUM_VERSION = "0.52.0".freeze
 MOCK_PORT = Stripe::StripeMock.start
 
 # Disable all real network connections except those that are outgoing to

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: stripe
 version: !ruby/object:Gem::Version
-  version: 4.12.0
+  version: 4.13.0
 platform: ruby
 authors:
 - Stripe
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-04-02 00:00:00.000000000 Z
+date: 2019-04-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -153,17 +153,13 @@ files:
 - stripe.gemspec
 - test/api_stub_helpers.rb
 - test/openapi/README.md
-- test/stripe/account_external_accounts_operations_test.rb
 - test/stripe/account_link_test.rb
-- test/stripe/account_login_links_operations_test.rb
-- test/stripe/account_persons_operations_test.rb
 - test/stripe/account_test.rb
 - test/stripe/alipay_account_test.rb
 - test/stripe/api_operations_test.rb
 - test/stripe/api_resource_test.rb
 - test/stripe/apple_pay_domain_test.rb
 - test/stripe/application_fee_refund_test.rb
-- test/stripe/application_fee_refunds_operations_test.rb
 - test/stripe/application_fee_test.rb
 - test/stripe/balance_test.rb
 - test/stripe/bank_account_test.rb
@@ -172,7 +168,6 @@ files:
 - test/stripe/country_spec_test.rb
 - test/stripe/coupon_test.rb
 - test/stripe/customer_card_test.rb
-- test/stripe/customer_sources_operations_test.rb
 - test/stripe/customer_test.rb
 - test/stripe/dispute_test.rb
 - test/stripe/ephemeral_key_test.rb
@@ -218,7 +213,6 @@ files:
 - test/stripe/stripe_response_test.rb
 - test/stripe/subscription_item_test.rb
 - test/stripe/subscription_schedule_revision_test.rb
-- test/stripe/subscription_schedule_revisions_operations_test.rb
 - test/stripe/subscription_schedule_test.rb
 - test/stripe/subscription_test.rb
 - test/stripe/terminal/connection_token_test.rb
@@ -226,7 +220,6 @@ files:
 - test/stripe/terminal/reader_test.rb
 - test/stripe/three_d_secure_test.rb
 - test/stripe/topup_test.rb
-- test/stripe/transfer_reversals_operations_test.rb
 - test/stripe/transfer_test.rb
 - test/stripe/usage_record_summary_test.rb
 - test/stripe/usage_record_test.rb
@@ -263,17 +256,13 @@ summary: Ruby bindings for the Stripe API
 test_files:
 - test/api_stub_helpers.rb
 - test/openapi/README.md
-- test/stripe/account_external_accounts_operations_test.rb
 - test/stripe/account_link_test.rb
-- test/stripe/account_login_links_operations_test.rb
-- test/stripe/account_persons_operations_test.rb
 - test/stripe/account_test.rb
 - test/stripe/alipay_account_test.rb
 - test/stripe/api_operations_test.rb
 - test/stripe/api_resource_test.rb
 - test/stripe/apple_pay_domain_test.rb
 - test/stripe/application_fee_refund_test.rb
-- test/stripe/application_fee_refunds_operations_test.rb
 - test/stripe/application_fee_test.rb
 - test/stripe/balance_test.rb
 - test/stripe/bank_account_test.rb
@@ -282,7 +271,6 @@ test_files:
 - test/stripe/country_spec_test.rb
 - test/stripe/coupon_test.rb
 - test/stripe/customer_card_test.rb
-- test/stripe/customer_sources_operations_test.rb
 - test/stripe/customer_test.rb
 - test/stripe/dispute_test.rb
 - test/stripe/ephemeral_key_test.rb
@@ -328,7 +316,6 @@ test_files:
 - test/stripe/stripe_response_test.rb
 - test/stripe/subscription_item_test.rb
 - test/stripe/subscription_schedule_revision_test.rb
-- test/stripe/subscription_schedule_revisions_operations_test.rb
 - test/stripe/subscription_schedule_test.rb
 - test/stripe/subscription_test.rb
 - test/stripe/terminal/connection_token_test.rb
@@ -336,7 +323,6 @@ test_files:
 - test/stripe/terminal/reader_test.rb
 - test/stripe/three_d_secure_test.rb
 - test/stripe/topup_test.rb
-- test/stripe/transfer_reversals_operations_test.rb
 - test/stripe/transfer_test.rb
 - test/stripe/usage_record_summary_test.rb
 - test/stripe/usage_record_test.rb

data/test/stripe/account_external_accounts_operations_test.rb

@@ -1,69 +0,0 @@
-# frozen_string_literal: true
-
-require ::File.expand_path("../../test_helper", __FILE__)
-
-module Stripe
-  class AccountExternalAccountsOperationsTest < Test::Unit::TestCase
-    setup do
-      @account_id = "acct_123"
-      @external_account_id = "ba_123"
-    end
-
-    context "#create_external_account" do
-      should "create an external account" do
-        external_account = Stripe::Account.create_external_account(
-          @account_id,
-          external_account: "btok_123"
-        )
-        assert_requested :post, "#{Stripe.api_base}/v1/accounts/#{@account_id}/external_accounts"
-        assert external_account.is_a?(Stripe::BankAccount)
-      end
-    end
-
-    context "#retrieve_external_account" do
-      should "retrieve an external account" do
-        external_account = Stripe::Account.retrieve_external_account(
-          @account_id,
-          @external_account_id
-        )
-        assert_requested :get, "#{Stripe.api_base}/v1/accounts/#{@account_id}/external_accounts/#{@external_account_id}"
-        assert external_account.is_a?(Stripe::BankAccount)
-      end
-    end
-
-    context "#update_external_account" do
-      should "update an external account" do
-        external_account = Stripe::Account.update_external_account(
-          @account_id,
-          @external_account_id,
-          metadata: { foo: "bar" }
-        )
-        assert_requested :post, "#{Stripe.api_base}/v1/accounts/#{@account_id}/external_accounts/#{@external_account_id}"
-        assert external_account.is_a?(Stripe::BankAccount)
-      end
-    end
-
-    context "#delete_external_account" do
-      should "delete an external_account" do
-        external_account = Stripe::Account.delete_external_account(
-          @account_id,
-          @external_account_id
-        )
-        assert_requested :delete, "#{Stripe.api_base}/v1/accounts/#{@account_id}/external_accounts/#{@external_account_id}"
-        assert external_account.deleted
-        assert_equal @external_account_id, external_account.id
-      end
-    end
-
-    context "#list_external_accounts" do
-      should "list the account's external accounts" do
-        external_accounts = Stripe::Account.list_external_accounts(
-          @account_id
-        )
-        assert_requested :get, "#{Stripe.api_base}/v1/accounts/#{@account_id}/external_accounts"
-        assert external_accounts.is_a?(Stripe::ListObject)
-        assert external_accounts.data.is_a?(Array)
-      end
-    end
-  end
-end

data/test/stripe/account_login_links_operations_test.rb

@@ -1,21 +0,0 @@
-# frozen_string_literal: true
-
-require ::File.expand_path("../../test_helper", __FILE__)
-
-module Stripe
-  class AccountLoginLinksOperationsTest < Test::Unit::TestCase
-    setup do
-      @account_id = "acct_123"
-    end
-
-    context "#create_login_link" do
-      should "create a login link" do
-        login_link = Stripe::Account.create_login_link(
-          @account_id
-        )
-        assert_requested :post, "#{Stripe.api_base}/v1/accounts/#{@account_id}/login_links"
-        assert login_link.is_a?(Stripe::LoginLink)
-      end
-    end
-  end
-end

data/test/stripe/account_persons_operations_test.rb

@@ -1,70 +0,0 @@
-# frozen_string_literal: true
-
-require ::File.expand_path("../../test_helper", __FILE__)
-
-module Stripe
-  class AccountPersonsOperationsTest < Test::Unit::TestCase
-    setup do
-      @account_id = "acct_123"
-      @person_id = "person_123"
-    end
-
-    context "#create_person" do
-      should "create a person" do
-        person = Stripe::Account.create_person(
-          @account_id,
-          first_name: "John",
-          last_name: "Doe"
-        )
-        assert_requested :post, "#{Stripe.api_base}/v1/accounts/#{@account_id}/persons"
-        assert person.is_a?(Stripe::Person)
-      end
-    end
-
-    context "#retrieve_person" do
-      should "retrieve a person" do
-        person = Stripe::Account.retrieve_person(
-          @account_id,
-          @person_id
-        )
-        assert_requested :get, "#{Stripe.api_base}/v1/accounts/#{@account_id}/persons/#{@person_id}"
-        assert person.is_a?(Stripe::Person)
-      end
-    end
-
-    context "#update_person" do
-      should "update a person" do
-        person = Stripe::Account.update_person(
-          @account_id,
-          @person_id,
-          first_name: "John"
-        )
-        assert_requested :post, "#{Stripe.api_base}/v1/accounts/#{@account_id}/persons/#{@person_id}"
-        assert person.is_a?(Stripe::Person)
-      end
-    end
-
-    context "#delete_person" do
-      should "delete an person" do
-        person = Stripe::Account.delete_person(
-          @account_id,
-          @person_id
-        )
-        assert_requested :delete, "#{Stripe.api_base}/v1/accounts/#{@account_id}/persons/#{@person_id}"
-        assert person.deleted
-        assert_equal @person_id, person.id
-      end
-    end
-
-    context "#list_persons" do
-      should "list the account's external accounts" do
-        persons = Stripe::Account.list_persons(
-          @account_id
-        )
-        assert_requested :get, "#{Stripe.api_base}/v1/accounts/#{@account_id}/persons"
-        assert persons.is_a?(Stripe::ListObject)
-        assert persons.data.is_a?(Array)
-      end
-    end
-  end
-end

data/test/stripe/application_fee_refunds_operations_test.rb

@@ -1,56 +0,0 @@
-# frozen_string_literal: true
-
-require ::File.expand_path("../../test_helper", __FILE__)
-
-module Stripe
-  class ApplicationFeeRefundsOperationsTest < Test::Unit::TestCase
-    setup do
-      @application_fee_id = "fee_123"
-      @refund_id = "fr_123"
-    end
-
-    context "#create_refund" do
-      should "create a refund" do
-        refund = Stripe::ApplicationFee.create_refund(
-          @application_fee_id
-        )
-        assert_requested :post, "#{Stripe.api_base}/v1/application_fees/#{@application_fee_id}/refunds"
-        assert refund.is_a?(Stripe::ApplicationFeeRefund)
-      end
-    end
-
-    context "#retrieve_refund" do
-      should "retrieve a refund" do
-        refund = Stripe::ApplicationFee.retrieve_refund(
-          @application_fee_id,
-          @refund_id
-        )
-        assert_requested :get, "#{Stripe.api_base}/v1/application_fees/#{@application_fee_id}/refunds/#{@refund_id}"
-        assert refund.is_a?(Stripe::ApplicationFeeRefund)
-      end
-    end
-
-    context "#update_refund" do
-      should "update a refund" do
-        refund = Stripe::ApplicationFee.update_refund(
-          @application_fee_id,
-          @refund_id,
-          metadata: { foo: "bar" }
-        )
-        assert_requested :post, "#{Stripe.api_base}/v1/application_fees/#{@application_fee_id}/refunds/#{@refund_id}"
-        assert refund.is_a?(Stripe::ApplicationFeeRefund)
-      end
-    end
-
-    context "#list_refunds" do
-      should "list the application fee's refuns" do
-        refunds = Stripe::ApplicationFee.list_refunds(
-          @application_fee_id
-        )
-        assert_requested :get, "#{Stripe.api_base}/v1/application_fees/#{@application_fee_id}/refunds"
-        assert refunds.is_a?(Stripe::ListObject)
-        assert refunds.data.is_a?(Array)
-      end
-    end
-  end
-end

data/test/stripe/customer_sources_operations_test.rb

@@ -1,64 +0,0 @@
-# frozen_string_literal: true
-
-require ::File.expand_path("../../test_helper", __FILE__)
-
-module Stripe
-  class CustomerSourcesOperationsTest < Test::Unit::TestCase
-    setup do
-      @customer_id = "cus_123"
-      @source_id = "ba_123"
-    end
-
-    context "#create_source" do
-      should "create a source" do
-        Stripe::Customer.create_source(
-          @customer_id,
-          source: "tok_123"
-        )
-        assert_requested :post, "#{Stripe.api_base}/v1/customers/#{@customer_id}/sources"
-      end
-    end
-
-    context "#retrieve_source" do
-      should "retrieve a source" do
-        Stripe::Customer.retrieve_source(
-          @customer_id,
-          @source_id
-        )
-        assert_requested :get, "#{Stripe.api_base}/v1/customers/#{@customer_id}/sources/#{@source_id}"
-      end
-    end
-
-    context "#update_source" do
-      should "update a source" do
-        Stripe::Customer.update_source(
-          @customer_id,
-          @source_id,
-          metadata: { foo: "bar" }
-        )
-        assert_requested :post, "#{Stripe.api_base}/v1/customers/#{@customer_id}/sources/#{@source_id}"
-      end
-    end
-
-    context "#delete_source" do
-      should "delete a source" do
-        Stripe::Customer.delete_source(
-          @customer_id,
-          @source_id
-        )
-        assert_requested :delete, "#{Stripe.api_base}/v1/customers/#{@customer_id}/sources/#{@source_id}"
-      end
-    end
-
-    context "#list_sources" do
-      should "list the customer's sources" do
-        sources = Stripe::Customer.list_sources(
-          @customer_id
-        )
-        assert_requested :get, "#{Stripe.api_base}/v1/customers/#{@customer_id}/sources"
-        assert sources.is_a?(Stripe::ListObject)
-        assert sources.data.is_a?(Array)
-      end
-    end
-  end
-end

data/test/stripe/subscription_schedule_revisions_operations_test.rb

@@ -1,35 +0,0 @@
-# frozen_string_literal: true
-
-require ::File.expand_path("../../test_helper", __FILE__)
-
-module Stripe
-  class SubscriptionScheduleRevisionsOperationsTest < Test::Unit::TestCase
-    setup do
-      @schedule_id = "sub_sched_123"
-      @revision_id = "sub_sched_rev_123"
-    end
-
-    context "#retrieve_revision" do
-      should "retrieve a subscription schedule revision" do
-        revision = Stripe::SubscriptionSchedule.retrieve_revision(
-          @schedule_id,
-          @revision_id
-        )
-        assert_requested :get, "#{Stripe.api_base}/v1/subscription_schedules/#{@schedule_id}/revisions/#{@revision_id}"
-        assert revision.is_a?(Stripe::SubscriptionScheduleRevision)
-      end
-    end
-
-    context "#list_revisions" do
-      should "list a subscription schedule's revisions" do
-        revisions = Stripe::SubscriptionSchedule.list_revisions(
-          @schedule_id
-        )
-        assert_requested :get, "#{Stripe.api_base}/v1/subscription_schedules/#{@schedule_id}/revisions"
-        assert revisions.is_a?(Stripe::ListObject)
-        assert revisions.data.is_a?(Array)
-        assert revisions.data[0].is_a?(Stripe::SubscriptionScheduleRevision)
-      end
-    end
-  end
-end

data/test/stripe/transfer_reversals_operations_test.rb

@@ -1,57 +0,0 @@
-# frozen_string_literal: true
-
-require ::File.expand_path("../../test_helper", __FILE__)
-
-module Stripe
-  class TransferReversalsOperationsTest < Test::Unit::TestCase
-    setup do
-      @transfer_id = "tr_123"
-      @reversal_id = "trr_123"
-    end
-
-    context "#create_reversal" do
-      should "create a reversal" do
-        reversal = Stripe::Transfer.create_reversal(
-          @transfer_id,
-          amount: 100
-        )
-        assert_requested :post, "#{Stripe.api_base}/v1/transfers/#{@transfer_id}/reversals"
-        assert reversal.is_a?(Stripe::Reversal)
-      end
-    end
-
-    context "#retrieve_reversal" do
-      should "retrieve a reversal" do
-        reversal = Stripe::Transfer.retrieve_reversal(
-          @transfer_id,
-          @reversal_id
-        )
-        assert_requested :get, "#{Stripe.api_base}/v1/transfers/#{@transfer_id}/reversals/#{@reversal_id}"
-        assert reversal.is_a?(Stripe::Reversal)
-      end
-    end
-
-    context "#update_reversal" do
-      should "update a reversal" do
-        reversal = Stripe::Transfer.update_reversal(
-          @transfer_id,
-          @reversal_id,
-          metadata: { foo: "bar" }
-        )
-        assert_requested :post, "#{Stripe.api_base}/v1/transfers/#{@transfer_id}/reversals/#{@reversal_id}"
-        assert reversal.is_a?(Stripe::Reversal)
-      end
-    end
-
-    context "#list_reversals" do
-      should "list the transfer's reversals" do
-        reversals = Stripe::Transfer.list_reversals(
-          @transfer_id
-        )
-        assert_requested :get, "#{Stripe.api_base}/v1/transfers/#{@transfer_id}/reversals"
-        assert reversals.is_a?(Stripe::ListObject)
-        assert reversals.data.is_a?(Array)
-      end
-    end
-  end
-end