stripe 4.10.0 → 5.0.0

data/test/stripe/issuing/authorization_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../../test_helper", __FILE__)
+require ::File.expand_path("../../test_helper", __dir__)
 
 module Stripe
   module Issuing
@@ -32,18 +32,40 @@ module Stripe
         assert authorization.is_a?(Stripe::Issuing::Authorization)
       end
 
-      should "be approveable" do
-        authorization = Stripe::Issuing::Authorization.retrieve("iauth_123")
-        authorization.approve
-        assert_requested :post, "#{Stripe.api_base}/v1/issuing/authorizations/iauth_123/approve"
-        assert authorization.is_a?(Stripe::Issuing::Authorization)
+      context ".approve" do
+        should "approve an authorization" do
+          payment_intent = Stripe::Issuing::Authorization.approve("iauth_123")
+
+          assert_requested :post, "#{Stripe.api_base}/v1/issuing/authorizations/iauth_123/approve"
+          assert payment_intent.is_a?(Stripe::Issuing::Authorization)
+        end
       end
 
-      should "be declineable" do
-        authorization = Stripe::Issuing::Authorization.retrieve("iauth_123")
-        authorization.decline
-        assert_requested :post, "#{Stripe.api_base}/v1/issuing/authorizations/iauth_123/decline"
-        assert authorization.is_a?(Stripe::Issuing::Authorization)
+      context "#approve" do
+        should "approve an authorization" do
+          authorization = Stripe::Issuing::Authorization.retrieve("iauth_123")
+          authorization.approve
+          assert_requested :post, "#{Stripe.api_base}/v1/issuing/authorizations/iauth_123/approve"
+          assert authorization.is_a?(Stripe::Issuing::Authorization)
+        end
+      end
+
+      context ".decline" do
+        should "decline an authorization" do
+          payment_intent = Stripe::Issuing::Authorization.decline("iauth_123")
+
+          assert_requested :post, "#{Stripe.api_base}/v1/issuing/authorizations/iauth_123/decline"
+          assert payment_intent.is_a?(Stripe::Issuing::Authorization)
+        end
+      end
+
+      context "#decline" do
+        should "decline an authorization" do
+          authorization = Stripe::Issuing::Authorization.retrieve("iauth_123")
+          authorization.decline
+          assert_requested :post, "#{Stripe.api_base}/v1/issuing/authorizations/iauth_123/decline"
+          assert authorization.is_a?(Stripe::Issuing::Authorization)
+        end
       end
     end
   end

data/test/stripe/issuing/card_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../../test_helper", __FILE__)
+require ::File.expand_path("../../test_helper", __dir__)
 
 module Stripe
   module Issuing
@@ -41,12 +41,21 @@ module Stripe
         assert card.is_a?(Stripe::Issuing::Card)
       end
 
-      should "be able to retrieve card details" do
-        card = Stripe::Issuing::Card.retrieve("ic_123")
+      context "#details" do
+        should "retrieve a card's details" do
+          card_details = Stripe::Issuing::Card.details("ic_123")
+          assert_requested :get, "#{Stripe.api_base}/v1/issuing/cards/ic_123/details"
+          assert card_details.is_a?(Stripe::Issuing::CardDetails)
+        end
+      end
 
-        card_details = card.details
-        assert_requested :get, "#{Stripe.api_base}/v1/issuing/cards/ic_123/details"
-        assert card_details.is_a?(Stripe::Issuing::CardDetails)
+      context ".details" do
+        should "retrieve a card's details" do
+          card = Stripe::Issuing::Card.retrieve("ic_123")
+          card_details = card.details
+          assert_requested :get, "#{Stripe.api_base}/v1/issuing/cards/ic_123/details"
+          assert card_details.is_a?(Stripe::Issuing::CardDetails)
+        end
       end
     end
   end

data/test/stripe/issuing/cardholder_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../../test_helper", __FILE__)
+require ::File.expand_path("../../test_helper", __dir__)
 
 module Stripe
   module Issuing

data/test/stripe/issuing/dispute_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../../test_helper", __FILE__)
+require ::File.expand_path("../../test_helper", __dir__)
 
 module Stripe
   module Issuing

data/test/stripe/issuing/transaction_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../../test_helper", __FILE__)
+require ::File.expand_path("../../test_helper", __dir__)
 
 module Stripe
   module Issuing

data/test/stripe/list_object_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../test_helper", __FILE__)
+require ::File.expand_path("../test_helper", __dir__)
 
 module Stripe
   class ListObjectTest < Test::Unit::TestCase
@@ -132,22 +132,6 @@ module Stripe
       next_list = list.previous_page
       assert_equal({ expand: ["data.source"], limit: 3 }, next_list.filters)
     end
-
-    #
-    # backward compatibility
-    #
-
-    # note that the name #all is deprecated, as is using it fetch the next page
-    # in a list
-    should "be able to retrieve full lists given a listobject" do
-      c = Stripe::Charge.all
-      assert c.is_a?(Stripe::ListObject)
-      assert_equal("/v1/charges", c.resource_url)
-      all = c.all
-      assert all.is_a?(Stripe::ListObject)
-      assert_equal("/v1/charges", all.resource_url)
-      assert all.data.is_a?(Array)
-    end
   end
 end
 

data/test/stripe/login_link_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../test_helper", __FILE__)
+require ::File.expand_path("../test_helper", __dir__)
 
 module Stripe
   class LoginLinkTest < Test::Unit::TestCase
@@ -12,7 +12,7 @@ module Stripe
           "data" => [],
           "has_more" => false,
           "object" => "list",
-          "url" =>  "/v1/accounts/acct_123/login_links",
+          "url" => "/v1/accounts/acct_123/login_links",
         },
       }
       @account = Stripe::Account.construct_from(account_fixture)

data/test/stripe/multipart_encoder_test.rb

@@ -0,0 +1,130 @@
+# frozen_string_literal: true
+
+require ::File.expand_path("../test_helper", __dir__)
+
+module Stripe
+  class MultipartEncoderTest < Test::Unit::TestCase
+    should "multipart encode parameters" do
+      Tempfile.create("image.jpg") do |f|
+        f.write "file-content"
+        f.flush
+        f.rewind
+
+        encoder = MultipartEncoder.new
+        encoder.encode(
+          file: f,
+          other_param: "other-param-content"
+        )
+        encoder.close
+        body = encoder.body
+
+        assert_equal <<~BODY.rstrip, body
+          --#{encoder.boundary}\r
+          Content-Disposition: form-data; name="file"; filename="#{::File.basename(f.path)}"\r
+          Content-Type: application/octet-stream\r
+          \r
+          file-content\r
+          --#{encoder.boundary}\r
+          Content-Disposition: form-data; name="other_param"\r
+          \r
+          other-param-content\r
+          --#{encoder.boundary}--
+        BODY
+      end
+    end
+
+    should "encode file-like objects" do
+      klass = Class.new do
+        def read
+          "klass-read-content"
+        end
+      end
+
+      encoder = MultipartEncoder.new
+      encoder.encode(
+        file_like: klass.new
+      )
+      encoder.close
+      body = encoder.body
+
+      assert_equal <<~BODY.rstrip, body
+        --#{encoder.boundary}\r
+        Content-Disposition: form-data; name="file_like"; filename="blob"\r
+        Content-Type: application/octet-stream\r
+        \r
+        klass-read-content\r
+        --#{encoder.boundary}--
+      BODY
+    end
+
+    should "escape quotes and line break characters in parameter names" do
+      encoder = MultipartEncoder.new
+      encoder.encode(
+        %("quoted\n\r") => "content"
+      )
+      encoder.close
+      body = encoder.body
+
+      assert_equal <<~BODY.rstrip, body
+        --#{encoder.boundary}\r
+        Content-Disposition: form-data; name="%22quoted  %22"\r
+        \r
+        content\r
+        --#{encoder.boundary}--
+      BODY
+    end
+
+    context ".encode" do
+      should "provide an easy encoding shortcut" do
+        body, content_type = MultipartEncoder.encode(
+          param: "content"
+        )
+        assert_include body, %(Content-Disposition: form-data; name="param")
+        assert_include content_type, "#{MultipartEncoder::MULTIPART_FORM_DATA}; boundary="
+      end
+    end
+
+    context "#body" do
+      should "error if not yet closed" do
+        encoder = MultipartEncoder.new
+
+        e = assert_raises RuntimeError do
+          encoder.body
+        end
+        assert_equal "object must be closed before getting body", e.message
+      end
+    end
+
+    context "#close" do
+      should "error if closed twice" do
+        encoder = MultipartEncoder.new
+        encoder.close
+
+        e = assert_raises RuntimeError do
+          encoder.close
+        end
+        assert_equal "object already closed", e.message
+      end
+    end
+
+    context "#content_type" do
+      should "produce a content type containing boundary" do
+        encoder = MultipartEncoder.new
+        assert_equal "#{MultipartEncoder::MULTIPART_FORM_DATA}; boundary=#{encoder.boundary}",
+                     encoder.content_type
+      end
+    end
+
+    context "#encode" do
+      should "error if already closed" do
+        encoder = MultipartEncoder.new
+        encoder.close
+
+        e = assert_raises RuntimeError do
+          encoder.encode(param: "content")
+        end
+        assert_equal "no more parameters can be written to closed object", e.message
+      end
+    end
+  end
+end

data/test/stripe/oauth_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../test_helper", __FILE__)
+require ::File.expand_path("../test_helper", __dir__)
 
 module Stripe
   class OAuthTest < Test::Unit::TestCase

data/test/stripe/order_return_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../test_helper", __FILE__)
+require ::File.expand_path("../test_helper", __dir__)
 
 module Stripe
   class OrderReturnTest < Test::Unit::TestCase

data/test/stripe/order_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../test_helper", __FILE__)
+require ::File.expand_path("../test_helper", __dir__)
 
 module Stripe
   class OrderTest < Test::Unit::TestCase
@@ -42,6 +42,22 @@ module Stripe
       should "pay an order" do
         order = Stripe::Order.retrieve("or_123")
         order = order.pay(source: "tok_123")
+        assert_requested :post, "#{Stripe.api_base}/v1/orders/#{order.id}/pay"
+        assert order.is_a?(Stripe::Order)
+      end
+
+      should "pay an order without additional arguments" do
+        order = Stripe::Order.retrieve("or_123")
+        order = order.pay
+        assert_requested :post, "#{Stripe.api_base}/v1/orders/#{order.id}/pay"
+        assert order.is_a?(Stripe::Order)
+      end
+    end
+
+    context ".pay" do
+      should "pay an order" do
+        order = Stripe::Order.pay("or_123", source: "tok_123")
+        assert_requested :post, "#{Stripe.api_base}/v1/orders/or_123/pay"
         assert order.is_a?(Stripe::Order)
       end
     end
@@ -49,8 +65,17 @@ module Stripe
     context "#return_order" do
       should "return an order" do
         order = Stripe::Order.retrieve("or_123")
-        order = order.return_order({})
-        assert order.is_a?(Stripe::OrderReturn)
+        order_return = order.return_order({})
+        assert_requested :post, "#{Stripe.api_base}/v1/orders/#{order.id}/returns"
+        assert order_return.is_a?(Stripe::OrderReturn)
+      end
+    end
+
+    context ".return_order" do
+      should "return an order" do
+        order_return = Stripe::Order.return_order("or_123")
+        assert_requested :post, "#{Stripe.api_base}/v1/orders/or_123/returns"
+        assert order_return.is_a?(Stripe::OrderReturn)
       end
     end
   end

data/test/stripe/payment_intent_test.rb

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../test_helper", __FILE__)
+require ::File.expand_path("../test_helper", __dir__)
 
 module Stripe
-  TEST_RESOURCE_ID = "pi_123".freeze
-
   class PaymentIntentTest < Test::Unit::TestCase
+    TEST_RESOURCE_ID = "pi_123"
+
     should "be listable" do
       payment_intents = Stripe::PaymentIntent.list
       assert_requested :get, "#{Stripe.api_base}/v1/payment_intents"
@@ -53,6 +53,15 @@ module Stripe
       end
     end
 
+    context ".cancel" do
+      should "cancel a payment_intent" do
+        payment_intent = Stripe::PaymentIntent.cancel("pi_123")
+
+        assert_requested :post, "#{Stripe.api_base}/v1/payment_intents/pi_123/cancel"
+        assert payment_intent.is_a?(Stripe::PaymentIntent)
+      end
+    end
+
     context "#capture" do
       should "capture a payment_intent" do
         payment_intent = Stripe::PaymentIntent.construct_from(id: "pi_123", object: "payment_intent")
@@ -65,16 +74,34 @@ module Stripe
       end
     end
 
+    context ".capture" do
+      should "capture a payment_intent" do
+        payment_intent = Stripe::PaymentIntent.capture("pi_123", amount_to_capture: 1234)
+
+        assert_requested :post, "#{Stripe.api_base}/v1/payment_intents/pi_123/capture"
+        assert payment_intent.is_a?(Stripe::PaymentIntent)
+      end
+    end
+
     context "#confirm" do
       should "confirm a payment_intent" do
         payment_intent = Stripe::PaymentIntent.construct_from(id: "pi_123", object: "payment_intent")
         payment_intent = payment_intent.confirm(
-          source: "src_123"
+          payment_method: "pm_123"
         )
 
         assert_requested :post, "#{Stripe.api_base}/v1/payment_intents/pi_123/confirm"
         assert payment_intent.is_a?(Stripe::PaymentIntent)
       end
     end
+
+    context ".confirm" do
+      should "confirm a payment_intent" do
+        payment_intent = Stripe::PaymentIntent.confirm("pi_123", payment_method: "pm_123")
+
+        assert_requested :post, "#{Stripe.api_base}/v1/payment_intents/pi_123/confirm"
+        assert payment_intent.is_a?(Stripe::PaymentIntent)
+      end
+    end
   end
 end

data/test/stripe/payment_method_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../test_helper", __FILE__)
+require ::File.expand_path("../test_helper", __dir__)
 
 module Stripe
   class PaymentMethodTest < Test::Unit::TestCase
@@ -53,6 +53,15 @@ module Stripe
       end
     end
 
+    context ".attach" do
+      should "attach payment_method" do
+        payment_method = Stripe::PaymentMethod.attach("pm_123", customer: "cus_123")
+
+        assert_requested :post, "#{Stripe.api_base}/v1/payment_methods/pm_123/attach"
+        assert payment_method.is_a?(Stripe::PaymentMethod)
+      end
+    end
+
     context "#detach" do
       should "detach payment_method" do
         payment_method = Stripe::PaymentMethod.construct_from(id: "pm_123", object: "payment_method")
@@ -62,5 +71,14 @@ module Stripe
         assert payment_method.is_a?(Stripe::PaymentMethod)
       end
     end
+
+    context ".detach" do
+      should "detach payment_method" do
+        payment_method = Stripe::PaymentMethod.detach("pm_123")
+
+        assert_requested :post, "#{Stripe.api_base}/v1/payment_methods/pm_123/detach"
+        assert payment_method.is_a?(Stripe::PaymentMethod)
+      end
+    end
   end
 end

data/test/stripe/payout_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../test_helper", __FILE__)
+require ::File.expand_path("../test_helper", __dir__)
 
 module Stripe
   class PayoutTest < Test::Unit::TestCase
@@ -46,5 +46,12 @@ module Stripe
         assert payout.is_a?(Stripe::Payout)
       end
     end
+
+    context ".cancel" do
+      should "cancel a payout" do
+        payout = Stripe::Payout.cancel("pm_123")
+        assert payout.is_a?(Stripe::Payout)
+      end
+    end
   end
 end

data/test/stripe/person_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../test_helper", __FILE__)
+require ::File.expand_path("../test_helper", __dir__)
 
 module Stripe
   class PersonTest < Test::Unit::TestCase

data/test/stripe/plan_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../test_helper", __FILE__)
+require ::File.expand_path("../test_helper", __dir__)
 
 module Stripe
   class PlanTest < Test::Unit::TestCase
@@ -12,8 +12,8 @@ module Stripe
     end
 
     should "be retrievable" do
-      plan = Stripe::Plan.retrieve("sapphire-elite")
-      assert_requested :get, "#{Stripe.api_base}/v1/plans/sapphire-elite"
+      plan = Stripe::Plan.retrieve("pl_123")
+      assert_requested :get, "#{Stripe.api_base}/v1/plans/pl_123"
       assert plan.is_a?(Stripe::Plan)
     end
 
@@ -21,9 +21,8 @@ module Stripe
       plan = Stripe::Plan.create(
         amount: 5000,
         interval: "month",
-        name: "Sapphire elite",
-        currency: "usd",
-        id: "sapphire-elite"
+        nickname: "Sapphire elite",
+        currency: "usd"
       )
       assert_requested :post, "#{Stripe.api_base}/v1/plans"
       assert plan.is_a?(Stripe::Plan)
@@ -33,9 +32,8 @@ module Stripe
       plan = Stripe::Plan.create(
         amount: 5000,
         interval: "month",
-        name: "Sapphire elite",
+        nickname: "Sapphire elite",
         currency: "usd",
-        id: "sapphire-elite",
         usage_type: "metered"
       )
       assert_requested :post, "#{Stripe.api_base}/v1/plans"
@@ -45,9 +43,8 @@ module Stripe
     should "be creatable with tiered configuration" do
       plan = Stripe::Plan.create(
         interval: "month",
-        name: "Sapphire elite",
+        nickname: "Sapphire elite",
         currency: "usd",
-        id: "sapphire-elite",
         billing_scheme: "tiered",
         tiers_mode: "volume",
         tiers: [{ up_to: 100, amount: 1000 }, { up_to: "inf", amount: 2000 }]
@@ -59,9 +56,8 @@ module Stripe
     should "be creatable with transform_usage" do
       plan = Stripe::Plan.create(
         interval: "month",
-        name: "Sapphire elite",
+        nickname: "Sapphire elite",
         currency: "usd",
-        id: "sapphire-elite",
         amount: 5000,
         transform_usage: { round: "up", divide_by: 50 }
       )
@@ -70,23 +66,33 @@ module Stripe
     end
 
     should "be saveable" do
-      plan = Stripe::Plan.retrieve("sapphire-elite")
+      plan = Stripe::Plan.retrieve("pl_123")
       plan.metadata["key"] = "value"
       plan.save
       assert_requested :post, "#{Stripe.api_base}/v1/plans/#{plan.id}"
     end
 
     should "be updateable" do
-      plan = Stripe::Plan.update("sapphire-elite", metadata: { foo: "bar" })
-      assert_requested :post, "#{Stripe.api_base}/v1/plans/sapphire-elite"
+      plan = Stripe::Plan.update("pl_123", metadata: { foo: "bar" })
+      assert_requested :post, "#{Stripe.api_base}/v1/plans/pl_123"
       assert plan.is_a?(Stripe::Plan)
     end
 
-    should "be deletable" do
-      plan = Stripe::Plan.retrieve("sapphire-elite")
-      plan = plan.delete
-      assert_requested :delete, "#{Stripe.api_base}/v1/plans/#{plan.id}"
-      assert plan.is_a?(Stripe::Plan)
+    context "#delete" do
+      should "be deletable" do
+        plan = Stripe::Plan.retrieve("pl_123")
+        plan = plan.delete
+        assert_requested :delete, "#{Stripe.api_base}/v1/plans/#{plan.id}"
+        assert plan.is_a?(Stripe::Plan)
+      end
+    end
+
+    context ".delete" do
+      should "be deletable" do
+        plan = Stripe::Plan.delete("pl_123")
+        assert_requested :delete, "#{Stripe.api_base}/v1/plans/pl_123"
+        assert plan.is_a?(Stripe::Plan)
+      end
     end
   end
 end

data/test/stripe/product_test.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../test_helper", __FILE__)
+require ::File.expand_path("../test_helper", __dir__)
 
 module Stripe
   class ProductTest < Test::Unit::TestCase
@@ -39,11 +39,21 @@ module Stripe
       assert product.is_a?(Stripe::Product)
     end
 
-    should "be deletable" do
-      product = Stripe::Product.retrieve("prod_123")
-      product = product.delete
-      assert_requested :delete, "#{Stripe.api_base}/v1/products/#{product.id}"
-      assert product.is_a?(Stripe::Product)
+    context "#delete" do
+      should "be deletable" do
+        product = Stripe::Product.retrieve("prod_123")
+        product = product.delete
+        assert_requested :delete, "#{Stripe.api_base}/v1/products/#{product.id}"
+        assert product.is_a?(Stripe::Product)
+      end
+    end
+
+    context ".delete" do
+      should "be deletable" do
+        product = Stripe::Product.delete("prod_123")
+        assert_requested :delete, "#{Stripe.api_base}/v1/products/prod_123"
+        assert product.is_a?(Stripe::Product)
+      end
     end
   end
 end

data/test/stripe/radar/early_fraud_warning_test.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+require ::File.expand_path("../../test_helper", __dir__)
+
+module Stripe
+  module Radar
+    class EarlyFraudWarningTest < Test::Unit::TestCase
+      should "be listable" do
+        warnings = Stripe::Radar::EarlyFraudWarning.list
+        assert_requested :get, "#{Stripe.api_base}/v1/radar/early_fraud_warnings"
+        assert warnings.data.is_a?(Array)
+        assert warnings.data[0].is_a?(Stripe::Radar::EarlyFraudWarning)
+      end
+
+      should "be retrievable" do
+        warning = Stripe::Radar::EarlyFraudWarning.retrieve("issfr_123")
+        assert_requested :get, "#{Stripe.api_base}/v1/radar/early_fraud_warnings/issfr_123"
+        assert warning.is_a?(Stripe::Radar::EarlyFraudWarning)
+      end
+    end
+  end
+end