stripe 5.21.0 → 5.25.0

data/lib/stripe/stripe_object.rb

@@ -375,7 +375,7 @@ module Stripe
       begin
         super
       rescue NoMethodError => e
-        # If we notice the accessed name if our set of transient values we can
+        # If we notice the accessed name of our set of transient values we can
         # give the user a slightly more helpful error message. If not, just
         # raise right away.
         raise unless @transient_values.include?(name)

data/lib/stripe/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Stripe
-  VERSION = "5.21.0"
+  VERSION = "5.25.0"
 end

data/test/stripe/account_link_test.rb

@@ -7,9 +7,9 @@ module Stripe
     should "be creatable" do
       link = Stripe::AccountLink.create(
         account: "acct_123",
-        failure_url: "https://stripe.com/failure",
-        success_url: "https://stripe.com/success",
-        type: "custom_account_verification"
+        refresh_url: "https://stripe.com/refresh",
+        return_url: "https://stripe.com/return",
+        type: "account_onboarding"
       )
       assert_requested :post, "#{Stripe.api_base}/v1/account_links"
       assert link.is_a?(Stripe::AccountLink)

data/test/stripe/api_resource_test.rb

@@ -111,15 +111,15 @@ module Stripe
     end
 
     should "send expand when fetching through ListObject" do
-      stub_request(:get, "#{Stripe.api_base}/v1/customers/cus_123")
-        .to_return(body: JSON.generate(customer_fixture))
+      stub_request(:get, "#{Stripe.api_base}/v1/charges/ch_123")
+        .to_return(body: JSON.generate(charge_fixture))
 
-      stub_request(:get, "#{Stripe.api_base}/v1/customers/cus_123/sources/cc_test_card")
-        .with(query: { "expand" => ["customer"] })
-        .to_return(body: JSON.generate(customer_fixture))
+      stub_request(:get, "#{Stripe.api_base}/v1/charges/ch_123/refunds/re_123")
+        .with(query: { "expand" => ["balance_transaction"] })
+        .to_return(body: JSON.generate(charge_fixture))
 
-      customer = Stripe::Customer.retrieve("cus_123")
-      customer.sources.retrieve(id: "cc_test_card", expand: [:customer])
+      charge = Stripe::Charge.retrieve("ch_123")
+      charge.refunds.retrieve(id: "re_123", expand: [:balance_transaction])
     end
 
     context "when specifying per-object credentials" do

data/test/stripe/checkout/session_test.rb

@@ -36,6 +36,18 @@ module Stripe
         assert_requested :get, "#{Stripe.api_base}/v1/checkout/sessions/cs_123"
         assert charge.is_a?(Stripe::Checkout::Session)
       end
+
+      context "#list_line_items" do
+        should "list the session's line items" do
+          sources = Stripe::Checkout::Session.list_line_items(
+            "cs_123"
+          )
+          assert_requested :get, "#{Stripe.api_base}/v1/checkout/sessions/cs_123/line_items"
+          assert sources.is_a?(Stripe::ListObject)
+          assert sources.data.is_a?(Array)
+          assert sources.data[0].is_a?(Stripe::LineItem)
+        end
+      end
     end
   end
 end

data/test/stripe/customer_card_test.rb

@@ -5,6 +5,12 @@ require ::File.expand_path("../test_helper", __dir__)
 module Stripe
   class CustomerCardTest < Test::Unit::TestCase
     setup do
+      # Unfortunately, the OpenAPI spec has an issue where the sources list has the wrong
+      # url so we need to mock this call instead.
+      customer_json = { id: "cus_123", object: "customer", sources: { object: "list", data: [], has_more: true, url: "/v1/customers/cus_123/sources" } }
+      stub_request(:get, "#{Stripe.api_base}/v1/customers/cus_123")
+        .to_return(body: JSON.generate(customer_json))
+
       @customer = Stripe::Customer.retrieve("cus_123")
     end
 

data/test/stripe/issuing/dispute_test.rb

@@ -6,7 +6,7 @@ module Stripe
   module Issuing
     class DisputeTest < Test::Unit::TestCase
       should "be creatable" do
-        dispute = Stripe::Issuing::Dispute.create
+        dispute = Stripe::Issuing::Dispute.create(transaction: "ipi_123")
 
         assert_requested :post, "#{Stripe.api_base}/v1/issuing/disputes"
         assert dispute.is_a?(Stripe::Issuing::Dispute)
@@ -30,6 +30,25 @@ module Stripe
         assert_requested :post, "#{Stripe.api_base}/v1/issuing/disputes/ich_123"
         assert dispute.is_a?(Stripe::Issuing::Dispute)
       end
+
+      context "#submit" do
+        should "submit the dispute" do
+          dispute = Stripe::Issuing::Dispute.retrieve("idp_123")
+          dispute = dispute.submit
+          assert_requested :post,
+                           "#{Stripe.api_base}/v1/issuing/disputes/idp_123/submit"
+          assert dispute.is_a?(Stripe::Issuing::Dispute)
+        end
+      end
+
+      context ".submit" do
+        should "submit the dispute" do
+          dispute = Stripe::Issuing::Dispute.submit("idp_123")
+          assert_requested :post,
+                           "#{Stripe.api_base}/v1/issuing/disputes/idp_123/submit"
+          assert dispute.is_a?(Stripe::Issuing::Dispute)
+        end
+      end
     end
   end
 end

data/test/stripe/promotion_code_test.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+require ::File.expand_path("../test_helper", __dir__)
+
+module Stripe
+  class PromotionCodeTest < Test::Unit::TestCase
+    should "be listable" do
+      promotion_codes = Stripe::PromotionCode.list
+      assert_requested :get, "#{Stripe.api_base}/v1/promotion_codes"
+      assert promotion_codes.data.is_a?(Array)
+      assert promotion_codes.first.is_a?(Stripe::PromotionCode)
+    end
+
+    should "be retrievable" do
+      coupon = Stripe::PromotionCode.retrieve("PROMO_123")
+      assert_requested :get, "#{Stripe.api_base}/v1/promotion_codes/PROMO_123"
+      assert coupon.is_a?(Stripe::PromotionCode)
+    end
+
+    should "be creatable" do
+      coupon = Stripe::PromotionCode.create(
+        coupon: "co_123",
+        code: "MYCODE"
+      )
+      assert_requested :post, "#{Stripe.api_base}/v1/promotion_codes"
+      assert coupon.is_a?(Stripe::PromotionCode)
+    end
+
+    should "be saveable" do
+      coupon = Stripe::PromotionCode.retrieve("PROMO_123")
+      coupon.metadata["key"] = "value"
+      coupon.save
+      assert_requested :post, "#{Stripe.api_base}/v1/promotion_codes/#{coupon.id}"
+    end
+
+    should "be updateable" do
+      coupon = Stripe::PromotionCode.update("PROMO_123", metadata: { key: "value" })
+      assert_requested :post, "#{Stripe.api_base}/v1/promotion_codes/PROMO_123"
+      assert coupon.is_a?(Stripe::PromotionCode)
+    end
+  end
+end

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.89.0"
+MOCK_MINIMUM_VERSION = "0.98.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.21.0
+  version: 5.25.0
 platform: ruby
 authors:
 - Stripe
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-04-29 00:00:00.000000000 Z
+date: 2020-09-02 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
@@ -104,6 +105,7 @@ files:
 - 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
@@ -136,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
@@ -194,6 +197,7 @@ files:
 - 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
@@ -208,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
@@ -311,6 +316,7 @@ test_files:
 - 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
@@ -325,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