stripe 5.20.0 → 5.24.0

data/test/stripe/stripe_configuration_test.rb

@@ -0,0 +1,128 @@
+# frozen_string_literal: true
+
+require ::File.expand_path("../test_helper", __dir__)
+
+module Stripe
+  class StripeConfigurationTest < Test::Unit::TestCase
+    context ".setup" do
+      should "initialize a new configuration with defaults" do
+        config = Stripe::StripeConfiguration.setup
+
+        assert_equal Stripe::DEFAULT_CA_BUNDLE_PATH, config.ca_bundle_path
+        assert_equal true, config.enable_telemetry
+        assert_equal true, config.verify_ssl_certs
+        assert_equal 2, config.max_network_retry_delay
+        assert_equal 0.5, config.initial_network_retry_delay
+        assert_equal 0, config.max_network_retries
+        assert_equal 30, config.open_timeout
+        assert_equal 80, config.read_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
+      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
+        end
+
+        assert_equal 100, config.open_timeout
+        assert_equal 100, config.read_timeout
+      end
+    end
+
+    context "#reverse_duplicate_merge" do
+      should "return a duplicate object with overrides" do
+        config = Stripe::StripeConfiguration.setup do |c|
+          c.open_timeout = 100
+        end
+
+        duped_config = config.reverse_duplicate_merge(read_timeout: 500)
+
+        assert_equal config.open_timeout, duped_config.open_timeout
+        assert_equal 500, duped_config.read_timeout
+      end
+    end
+
+    context "#max_network_retries=" do
+      should "coerce the option into an integer" do
+        config = Stripe::StripeConfiguration.setup
+
+        config.max_network_retries = "10"
+        assert_equal 10, config.max_network_retries
+      end
+    end
+
+    context "#log_level=" do
+      should "be backwards compatible with old values" do
+        config = Stripe::StripeConfiguration.setup
+
+        config.log_level = "debug"
+        assert_equal Stripe::LEVEL_DEBUG, config.log_level
+
+        config.log_level = "info"
+        assert_equal Stripe::LEVEL_INFO, config.log_level
+      end
+
+      should "raise an error if the value isn't valid" do
+        config = Stripe::StripeConfiguration.setup
+
+        assert_raises ArgumentError do
+          config.log_level = "Foo"
+        end
+      end
+    end
+
+    context "options that require all connection managers to be cleared" do
+      should "clear when setting allow ca_bundle_path" do
+        config = Stripe::StripeConfiguration.setup
+
+        StripeClient.expects(:clear_all_connection_managers)
+        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)
+        config.open_timeout = 10
+      end
+
+      should "clear when setting read timeout" do
+        config = Stripe::StripeConfiguration.setup
+
+        StripeClient.expects(:clear_all_connection_managers)
+        config.read_timeout = 10
+      end
+
+      should "clear when setting uploads_base" do
+        config = Stripe::StripeConfiguration.setup
+
+        StripeClient.expects(:clear_all_connection_managers)
+        config.uploads_base = "https://other.stripe.com"
+      end
+
+      should "clearn when setting api_base to be configured" do
+        config = Stripe::StripeConfiguration.setup
+
+        StripeClient.expects(:clear_all_connection_managers)
+        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)
+        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)
+        config.verify_ssl_certs = false
+      end
+    end
+  end
+end

data/test/stripe/subscription_item_test.rb

@@ -22,7 +22,7 @@ module Stripe
 
     should "be creatable" do
       item = Stripe::SubscriptionItem.create(
-        plan: "sapphire-elite",
+        price: "sapphire-elite",
         quantity: 3,
         subscription: "sub_123"
       )

data/test/stripe_test.rb

@@ -23,28 +23,97 @@ class StripeTest < Test::Unit::TestCase
     end
   end
 
-  should "allow ca_bundle_path to be configured" do
-    begin
-      old = Stripe.ca_bundle_path
-      Stripe.ca_bundle_path = "path/to/ca/bundle"
-      assert_equal "path/to/ca/bundle", Stripe.ca_bundle_path
-    ensure
-      Stripe.ca_bundle_path = old
+  context "forwardable configurations" do
+    context "internal configurations" do
+      should "return the certificate store" do
+        assert Stripe.ca_store.is_a?(OpenSSL::X509::Store)
+      end
+
+      should "return the max_network_retry_delay" do
+        assert_equal 2, Stripe.max_network_retry_delay
+      end
+
+      should "return the initial_network_retry_delay" do
+        assert_equal 0.5, Stripe.initial_network_retry_delay
+      end
     end
-  end
 
-  should "allow max_network_retries to be configured" do
-    begin
-      old = Stripe.max_network_retries
-      Stripe.max_network_retries = 99
-      assert_equal 99, Stripe.max_network_retries
-    ensure
-      Stripe.max_network_retries = old
+    should "allow ca_bundle_path to be configured" do
+      Stripe::StripeClient.expects(:clear_all_connection_managers)
+      Stripe.ca_bundle_path = "/path/to/ca/bundle"
+      assert_equal "/path/to/ca/bundle", Stripe.ca_bundle_path
     end
-  end
 
-  should "have default open and read timeouts" do
-    assert_equal Stripe.open_timeout, 30
-    assert_equal Stripe.read_timeout, 80
+    should "allow open timeout to be configured" do
+      Stripe.open_timeout = 10
+      assert_equal 10, Stripe.open_timeout
+    end
+
+    should "allow read timeout to be configured" do
+      Stripe.read_timeout = 10
+      assert_equal 10, Stripe.read_timeout
+    end
+
+    should "allow api_key to be configured" do
+      Stripe.api_key = "sk_local_test"
+      assert_equal "sk_local_test", Stripe.api_key
+    end
+
+    should "allow stripe_account to be configured" do
+      Stripe.stripe_account = "acct_1234"
+      assert_equal "acct_1234", Stripe.stripe_account
+    end
+
+    should "allow enable_telemetry to be configured" do
+      begin
+        old = Stripe.enable_telemetry?
+
+        Stripe.enable_telemetry = false
+        assert_equal false, Stripe.enable_telemetry?
+      ensure
+        Stripe.enable_telemetry = old
+      end
+    end
+
+    should "allow log_level to be configured" do
+      Stripe.log_level = "debug"
+      assert_equal ::Logger::DEBUG, Stripe.log_level
+    end
+
+    should "allow logger to be configured" do
+      logger = Object.new
+      Stripe.logger = logger
+      assert_equal logger, Stripe.logger
+    end
+
+    should "allow proxy to be configured" do
+      Stripe.proxy = "http://proxy"
+      assert_equal "http://proxy", Stripe.proxy
+    end
+
+    should "allow uploads_base to be configured" do
+      Stripe.uploads_base = "https://other.stripe.com"
+      assert_equal "https://other.stripe.com", Stripe.uploads_base
+    end
+
+    should "allow api_base to be configured" do
+      Stripe.api_base = "https://other.stripe.com"
+      assert_equal "https://other.stripe.com", Stripe.api_base
+    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
+    end
+
+    should "allow verify_ssl_certs to be configured" do
+      Stripe.verify_ssl_certs = false
+      assert_equal false, Stripe.verify_ssl_certs
+    end
+
+    should "allow client_id to be configured" do
+      Stripe.client_id = "client"
+      assert_equal "client", Stripe.client_id
+    end
   end
 end

data/test/test_helper.rb

@@ -16,7 +16,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.88.0"
+MOCK_MINIMUM_VERSION = "0.95.0"
 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: 5.20.0
+  version: 5.24.0
 platform: ruby
 authors:
 - Stripe
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-04-27 00:00:00.000000000 Z
+date: 2020-08-26 00:00:00.000000000 Z
 dependencies: []
 description: Stripe is the easiest way to accept payments online.  See https://stripe.com
   for details.
@@ -93,6 +93,7 @@ files:
 - lib/stripe/resources/issuing/cardholder.rb
 - lib/stripe/resources/issuing/dispute.rb
 - lib/stripe/resources/issuing/transaction.rb
+- lib/stripe/resources/line_item.rb
 - lib/stripe/resources/login_link.rb
 - lib/stripe/resources/mandate.rb
 - lib/stripe/resources/order.rb
@@ -102,7 +103,9 @@ files:
 - lib/stripe/resources/payout.rb
 - lib/stripe/resources/person.rb
 - lib/stripe/resources/plan.rb
+- lib/stripe/resources/price.rb
 - lib/stripe/resources/product.rb
+- lib/stripe/resources/promotion_code.rb
 - lib/stripe/resources/radar/early_fraud_warning.rb
 - lib/stripe/resources/radar/value_list.rb
 - lib/stripe/resources/radar/value_list_item.rb
@@ -135,6 +138,7 @@ files:
 - lib/stripe/resources/webhook_endpoint.rb
 - lib/stripe/singleton_api_resource.rb
 - lib/stripe/stripe_client.rb
+- lib/stripe/stripe_configuration.rb
 - lib/stripe/stripe_object.rb
 - lib/stripe/stripe_response.rb
 - lib/stripe/util.rb
@@ -191,7 +195,9 @@ files:
 - test/stripe/payout_test.rb
 - test/stripe/person_test.rb
 - test/stripe/plan_test.rb
+- test/stripe/price_test.rb
 - test/stripe/product_test.rb
+- test/stripe/promotion_code_test.rb
 - test/stripe/radar/early_fraud_warning_test.rb
 - test/stripe/radar/value_list_item_test.rb
 - test/stripe/radar/value_list_test.rb
@@ -206,6 +212,7 @@ files:
 - test/stripe/sku_test.rb
 - test/stripe/source_test.rb
 - test/stripe/stripe_client_test.rb
+- test/stripe/stripe_configuration_test.rb
 - test/stripe/stripe_object_test.rb
 - test/stripe/stripe_response_test.rb
 - test/stripe/subscription_item_test.rb
@@ -307,7 +314,9 @@ test_files:
 - test/stripe/payout_test.rb
 - test/stripe/person_test.rb
 - test/stripe/plan_test.rb
+- test/stripe/price_test.rb
 - test/stripe/product_test.rb
+- test/stripe/promotion_code_test.rb
 - test/stripe/radar/early_fraud_warning_test.rb
 - test/stripe/radar/value_list_item_test.rb
 - test/stripe/radar/value_list_test.rb
@@ -322,6 +331,7 @@ test_files:
 - test/stripe/sku_test.rb
 - test/stripe/source_test.rb
 - test/stripe/stripe_client_test.rb
+- test/stripe/stripe_configuration_test.rb
 - test/stripe/stripe_object_test.rb
 - test/stripe/stripe_response_test.rb
 - test/stripe/subscription_item_test.rb