stripe 4.9.0 → 5.1.0

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

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