stripe 4.9.0 → 5.28.0

data/test/stripe/errors_test.rb

@@ -1,19 +1,52 @@
 # 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 "#idempotent_replayed?" do
+        should "initialize from header" do
+          e = StripeError.new("message", http_headers: { "idempotent-replayed" => "true" })
+          assert_equal true, e.idempotent_replayed?
+        end
+
+        should "be 'falsey' by default" do
+          e = StripeError.new("message")
+          refute_equal true, e.idempotent_replayed?
+        end
+      end
 
-        e = StripeError.new("message", http_status: 200)
-        assert_equal "(Status 200) message", e.to_s
+      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: 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/instrumentation_test.rb

@@ -0,0 +1,74 @@
+# frozen_string_literal: true
+
+require_relative "../test_helper"
+
+module Stripe
+  class InstrumentationTest < Test::Unit::TestCase
+    context ".notify" do
+      teardown do
+        Stripe::Instrumentation.send(:subscribers).clear
+      end
+
+      should "notify subscribers for the right topic" do
+        sub1_events = []
+        Stripe::Instrumentation.subscribe(:test1, :sub1) { |event| sub1_events << event }
+        sub2_events = []
+        Stripe::Instrumentation.subscribe(:test2, :sub2) { |event| sub2_events << event }
+
+        Stripe::Instrumentation.notify(:test1, "hello")
+        assert_equal(1, sub1_events.size)
+        assert_equal(0, sub2_events.size)
+      end
+
+      should "notify multiple subscribers of the same topic" do
+        sub1_events = []
+        Stripe::Instrumentation.subscribe(:test, :sub1) { |event| sub1_events << event }
+        sub2_events = []
+        Stripe::Instrumentation.subscribe(:test, :sub2) { |event| sub2_events << event }
+
+        Stripe::Instrumentation.notify(:test, "hello")
+        assert_equal(1, sub1_events.size)
+        assert_equal(1, sub2_events.size)
+      end
+
+      should "not notify a subscriber once it has unsubscribed" do
+        events = []
+        Stripe::Instrumentation.subscribe(:test, :sub) { |event| events << event }
+
+        Stripe::Instrumentation.notify(:test, "hello")
+        assert_equal(1, events.size)
+
+        Stripe::Instrumentation.unsubscribe(:test, :sub)
+        Stripe::Instrumentation.notify(:test, "hello")
+        assert_equal(1, events.size)
+      end
+    end
+
+    context "RequestEventBegin" do
+      should "return a frozen object" do
+        event = Stripe::Instrumentation::RequestBeginEvent.new(
+          method: :get,
+          path: "/v1/test",
+          user_data: nil
+        )
+
+        assert(event.frozen?)
+      end
+    end
+
+    context "RequestEventEnd" do
+      should "return a frozen object" do
+        event = Stripe::Instrumentation::RequestEndEvent.new(
+          duration: 0.1,
+          http_status: 200,
+          method: :get,
+          num_retries: 0,
+          path: "/v1/test",
+          user_data: nil
+        )
+
+        assert(event.frozen?)
+      end
+    end
+  end
+end

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

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

data/test/stripe/invoice_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 InvoiceTest < Test::Unit::TestCase
@@ -38,11 +38,21 @@ module Stripe
       assert invoice.is_a?(Stripe::Invoice)
     end
 
-    should "be deletable" do
-      invoice = Stripe::Invoice.retrieve("in_123")
-      invoice = invoice.delete
-      assert_requested :delete, "#{Stripe.api_base}/v1/invoices/#{invoice.id}"
-      assert invoice.is_a?(Stripe::Invoice)
+    context "#delete" do
+      should "be deletable" do
+        invoice = Stripe::Invoice.retrieve("in_123")
+        invoice = invoice.delete
+        assert_requested :delete, "#{Stripe.api_base}/v1/invoices/#{invoice.id}"
+        assert invoice.is_a?(Stripe::Invoice)
+      end
+    end
+
+    context ".delete" do
+      should "be deletable" do
+        invoice = Stripe::Invoice.delete("in_123")
+        assert_requested :delete, "#{Stripe.api_base}/v1/invoices/in_123"
+        assert invoice.is_a?(Stripe::Invoice)
+      end
     end
 
     context "#finalize" do
@@ -55,6 +65,14 @@ module Stripe
       end
     end
 
+    context ".finalize" do
+      should "finalize invoice" do
+        invoice = Stripe::Invoice.finalize_invoice("in_123")
+        assert_requested :post, "#{Stripe.api_base}/v1/invoices/in_123/finalize"
+        assert invoice.is_a?(Stripe::Invoice)
+      end
+    end
+
     context "#mark_uncollectible" do
       should "mark invoice as uncollectible" do
         invoice = Stripe::Invoice.retrieve("in_123")
@@ -65,6 +83,14 @@ module Stripe
       end
     end
 
+    context ".mark_uncollectible" do
+      should "mark invoice as uncollectible" do
+        invoice = Stripe::Invoice.mark_uncollectible("in_123")
+        assert_requested :post, "#{Stripe.api_base}/v1/invoices/in_123/mark_uncollectible"
+        assert invoice.is_a?(Stripe::Invoice)
+      end
+    end
+
     context "#pay" do
       should "pay invoice" do
         invoice = Stripe::Invoice.retrieve("in_123")
@@ -88,7 +114,17 @@ module Stripe
       end
     end
 
-    context "#send" do
+    context ".pay" do
+      should "pay invoice" do
+        invoice = Stripe::Invoice.pay("in_123", source: "src_123")
+        assert_requested :post,
+                         "#{Stripe.api_base}/v1/invoices/in_123/pay",
+                         body: { source: "src_123" }
+        assert invoice.is_a?(Stripe::Invoice)
+      end
+    end
+
+    context "#send_invoice" do
       should "send invoice" do
         invoice = Stripe::Invoice.retrieve("in_123")
         invoice = invoice.send_invoice
@@ -98,7 +134,15 @@ module Stripe
       end
     end
 
-    context "#upcoming" do
+    context ".send_invoice" do
+      should "send invoice" do
+        invoice = Stripe::Invoice.send_invoice("in_123")
+        assert_requested :post, "#{Stripe.api_base}/v1/invoices/in_123/send"
+        assert invoice.is_a?(Stripe::Invoice)
+      end
+    end
+
+    context ".upcoming" do
       should "retrieve upcoming invoices" do
         invoice = Stripe::Invoice.upcoming(
           customer: "cus_123",
@@ -148,7 +192,23 @@ module Stripe
       end
     end
 
-    context "#void" do
+    context ".list_upcoming_line_items" do
+      should "retrieve upcoming invoices" do
+        line_items = Stripe::Invoice.list_upcoming_line_items(
+          customer: "cus_123",
+          subscription: "sub_123"
+        )
+        assert_requested :get, "#{Stripe.api_base}/v1/invoices/upcoming/lines",
+                         query: {
+                           customer: "cus_123",
+                           subscription: "sub_123",
+                         }
+        assert line_items.data.is_a?(Array)
+        assert line_items.data[0].is_a?(Stripe::InvoiceLineItem)
+      end
+    end
+
+    context "#void_invoice" do
       should "void invoice" do
         invoice = Stripe::Invoice.retrieve("in_123")
         invoice = invoice.void_invoice
@@ -157,5 +217,13 @@ module Stripe
         assert invoice.is_a?(Stripe::Invoice)
       end
     end
+
+    context ".void_invoice" do
+      should "void invoice" do
+        invoice = Stripe::Invoice.void_invoice("in_123")
+        assert_requested :post, "#{Stripe.api_base}/v1/invoices/in_123/void"
+        assert invoice.is_a?(Stripe::Invoice)
+      end
+    end
   end
 end

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