stripe 4.9.0 → 5.1.0

data/test/stripe/customer_balance_transaction_test.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+require ::File.expand_path("../../test_helper", __FILE__)
+
+module Stripe
+  class CustomerBalanceTransactionTest < Test::Unit::TestCase
+    context "#resource_url" do
+      should "return a resource URL" do
+        transaction = Stripe::CustomerBalanceTransaction.construct_from(
+          id: "cbtxn_123",
+          customer: "cus_123"
+        )
+        assert_equal "/v1/customers/cus_123/balance_transactions/cbtxn_123",
+                     transaction.resource_url
+      end
+
+      should "raise without a customer" do
+        transaction = Stripe::CustomerBalanceTransaction.construct_from(id: "cbtxn_123")
+        assert_raises NotImplementedError do
+          transaction.resource_url
+        end
+      end
+    end
+
+    should "raise on #retrieve" do
+      assert_raises NotImplementedError do
+        Stripe::CustomerBalanceTransaction.retrieve("cbtxn_123")
+      end
+    end
+
+    should "raise on #update" do
+      assert_raises NotImplementedError do
+        Stripe::CustomerBalanceTransaction.update("cbtxn_123")
+      end
+    end
+  end
+end

data/test/stripe/customer_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
   class CustomerCardTest < Test::Unit::TestCase

data/test/stripe/customer_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 CustomerTest < Test::Unit::TestCase
@@ -36,63 +36,89 @@ module Stripe
       assert customer.is_a?(Stripe::Customer)
     end
 
-    should "be deletable" do
-      customer = Stripe::Customer.retrieve("cus_123")
-      customer = customer.delete
-      assert_requested :delete, "#{Stripe.api_base}/v1/customers/#{customer.id}"
-      assert customer.is_a?(Stripe::Customer)
+    context "#delete" do
+      should "be deletable" do
+        customer = Stripe::Customer.retrieve("cus_123")
+        customer = customer.delete
+        assert_requested :delete, "#{Stripe.api_base}/v1/customers/#{customer.id}"
+        assert customer.is_a?(Stripe::Customer)
+      end
     end
 
-    context "#create_subscription" do
-      should "create a new subscription" do
-        customer = Stripe::Customer.retrieve("cus_123")
-        subscription = customer.create_subscription(items: [{ plan: "silver" }])
-        assert subscription.is_a?(Stripe::Subscription)
+    context ".delete" do
+      should "be deletable" do
+        customer = Stripe::Customer.delete("cus_123")
+        assert_requested :delete, "#{Stripe.api_base}/v1/customers/cus_123"
+        assert customer.is_a?(Stripe::Customer)
       end
     end
 
-    context "#create_upcoming_invoice" do
-      should "create a new invoice" do
+    context "#delete_discount" do
+      should "delete a discount" do
         customer = Stripe::Customer.retrieve("cus_123")
-        invoice = customer.create_upcoming_invoice
-        assert invoice.is_a?(Stripe::Invoice)
+        customer = customer.delete_discount
+        assert_requested :delete, "#{Stripe.api_base}/v1/customers/#{customer.id}/discount"
+        assert customer.is_a?(Stripe::Customer)
       end
     end
 
-    context "#update_subscription" do
-      should "update a subscription" do
-        customer = Stripe::Customer.retrieve("cus_123")
+    context ".delete_discount" do
+      should "delete a discount" do
+        discount = Stripe::Customer.delete_discount("cus_123")
+        assert_requested :delete, "#{Stripe.api_base}/v1/customers/cus_123/discount"
+        assert discount.is_a?(Stripe::Discount)
+      end
+    end
 
-        # deprecated API and not in schema
-        stub_request(:post, "#{Stripe.api_base}/v1/customers/#{customer.id}/subscription")
-          .with(body: { plan: "silver" })
-          .to_return(body: JSON.generate(object: "subscription"))
-        subscription = customer.update_subscription(plan: "silver")
-        assert subscription.is_a?(Stripe::Subscription)
+    context "#create_source" do
+      should "create a source" do
+        Stripe::Customer.create_source(
+          "cus_123",
+          source: "tok_123"
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/customers/cus_123/sources"
       end
     end
 
-    context "#cancel_subscription" do
-      should "cancel a subscription" do
-        customer = Stripe::Customer.retrieve("cus_123")
+    context "#retrieve_source" do
+      should "retrieve a source" do
+        Stripe::Customer.retrieve_source(
+          "cus_123",
+          "ba_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/customers/cus_123/sources/ba_123"
+      end
+    end
 
-        # deprecated API and not in schema
-        stub_request(:delete, "#{Stripe.api_base}/v1/customers/#{customer.id}/subscription")
-          .with(query: { at_period_end: "true" })
-          .to_return(body: JSON.generate(object: "subscription"))
-        subscription = customer.cancel_subscription(at_period_end: "true")
-        assert subscription.is_a?(Stripe::Subscription)
+    context "#update_source" do
+      should "update a source" do
+        Stripe::Customer.update_source(
+          "cus_123",
+          "ba_123",
+          metadata: { foo: "bar" }
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/customers/cus_123/sources/ba_123"
       end
     end
 
-    context "#delete_discount" do
-      should "delete a discount" do
-        customer = Stripe::Customer.retrieve("cus_123")
+    context "#delete_source" do
+      should "delete a source" do
+        Stripe::Customer.delete_source(
+          "cus_123",
+          "ba_123"
+        )
+        assert_requested :delete, "#{Stripe.api_base}/v1/customers/cus_123/sources/ba_123"
+      end
+    end
 
-        stub_request(:delete, "#{Stripe.api_base}/v1/customers/#{customer.id}/discount")
-          .to_return(body: JSON.generate(object: "customer"))
-        discount = customer.delete_discount
-        assert discount.is_a?(Stripe::Customer)
+    context "#list_sources" do
+      should "list the customer's sources" do
+        sources = Stripe::Customer.list_sources(
+          "cus_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/customers/cus_123/sources"
+        assert sources.is_a?(Stripe::ListObject)
+        assert sources.data.is_a?(Array)
       end
     end
 
@@ -111,5 +137,90 @@ module Stripe
         assert_equal true, c.source.save_with_parent
       end
     end
+
+    context "#create_tax_id" do
+      should "create a tax id" do
+        Stripe::Customer.create_tax_id(
+          "cus_123",
+          type: "eu_vat",
+          value: "11111"
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/customers/cus_123/tax_ids"
+      end
+    end
+
+    context "#retrieve_tax_id" do
+      should "retrieve a tax id" do
+        Stripe::Customer.retrieve_tax_id(
+          "cus_123",
+          "txi_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/customers/cus_123/tax_ids/txi_123"
+      end
+    end
+
+    context "#delete_tax_id" do
+      should "delete a tax id" do
+        Stripe::Customer.delete_tax_id(
+          "cus_123",
+          "txi_123"
+        )
+        assert_requested :delete, "#{Stripe.api_base}/v1/customers/cus_123/tax_ids/txi_123"
+      end
+    end
+
+    context "#list_tax_ids" do
+      should "list the customer's tax ids" do
+        sources = Stripe::Customer.list_tax_ids(
+          "cus_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/customers/cus_123/tax_ids"
+        assert sources.is_a?(Stripe::ListObject)
+        assert sources.data.is_a?(Array)
+      end
+    end
+
+    context "#create_balance_transaction" do
+      should "create a customer balance transaction" do
+        Stripe::Customer.create_balance_transaction(
+          "cus_123",
+          amount: 1234,
+          currency: "usd"
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/customers/cus_123/balance_transactions"
+      end
+    end
+
+    context "#retrieve_balance_transaction" do
+      should "retrieve a customer balance transaction" do
+        Stripe::Customer.retrieve_balance_transaction(
+          "cus_123",
+          "cbtxn_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/customers/cus_123/balance_transactions/cbtxn_123"
+      end
+    end
+
+    context "#update_balance_transaction" do
+      should "update a customer balance transaction" do
+        Stripe::Customer.update_balance_transaction(
+          "cus_123",
+          "cbtxn_123",
+          description: "new"
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/customers/cus_123/balance_transactions/cbtxn_123"
+      end
+    end
+
+    context "#list_balance_transactions" do
+      should "list the customer balance transactions" do
+        sources = Stripe::Customer.list_balance_transactions(
+          "cus_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/customers/cus_123/balance_transactions"
+        assert sources.is_a?(Stripe::ListObject)
+        assert sources.data.is_a?(Array)
+      end
+    end
   end
 end

data/test/stripe/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
   class DisputeTest < Test::Unit::TestCase
@@ -36,6 +36,15 @@ module Stripe
         dispute.close
         assert_requested :post,
                          "#{Stripe.api_base}/v1/disputes/#{dispute.id}/close"
+        assert dispute.is_a?(Stripe::Dispute)
+      end
+    end
+
+    context ".close" do
+      should "close a dispute" do
+        dispute = Stripe::Dispute.close("dp_123")
+        assert_requested :post, "#{Stripe.api_base}/v1/disputes/dp_123/close"
+        assert dispute.is_a?(Stripe::Dispute)
       end
     end
   end

data/test/stripe/ephemeral_key_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 EphemeralKeyTest < Test::Unit::TestCase
@@ -82,5 +82,12 @@ module Stripe
         assert_requested :delete, "#{Stripe.api_base}/v1/ephemeral_keys/#{key.id}"
       end
     end
+
+    context ".delete" do
+      should "succeed" do
+        Stripe::EphemeralKey.delete("ephkey_123")
+        assert_requested :delete, "#{Stripe.api_base}/v1/ephemeral_keys/ephkey_123"
+      end
+    end
   end
 end

data/test/stripe/errors_test.rb

@@ -1,19 +1,40 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../test_helper", __FILE__)
+require ::File.expand_path("../test_helper", __dir__)
 
 module Stripe
   class StripeErrorTest < Test::Unit::TestCase
-    context "#to_s" do
-      should "convert to string" do
-        e = StripeError.new("message")
-        assert_equal "message", e.to_s
+    context "StripeError" do
+      context "#initialize" do
+        should "initialize error if json_body is set" do
+          e = StripeError.new("message", json_body: { error: { code: "some_error" } })
+          assert_not_nil e.error
+          assert_equal "some_error", e.error.code
+          assert_nil e.error.charge
+        end
+      end
+
+      context "#to_s" do
+        should "convert to string" do
+          e = StripeError.new("message")
+          assert_equal "message", e.to_s
+
+          e = StripeError.new("message", http_status: 200)
+          assert_equal "(Status 200) message", e.to_s
 
-        e = StripeError.new("message", http_status: 200)
-        assert_equal "(Status 200) message", e.to_s
+          e = StripeError.new("message", http_status: nil, http_body: nil, json_body: nil, http_headers: { request_id: "request-id" })
+          assert_equal "(Request request-id) message", e.to_s
+        end
+      end
+    end
 
-        e = StripeError.new("message", http_status: nil, http_body: nil, json_body: nil, http_headers: { request_id: "request-id" })
-        assert_equal "(Request request-id) message", e.to_s
+    context "OAuth::OAuthError" do
+      context "#initialize" do
+        should "initialize error if json_body is set" do
+          e = OAuth::OAuthError.new("message", "description", json_body: { error: "some_oauth_error" })
+          assert_not_nil e.error
+          assert_equal "some_oauth_error", e.error.error
+        end
       end
     end
   end

data/test/stripe/exchange_rate_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 ExchangeRateTest < Test::Unit::TestCase

data/test/stripe/file_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 FileLinkTest < Test::Unit::TestCase

data/test/stripe/file_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 FileTest < Test::Unit::TestCase
@@ -31,7 +31,8 @@ module Stripe
       should "be creatable with a File" do
         file = Stripe::File.create(
           purpose: "dispute_evidence",
-          file: ::File.new(__FILE__)
+          file: ::File.new(__FILE__),
+          file_link_data: { create: true }
         )
         assert_requested :post, "#{Stripe.uploads_base}/v1/files"
         assert file.is_a?(Stripe::File)
@@ -44,20 +45,33 @@ module Stripe
 
         file = Stripe::File.create(
           purpose: "dispute_evidence",
-          file: tempfile
+          file: tempfile,
+          file_link_data: { create: true }
         )
         assert_requested :post, "#{Stripe.uploads_base}/v1/files"
         assert file.is_a?(Stripe::File)
       end
 
-      should "be creatable with Faraday::UploadIO" do
+      should "be creatable with a string" do
         file = Stripe::File.create(
           purpose: "dispute_evidence",
-          file: Faraday::UploadIO.new(::File.new(__FILE__), nil)
+          file: "my-file-contents",
+          file_link_data: { create: true }
         )
         assert_requested :post, "#{Stripe.uploads_base}/v1/files"
         assert file.is_a?(Stripe::File)
       end
+
+      should "raise given a file object that doesn't respond to #read" do
+        e = assert_raises(ArgumentError) do
+          Stripe::File.create(
+            purpose: "dispute_evidence",
+            file: Object.new,
+            file_link_data: { create: true }
+          )
+        end
+        assert_equal "file must respond to `#read`", e.message
+      end
     end
 
     should "be deserializable when `object=file`" do

data/test/stripe/invoice_item_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 InvoiceItemTest < Test::Unit::TestCase
@@ -44,12 +44,23 @@ module Stripe
       assert item.is_a?(Stripe::InvoiceItem)
     end
 
-    should "be deletable" do
-      item = Stripe::InvoiceItem.retrieve("ii_123")
-      item = item.delete
-      assert_requested :delete,
-                       "#{Stripe.api_base}/v1/invoiceitems/#{item.id}"
-      assert item.is_a?(Stripe::InvoiceItem)
+    context "#delete" do
+      should "be deletable" do
+        item = Stripe::InvoiceItem.retrieve("ii_123")
+        item = item.delete
+        assert_requested :delete,
+                         "#{Stripe.api_base}/v1/invoiceitems/#{item.id}"
+        assert item.is_a?(Stripe::InvoiceItem)
+      end
+    end
+
+    context ".delete" do
+      should "be deletable" do
+        item = Stripe::InvoiceItem.delete("ii_123")
+        assert_requested :delete,
+                         "#{Stripe.api_base}/v1/invoiceitems/ii_123"
+        assert item.is_a?(Stripe::InvoiceItem)
+      end
     end
   end
 end