mirror of
https://github.com/stripe/stripe-ruby.git
synced 2025-10-05 00:02:50 -04:00
Adds the magic `frozen_string_literal: true` comment to every file and enables a Rubocop rule to make sure that it's always going to be there going forward as well. See here for more background [1], but the basic idea is that unlike many other languages, static strings in code are mutable by default. This has since been acknowledged as not a particularly good idea, and the intention is to rectify the mistake when Ruby 3 comes out, where all string literals will be frozen. The `frozen_string_literal` magic comment was introduced in Ruby 2.3 as a way of easing the transition, and allows libraries and projects to freeze their literals in advance. I don't think this is breaking in any way: it's possible that users might've been pulling out one of are literals somehow and mutating it, but that would probably not have been useful for anything and would certainly not be recommended, so I'm quite comfortable pushing this change through as a minor version. As discussed in #641. [1] https://stackoverflow.com/a/37799399
70 lines
2.4 KiB
Ruby
70 lines
2.4 KiB
Ruby
# 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
|