Merge branch 'master' into pakrym/Suppor_testing_methods_that_hit_fil…

…es_endpoint

OPENAPI_VERSION

@@ -1 +1 @@
-v413
+v425

lib/stripe/resources/quote.rb

@@ -56,6 +56,18 @@ def list_line_items(params = {}, opts = {})
       )
     end
 
+    def pdf(params = {}, opts = {}, &read_body_chunk_block)
+      config = opts[:client]&.config || Stripe.config
+      opts = { api_base: config.uploads_base }.merge(opts)
+      request_stream(
+        method: :get,
+        path: format("/v1/quotes/%<quote>s/pdf", { quote: CGI.escape(self["id"]) }),
+        params: params,
+        opts: opts,
+        &read_body_chunk_block
+      )
+    end
+
     def self.accept(quote, params = {}, opts = {})
       request_stripe_object(
         method: :post,
@@ -101,46 +113,16 @@ def self.list_line_items(quote, params = {}, opts = {})
       )
     end
 
-    def pdf(params = {}, opts = {}, &read_body_chunk_block)
-      unless block_given?
-        raise ArgumentError, "A read_body_chunk_block block parameter is required when calling the pdf method."
-      end
-
+    def self.pdf(quote, params = {}, opts = {}, &read_body_chunk_block)
       config = opts[:client]&.config || Stripe.config
-
-      request_stream(
-        method: :get,
-        path: resource_url + "/pdf",
-        params: params,
-        opts: {
-          api_base: config.uploads_base,
-        }.merge(opts),
-        &read_body_chunk_block
-      )
-    end
-
-    def self.pdf(id, params = {}, opts = {}, &read_body_chunk_block)
-      unless id.is_a?(String)
-        raise ArgumentError,
-              "id should be a string representing the ID of an API resource"
-      end
-
-      unless block_given?
-        raise ArgumentError, "A read_body_chunk_block block parameter is required when calling the pdf method."
-      end
-
-      config = opts[:client]&.config || Stripe.config
-
-      resp = execute_resource_request_stream(
+      opts = { api_base: config.uploads_base }.merge(opts)
+      execute_resource_request_stream(
         :get,
-        "#{resource_url}/#{CGI.escape(id)}/pdf",
+        format("/v1/quotes/%<quote>s/pdf", { quote: CGI.escape(quote) }),
         params,
-        {
-          api_base: config.uploads_base,
-        }.merge(opts),
+        opts,
         &read_body_chunk_block
       )
-      resp
     end
   end
 end

lib/stripe/resources/shipping_rate.rb

@@ -2,9 +2,8 @@
 # frozen_string_literal: true
 
 module Stripe
-  # Shipping rates describe the price of shipping presented to your customers and can be
-  # applied to [Checkout Sessions](https://stripe.com/docs/payments/checkout/shipping)
-  # and [Orders](https://stripe.com/docs/orders/shipping) to collect shipping costs.
+  # Shipping rates describe the price of shipping presented to your customers and
+  # applied to a purchase. For more information, see [Charge for shipping](https://stripe.com/docs/payments/during-payment/charge-shipping).
   class ShippingRate < APIResource
     extend Stripe::APIOperations::Create
     extend Stripe::APIOperations::List

test/stripe/generated_examples_test.rb

@@ -30,6 +30,12 @@ class CodegennedExampleTest < Test::Unit::TestCase
         assert_requested :get, "#{Stripe.api_base}/v1/accounts?limit=3"
       end
     end
+    context "Account.persons" do
+      should "support requests with args: limit, parent_id" do
+        Stripe::Account.persons("acct_xxxxxxxxxxxxx", { limit: 3 })
+        assert_requested :get, "#{Stripe.api_base}/v1/accounts/acct_xxxxxxxxxxxxx/persons?limit=3"
+      end
+    end
     context "Account.reject" do
       should "support requests with args: reason, id" do
         Stripe::Account.reject("acct_xxxxxxxxxxxxx", { reason: "fraud" })
@@ -605,6 +611,62 @@ class CodegennedExampleTest < Test::Unit::TestCase
         assert_requested :get, "#{Stripe.api_base}/v1/events/evt_xxxxxxxxxxxxx?"
       end
     end
+    context "ExternalAccount.create" do
+      should "support requests with args: external_account, parent_id" do
+        Stripe::Account.create_external_account(
+          "acct_xxxxxxxxxxxxx",
+          { external_account: "btok_xxxxxxxxxxxxx" }
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/accounts/acct_xxxxxxxxxxxxx/external_accounts"
+      end
+      should "support requests with args: external_account, parent_id2" do
+        Stripe::Account.create_external_account(
+          "acct_xxxxxxxxxxxxx",
+          { external_account: "tok_xxxx_debit" }
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/accounts/acct_xxxxxxxxxxxxx/external_accounts"
+      end
+    end
+    context "ExternalAccount.delete" do
+      should "support requests with args: parent_id, id" do
+        Stripe::Account.delete_external_account(
+          "acct_xxxxxxxxxxxxx",
+          "ba_xxxxxxxxxxxxx"
+        )
+        assert_requested :delete, "#{Stripe.api_base}/v1/accounts/acct_xxxxxxxxxxxxx/external_accounts/ba_xxxxxxxxxxxxx?"
+      end
+      should "support requests with args: parent_id, id2" do
+        Stripe::Account.delete_external_account(
+          "acct_xxxxxxxxxxxxx",
+          "card_xxxxxxxxxxxxx"
+        )
+        assert_requested :delete, "#{Stripe.api_base}/v1/accounts/acct_xxxxxxxxxxxxx/external_accounts/card_xxxxxxxxxxxxx?"
+      end
+    end
+    context "ExternalAccount.list" do
+      should "support requests with args: limit, parent_id" do
+        Stripe::Account.list_external_accounts("acct_xxxxxxxxxxxxx", { limit: 3 })
+        assert_requested :get, "#{Stripe.api_base}/v1/accounts/acct_xxxxxxxxxxxxx/external_accounts?limit=3"
+      end
+    end
+    context "ExternalAccount.update" do
+      should "support requests with args: metadata, parent_id, id" do
+        Stripe::Account.update_external_account(
+          "acct_xxxxxxxxxxxxx",
+          "ba_xxxxxxxxxxxxx",
+          { metadata: { order_id: "6735" } }
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/accounts/acct_xxxxxxxxxxxxx/external_accounts/ba_xxxxxxxxxxxxx"
+      end
+      should "support requests with args: metadata, parent_id, id2" do
+        Stripe::Account.update_external_account(
+          "acct_xxxxxxxxxxxxx",
+          "card_xxxxxxxxxxxxx",
+          { metadata: { order_id: "6735" } }
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/accounts/acct_xxxxxxxxxxxxx/external_accounts/card_xxxxxxxxxxxxx"
+      end
+    end
     context "File.list" do
       should "support requests with args: limit" do
         Stripe::File.list({ limit: 3 })
@@ -1306,30 +1368,58 @@ class CodegennedExampleTest < Test::Unit::TestCase
         assert_requested :post, "#{Stripe.api_base}/v1/payment_methods/pm_xxxxxxxxxxxxx"
       end
     end
-    context "PaymentSource.update" do
-      should "support requests with args: customer, card, account_holder_name" do
-        Stripe::Customer.update_source(
-          "cus_123",
-          "card_123",
-          { account_holder_name: "Kamil" }
+    context "PaymentSource.create" do
+      should "support requests with args: source, parent_id" do
+        Stripe::Customer.create_source(
+          "cus_xxxxxxxxxxxxx",
+          { source: "btok_xxxxxxxxxxxxx" }
         )
-        assert_requested :post, "#{Stripe.api_base}/v1/customers/cus_123/sources/card_123"
+        assert_requested :post, "#{Stripe.api_base}/v1/customers/cus_xxxxxxxxxxxxx/sources"
       end
-      should "support requests with args: metadata, parent_id, id" do
-        Stripe::Customer.update_source(
+      should "support requests with args: source, parent_id2" do
+        Stripe::Customer.create_source(
           "cus_xxxxxxxxxxxxx",
-          "ba_xxxxxxxxxxxxx",
-          { metadata: { order_id: "6735" } }
+          { source: "tok_xxxx" }
         )
-        assert_requested :post, "#{Stripe.api_base}/v1/customers/cus_xxxxxxxxxxxxx/sources/ba_xxxxxxxxxxxxx"
+        assert_requested :post, "#{Stripe.api_base}/v1/customers/cus_xxxxxxxxxxxxx/sources"
       end
-      should "support requests with args: name, parent_id, id" do
-        Stripe::Customer.update_source(
+    end
+    context "PaymentSource.list" do
+      should "support requests with args: object, limit, parent_id" do
+        Stripe::Customer.list_sources(
           "cus_xxxxxxxxxxxxx",
-          "card_xxxxxxxxxxxxx",
-          { name: "Jenny Rosen" }
+          {
+            object: "bank_account",
+            limit: 3,
+          }
         )
-        assert_requested :post, "#{Stripe.api_base}/v1/customers/cus_xxxxxxxxxxxxx/sources/card_xxxxxxxxxxxxx"
+        assert_requested :get, "#{Stripe.api_base}/v1/customers/cus_xxxxxxxxxxxxx/sources?object=bank_account&limit=3"
+      end
+      should "support requests with args: object, limit, parent_id2" do
+        Stripe::Customer.list_sources(
+          "cus_xxxxxxxxxxxxx",
+          {
+            object: "card",
+            limit: 3,
+          }
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/customers/cus_xxxxxxxxxxxxx/sources?object=card&limit=3"
+      end
+    end
+    context "PaymentSource.retrieve" do
+      should "support requests with args: parent_id, id" do
+        Stripe::Customer.retrieve_source(
+          "cus_xxxxxxxxxxxxx",
+          "ba_xxxxxxxxxxxxx"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/customers/cus_xxxxxxxxxxxxx/sources/ba_xxxxxxxxxxxxx?"
+      end
+      should "support requests with args: parent_id, id2" do
+        Stripe::Customer.retrieve_source(
+          "cus_xxxxxxxxxxxxx",
+          "card_xxxxxxxxxxxxx"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/customers/cus_xxxxxxxxxxxxx/sources/card_xxxxxxxxxxxxx?"
       end
     end
     context "Payout.cancel" do
@@ -1374,12 +1464,6 @@ class CodegennedExampleTest < Test::Unit::TestCase
         assert_requested :post, "#{Stripe.api_base}/v1/payouts/po_xxxxxxxxxxxxx"
       end
     end
-    context "Person.list" do
-      should "support requests with args: limit, parent_id" do
-        Stripe::Account.list_persons("acct_xxxxxxxxxxxxx", { limit: 3 })
-        assert_requested :get, "#{Stripe.api_base}/v1/accounts/acct_xxxxxxxxxxxxx/persons?limit=3"
-      end
-    end
     context "Person.retrieve" do
       should "support requests with args: parent_id, id" do
         Stripe::Account.retrieve_person(