stripe 4.9.0 → 5.28.0

data/test/stripe/subscription_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 SubscriptionTest < Test::Unit::TestCase
@@ -41,19 +41,39 @@ module Stripe
       assert subscription.is_a?(Stripe::Subscription)
     end
 
-    should "be deletable" do
-      subscription = Stripe::Subscription.retrieve("sub_123")
-      subscription = subscription.delete
-      assert_requested :delete,
-                       "#{Stripe.api_base}/v1/subscriptions/#{subscription.id}"
-      assert subscription.is_a?(Stripe::Subscription)
+    context "#delete" do
+      should "be deletable" do
+        subscription = Stripe::Subscription.retrieve("sub_123")
+        subscription = subscription.delete
+        assert_requested :delete,
+                         "#{Stripe.api_base}/v1/subscriptions/#{subscription.id}"
+        assert subscription.is_a?(Stripe::Subscription)
+      end
+    end
+
+    context ".delete" do
+      should "be deletable" do
+        subscription = Stripe::Subscription.delete("sub_123")
+        assert_requested :delete,
+                         "#{Stripe.api_base}/v1/subscriptions/sub_123"
+        assert subscription.is_a?(Stripe::Subscription)
+      end
     end
 
     context "#delete_discount" do
       should "be able to delete a subscriptions's discount" do
         subscription = Stripe::Subscription.retrieve("sub_123")
-        subscription = subscription.delete_discount
-        assert subscription.is_a?(Stripe::Subscription)
+        discount = subscription.delete_discount
+        assert_requested :delete, "#{Stripe.api_base}/v1/subscriptions/sub_123/discount"
+        assert discount.is_a?(Stripe::Discount)
+      end
+    end
+
+    context ".delete_discount" do
+      should "be able to delete a subscriptions's discount" do
+        discount = Stripe::Subscription.delete_discount("sub_123")
+        assert_requested :delete, "#{Stripe.api_base}/v1/subscriptions/sub_123/discount"
+        assert discount.is_a?(Stripe::Discount)
       end
     end
   end

data/test/stripe/tax_id_test.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+require ::File.expand_path("../test_helper", __dir__)
+
+module Stripe
+  class TaxIdTest < Test::Unit::TestCase
+    context "#resource_url" do
+      should "return a resource URL" do
+        tax_id = Stripe::TaxId.construct_from(
+          id: "txi_123",
+          customer: "cus_123"
+        )
+        assert_equal "/v1/customers/cus_123/tax_ids/txi_123",
+                     tax_id.resource_url
+      end
+
+      should "raise without a customer" do
+        tax_id = Stripe::TaxId.construct_from(id: "txi_123")
+        assert_raises NotImplementedError do
+          tax_id.resource_url
+        end
+      end
+    end
+
+    should "raise on #retrieve" do
+      assert_raises NotImplementedError do
+        Stripe::TaxId.retrieve("txi_123")
+      end
+    end
+  end
+end

data/test/stripe/tax_rate_test.rb

@@ -0,0 +1,43 @@
+# frozen_string_literal: true
+
+require ::File.expand_path("../test_helper", __dir__)
+
+module Stripe
+  class TaxRateTest < Test::Unit::TestCase
+    should "be listable" do
+      tax_rates = Stripe::TaxRate.list
+      assert_requested :get, "#{Stripe.api_base}/v1/tax_rates"
+      assert tax_rates.data.is_a?(Array)
+      assert tax_rates.first.is_a?(Stripe::TaxRate)
+    end
+
+    should "be retrievable" do
+      tax_rate = Stripe::TaxRate.retrieve("txr_123")
+      assert_requested :get, "#{Stripe.api_base}/v1/tax_rates/txr_123"
+      assert tax_rate.is_a?(Stripe::TaxRate)
+    end
+
+    should "be creatable" do
+      tax_rate = Stripe::TaxRate.create(
+        display_name: "name",
+        inclusive: false,
+        percentage: 10.15
+      )
+      assert_requested :post, "#{Stripe.api_base}/v1/tax_rates"
+      assert tax_rate.is_a?(Stripe::TaxRate)
+    end
+
+    should "be saveable" do
+      tax_rate = Stripe::TaxRate.retrieve("txr_123")
+      tax_rate.metadata["key"] = "value"
+      tax_rate.save
+      assert_requested :post, "#{Stripe.api_base}/v1/tax_rates/#{tax_rate.id}"
+    end
+
+    should "be updateable" do
+      tax_rate = Stripe::TaxRate.update("txr_123", metadata: { key: "value" })
+      assert_requested :post, "#{Stripe.api_base}/v1/tax_rates/txr_123"
+      assert tax_rate.is_a?(Stripe::TaxRate)
+    end
+  end
+end

data/test/stripe/terminal/connection_token_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 Terminal

data/test/stripe/terminal/location_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 Terminal
@@ -46,6 +46,23 @@ module Stripe
         assert_requested :post, "#{Stripe.api_base}/v1/terminal/locations/loc_123"
         assert location.is_a?(Stripe::Terminal::Location)
       end
+
+      context "#delete" do
+        should "be deletable" do
+          location = Stripe::Terminal::Location.retrieve("loc_123")
+          location = location.delete
+          assert_requested :delete, "#{Stripe.api_base}/v1/terminal/locations/#{location.id}"
+          assert location.is_a?(Stripe::Terminal::Location)
+        end
+      end
+
+      context ".delete" do
+        should "be deletable" do
+          location = Stripe::Terminal::Location.delete("loc_123")
+          assert_requested :delete, "#{Stripe.api_base}/v1/terminal/locations/loc_123"
+          assert location.is_a?(Stripe::Terminal::Location)
+        end
+      end
     end
   end
 end

data/test/stripe/terminal/reader_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 Terminal
@@ -40,6 +40,23 @@ module Stripe
         assert_requested :post, "#{Stripe.api_base}/v1/terminal/readers/rdr_123"
         assert reader.is_a?(Stripe::Terminal::Reader)
       end
+
+      context "#delete" do
+        should "be deletable" do
+          reader = Stripe::Terminal::Reader.retrieve("rdr_123")
+          reader = reader.delete
+          assert_requested :delete, "#{Stripe.api_base}/v1/terminal/readers/#{reader.id}"
+          assert reader.is_a?(Stripe::Terminal::Reader)
+        end
+      end
+
+      context ".delete" do
+        should "be deletable" do
+          reader = Stripe::Terminal::Reader.delete("rdr_123")
+          assert_requested :delete, "#{Stripe.api_base}/v1/terminal/readers/rdr_123"
+          assert reader.is_a?(Stripe::Terminal::Reader)
+        end
+      end
     end
   end
 end

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

data/test/stripe/topup_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 TopupTest < Test::Unit::TestCase
@@ -50,5 +50,13 @@ module Stripe
         assert topup.is_a?(Stripe::Topup)
       end
     end
+
+    context ".cancel" do
+      should "cancel the topup" do
+        topup = Stripe::Topup.cancel("tu_123")
+        assert_requested :post, "#{Stripe.api_base}/v1/topups/tu_123/cancel"
+        assert topup.is_a?(Stripe::Topup)
+      end
+    end
   end
 end

data/test/stripe/transfer_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 TransferTest < Test::Unit::TestCase
@@ -39,5 +39,50 @@ module Stripe
       assert_requested :post, "#{Stripe.api_base}/v1/transfers/tr_123"
       assert transfer.is_a?(Stripe::Transfer)
     end
+
+    context "#create_reversal" do
+      should "create a reversal" do
+        reversal = Stripe::Transfer.create_reversal(
+          "tr_123",
+          amount: 100
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/transfers/tr_123/reversals"
+        assert reversal.is_a?(Stripe::Reversal)
+      end
+    end
+
+    context "#retrieve_reversal" do
+      should "retrieve a reversal" do
+        reversal = Stripe::Transfer.retrieve_reversal(
+          "tr_123",
+          "trr_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/transfers/tr_123/reversals/trr_123"
+        assert reversal.is_a?(Stripe::Reversal)
+      end
+    end
+
+    context "#update_reversal" do
+      should "update a reversal" do
+        reversal = Stripe::Transfer.update_reversal(
+          "tr_123",
+          "trr_123",
+          metadata: { foo: "bar" }
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/transfers/tr_123/reversals/trr_123"
+        assert reversal.is_a?(Stripe::Reversal)
+      end
+    end
+
+    context "#list_reversals" do
+      should "list the transfer's reversals" do
+        reversals = Stripe::Transfer.list_reversals(
+          "tr_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/transfers/tr_123/reversals"
+        assert reversals.is_a?(Stripe::ListObject)
+        assert reversals.data.is_a?(Array)
+      end
+    end
   end
 end

data/test/stripe/usage_record_summary_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 UsageRecordSummaryTest < Test::Unit::TestCase
@@ -9,11 +9,21 @@ module Stripe
     end
 
     should "be listable" do
-      transactions = @sub_item.usage_record_summaries
+      old_stderr = $stderr
+      $stderr = StringIO.new
 
-      assert_requested :get, "#{Stripe.api_base}/v1/subscription_items/#{@sub_item.id}/usage_record_summaries"
-      assert transactions.data.is_a?(Array)
-      assert transactions.first.is_a?(Stripe::UsageRecordSummary)
+      begin
+        transactions = @sub_item.usage_record_summaries
+
+        assert_requested :get, "#{Stripe.api_base}/v1/subscription_items/#{@sub_item.id}/usage_record_summaries"
+        assert transactions.data.is_a?(Array)
+        assert transactions.first.is_a?(Stripe::UsageRecordSummary)
+
+        assert_include $stderr.string,
+                       "use SubscriptionItem.list_usage_record_summaries instead"
+      ensure
+        $stderr = old_stderr
+      end
     end
   end
 end

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

data/test/stripe/webhook_endpoint_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 WebhookEndpointTest < Test::Unit::TestCase
@@ -38,5 +38,22 @@ module Stripe
       assert_requested :post, "#{Stripe.api_base}/v1/webhook_endpoints/we_123"
       assert webhook_endpoint.is_a?(Stripe::WebhookEndpoint)
     end
+
+    context "#delete" do
+      should "be deletable" do
+        webhook_endpoint = Stripe::WebhookEndpoint.retrieve("we_123")
+        webhook_endpoint = webhook_endpoint.delete
+        assert_requested :delete, "#{Stripe.api_base}/v1/webhook_endpoints/#{webhook_endpoint.id}"
+        assert webhook_endpoint.is_a?(Stripe::WebhookEndpoint)
+      end
+    end
+
+    context ".delete" do
+      should "be deletable" do
+        webhook_endpoint = Stripe::WebhookEndpoint.delete("we_123")
+        assert_requested :delete, "#{Stripe.api_base}/v1/webhook_endpoints/we_123"
+        assert webhook_endpoint.is_a?(Stripe::WebhookEndpoint)
+      end
+    end
   end
 end

data/test/stripe/webhook_test.rb

@@ -1,24 +1,63 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../test_helper", __FILE__)
+require ::File.expand_path("../test_helper", __dir__)
 
 module Stripe
   class WebhookTest < Test::Unit::TestCase
-    EVENT_PAYLOAD = <<-PAYLOAD.freeze
+    EVENT_PAYLOAD = <<~PAYLOAD
       {
         "id": "evt_test_webhook",
         "object": "event"
       }
     PAYLOAD
-    SECRET = "whsec_test_secret".freeze
+    SECRET = "whsec_test_secret"
 
     def generate_header(opts = {})
-      opts[:timestamp] ||= Time.now.to_i
+      opts[:timestamp] ||= Time.now
       opts[:payload] ||= EVENT_PAYLOAD
       opts[:secret] ||= SECRET
       opts[:scheme] ||= Stripe::Webhook::Signature::EXPECTED_SCHEME
-      opts[:signature] ||= Stripe::Webhook::Signature.send(:compute_signature, "#{opts[:timestamp]}.#{opts[:payload]}", opts[:secret])
-      "t=#{opts[:timestamp]},#{opts[:scheme]}=#{opts[:signature]}"
+      opts[:signature] ||= Stripe::Webhook::Signature.compute_signature(
+        opts[:timestamp],
+        opts[:payload],
+        opts[:secret]
+      )
+      Stripe::Webhook::Signature.generate_header(
+        opts[:timestamp],
+        opts[:signature],
+        scheme: opts[:scheme]
+      )
+    end
+
+    context ".compute_signature" do
+      should "compute a signature which can then be verified" do
+        timestamp = Time.now
+        signature = Stripe::Webhook::Signature.compute_signature(
+          timestamp,
+          EVENT_PAYLOAD,
+          SECRET
+        )
+        header = generate_header(timestamp: timestamp, signature: signature)
+        assert(Stripe::Webhook::Signature.verify_header(EVENT_PAYLOAD, header, SECRET))
+      end
+    end
+
+    context ".generate_header" do
+      should "generate a header in valid format" do
+        timestamp = Time.now
+        signature = Stripe::Webhook::Signature.compute_signature(
+          timestamp,
+          EVENT_PAYLOAD,
+          SECRET
+        )
+        scheme = "v1"
+        header = Stripe::Webhook::Signature.generate_header(
+          timestamp,
+          signature,
+          scheme: scheme
+        )
+        assert_equal("t=#{timestamp.to_i},#{scheme}=#{signature}", header)
+      end
     end
 
     context ".construct_event" do
@@ -46,7 +85,7 @@ module Stripe
 
     context ".verify_signature_header" do
       should "raise a SignatureVerificationError when the header does not have the expected format" do
-        header = 'i\'m not even a real signature header'
+        header = "i'm not even a real signature header"
         e = assert_raises(Stripe::SignatureVerificationError) do
           Stripe::Webhook::Signature.verify_header(EVENT_PAYLOAD, header, "secret")
         end
@@ -70,7 +109,7 @@ module Stripe
       end
 
       should "raise a SignatureVerificationError when the timestamp is not within the tolerance" do
-        header = generate_header(timestamp: Time.now.to_i - 15)
+        header = generate_header(timestamp: Time.now - 15)
         e = assert_raises(Stripe::SignatureVerificationError) do
           Stripe::Webhook::Signature.verify_header(EVENT_PAYLOAD, header, SECRET, tolerance: 10)
         end
@@ -88,7 +127,7 @@ module Stripe
       end
 
       should "return true when the header contains a valid signature and the timestamp is off but no tolerance is provided" do
-        header = generate_header(timestamp: 12_345)
+        header = generate_header(timestamp: Time.at(12_345))
         assert(Stripe::Webhook::Signature.verify_header(EVENT_PAYLOAD, header, SECRET))
       end
     end