stripe 5.27.0 → 5.31.0

data/test/stripe/stripe_configuration_test.rb

@@ -16,19 +16,23 @@ module Stripe
         assert_equal 0, config.max_network_retries
         assert_equal 30, config.open_timeout
         assert_equal 80, config.read_timeout
+        assert_equal 30, config.write_timeout
         assert_equal "https://api.stripe.com", config.api_base
         assert_equal "https://connect.stripe.com", config.connect_base
         assert_equal "https://files.stripe.com", config.uploads_base
+        assert_equal nil, config.api_version
       end
 
       should "allow for overrides when a block is passed" do
         config = Stripe::StripeConfiguration.setup do |c|
           c.open_timeout = 100
           c.read_timeout = 100
+          c.write_timeout = 100 if WRITE_TIMEOUT_SUPPORTED
         end
 
         assert_equal 100, config.open_timeout
         assert_equal 100, config.read_timeout
+        assert_equal 100, config.write_timeout if WRITE_TIMEOUT_SUPPORTED
       end
     end
 
@@ -38,7 +42,7 @@ module Stripe
           c.open_timeout = 100
         end
 
-        duped_config = config.reverse_duplicate_merge(read_timeout: 500)
+        duped_config = config.reverse_duplicate_merge(read_timeout: 500, api_version: "2018-08-02")
 
         assert_equal config.open_timeout, duped_config.open_timeout
         assert_equal 500, duped_config.read_timeout
@@ -54,6 +58,24 @@ module Stripe
       end
     end
 
+    context "#max_network_retry_delay=" do
+      should "coerce the option into an integer" do
+        config = Stripe::StripeConfiguration.setup
+
+        config.max_network_retry_delay = "10"
+        assert_equal 10, config.max_network_retry_delay
+      end
+    end
+
+    context "#initial_network_retry_delay=" do
+      should "coerce the option into an integer" do
+        config = Stripe::StripeConfiguration.setup
+
+        config.initial_network_retry_delay = "10"
+        assert_equal 10, config.initial_network_retry_delay
+      end
+    end
+
     context "#log_level=" do
       should "be backwards compatible with old values" do
         config = Stripe::StripeConfiguration.setup
@@ -78,51 +100,60 @@ module Stripe
       should "clear when setting allow ca_bundle_path" do
         config = Stripe::StripeConfiguration.setup
 
-        StripeClient.expects(:clear_all_connection_managers)
+        StripeClient.expects(:clear_all_connection_managers).with(config: config)
         config.ca_bundle_path = "/path/to/ca/bundle"
       end
 
       should "clear when setting open timeout" do
         config = Stripe::StripeConfiguration.setup
 
-        StripeClient.expects(:clear_all_connection_managers)
+        StripeClient.expects(:clear_all_connection_managers).with(config: config)
         config.open_timeout = 10
       end
 
       should "clear when setting read timeout" do
         config = Stripe::StripeConfiguration.setup
 
-        StripeClient.expects(:clear_all_connection_managers)
+        StripeClient.expects(:clear_all_connection_managers).with(config: config)
         config.read_timeout = 10
       end
 
       should "clear when setting uploads_base" do
         config = Stripe::StripeConfiguration.setup
 
-        StripeClient.expects(:clear_all_connection_managers)
+        StripeClient.expects(:clear_all_connection_managers).with(config: config)
         config.uploads_base = "https://other.stripe.com"
       end
 
-      should "clearn when setting api_base to be configured" do
+      should "clear when setting api_base to be configured" do
         config = Stripe::StripeConfiguration.setup
 
-        StripeClient.expects(:clear_all_connection_managers)
+        StripeClient.expects(:clear_all_connection_managers).with(config: config)
         config.api_base = "https://other.stripe.com"
       end
 
       should "clear when setting connect_base" do
         config = Stripe::StripeConfiguration.setup
 
-        StripeClient.expects(:clear_all_connection_managers)
+        StripeClient.expects(:clear_all_connection_managers).with(config: config)
         config.connect_base = "https://other.stripe.com"
       end
 
       should "clear when setting verify_ssl_certs" do
         config = Stripe::StripeConfiguration.setup
 
-        StripeClient.expects(:clear_all_connection_managers)
+        StripeClient.expects(:clear_all_connection_managers).with(config: config)
         config.verify_ssl_certs = false
       end
     end
+
+    context "#key" do
+      should "generate the same key when values are identicial" do
+        assert_equal StripeConfiguration.setup.key, StripeConfiguration.setup.key
+
+        custom_config = StripeConfiguration.setup { |c| c.open_timeout = 1000 }
+        refute_equal StripeConfiguration.setup.key, custom_config.key
+      end
+    end
   end
 end

data/test/stripe/stripe_object_test.rb

@@ -496,5 +496,15 @@ module Stripe
         assert obj.method(:id).is_a?(Method)
       end
     end
+
+    should "ignore properties that are reserved names" do
+      obj = Stripe::StripeObject.construct_from(metadata: { class: "something" })
+
+      # See comment on `StripeObject::RESERVED_FIELD_NAMES`
+      assert_equal Stripe::StripeObject, obj.metadata.class
+
+      # Value still accessible with hash syntax
+      assert_equal "something", obj.metadata[:class]
+    end
   end
 end

data/test/stripe_test.rb

@@ -54,6 +54,19 @@ class StripeTest < Test::Unit::TestCase
       assert_equal 10, Stripe.read_timeout
     end
 
+    if WRITE_TIMEOUT_SUPPORTED
+      should "allow write timeout to be configured" do
+        Stripe.write_timeout = 10
+        assert_equal 10, Stripe.write_timeout
+      end
+    else
+      should "raise when write timeout to be configured is not supported" do
+        assert_raises NotImplementedError do
+          Stripe.write_timeout = 10
+        end
+      end
+    end
+
     should "allow api_key to be configured" do
       Stripe.api_key = "sk_local_test"
       assert_equal "sk_local_test", Stripe.api_key
@@ -101,6 +114,11 @@ class StripeTest < Test::Unit::TestCase
       assert_equal "https://other.stripe.com", Stripe.api_base
     end
 
+    should "allow api_version to be configured" do
+      Stripe.api_version = "2018-02-28"
+      assert_equal "2018-02-28", Stripe.api_version
+    end
+
     should "allow connect_base to be configured" do
       Stripe.connect_base = "https://other.stripe.com"
       assert_equal "https://other.stripe.com", Stripe.connect_base

data/test/test_helper.rb

@@ -1,8 +1,5 @@
 # frozen_string_literal: true
 
-require "coveralls"
-Coveralls.wear!("test_frameworks")
-
 require "stripe"
 require "test/unit"
 require "mocha/setup"
@@ -16,7 +13,7 @@ require ::File.expand_path("test_data", __dir__)
 require ::File.expand_path("stripe_mock", __dir__)
 
 # If changing this number, please also change it in `.travis.yml`.
-MOCK_MINIMUM_VERSION = "0.101.0"
+MOCK_MINIMUM_VERSION = "0.103.0"
 MOCK_PORT = Stripe::StripeMock.start
 
 # Disable all real network connections except those that are outgoing to
@@ -56,6 +53,8 @@ module Test
       include Stripe::TestData
       include Mocha
 
+      WRITE_TIMEOUT_SUPPORTED = Net::HTTP.instance_methods.include?(:write_timeout=)
+
       setup do
         Stripe.api_key = "sk_test_123"
         Stripe.api_base = "http://localhost:#{MOCK_PORT}"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: stripe
 version: !ruby/object:Gem::Version
-  version: 5.27.0
+  version: 5.31.0
 platform: ruby
 authors:
 - Stripe
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-10-14 00:00:00.000000000 Z
+date: 2021-04-02 00:00:00.000000000 Z
 dependencies: []
 description: Stripe is the easiest way to accept payments online.  See https://stripe.com
   for details.
@@ -21,6 +21,7 @@ files:
 - ".editorconfig"
 - ".gitattributes"
 - ".github/ISSUE_TEMPLATE.md"
+- ".github/workflows/ci.yml"
 - ".gitignore"
 - ".rubocop.yml"
 - ".rubocop_todo.yml"
@@ -64,6 +65,7 @@ files:
 - lib/stripe/resources/balance.rb
 - lib/stripe/resources/balance_transaction.rb
 - lib/stripe/resources/bank_account.rb
+- lib/stripe/resources/billing_portal/configuration.rb
 - lib/stripe/resources/billing_portal/session.rb
 - lib/stripe/resources/bitcoin_receiver.rb
 - lib/stripe/resources/bitcoin_transaction.rb
@@ -158,6 +160,7 @@ files:
 - test/stripe/balance_test.rb
 - test/stripe/balance_transaction_test.rb
 - test/stripe/bank_account_test.rb
+- test/stripe/billing_portal/configuration_test.rb
 - test/stripe/billing_portal/session_test.rb
 - test/stripe/capability_test.rb
 - test/stripe/charge_test.rb
@@ -246,7 +249,7 @@ metadata:
   github_repo: ssh://github.com/stripe/stripe-ruby
   homepage_uri: https://stripe.com/docs/api/ruby
   source_code_uri: https://github.com/stripe/stripe-ruby
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -262,7 +265,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.1.2
-signing_key: 
+signing_key:
 specification_version: 4
 summary: Ruby bindings for the Stripe API
 test_files:
@@ -278,6 +281,7 @@ test_files:
 - test/stripe/balance_test.rb
 - test/stripe/balance_transaction_test.rb
 - test/stripe/bank_account_test.rb
+- test/stripe/billing_portal/configuration_test.rb
 - test/stripe/billing_portal/session_test.rb
 - test/stripe/capability_test.rb
 - test/stripe/charge_test.rb