stripe 4.9.0 → 5.1.0

data/test/stripe/stripe_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 StripeObjectTest < Test::Unit::TestCase
@@ -52,9 +52,9 @@ module Stripe
             2,
           ],
           map: {
-            :"0" => StripeObject.construct_from({ id: "index0" }, opts),
-            :"1" => "index1",
-            :"2" => 2,
+            "0": StripeObject.construct_from({ id: "index0" }, opts),
+            "1": "index1",
+            "2": 2,
           },
         }
 
@@ -235,20 +235,6 @@ module Stripe
       assert_equal true, obj.send(:metaclass).method_defined?(:foo)
     end
 
-    should "warn that #refresh_from is deprecated" do
-      old_stderr = $stderr
-      $stderr = StringIO.new
-      begin
-        obj = Stripe::StripeObject.construct_from({})
-        obj.refresh_from({}, {})
-        message = "NOTE: Stripe::StripeObject#refresh_from is " \
-                  "deprecated; use #update_attributes instead"
-        assert_match Regexp.new(message), $stderr.string
-      ensure
-        $stderr = old_stderr
-      end
-    end
-
     should "pass opts down to children when initializing" do
       opts = { custom: "opts" }
 
@@ -302,14 +288,14 @@ module Stripe
     end
 
     should "#serialize_params on an array that shortens" do
-      obj = Stripe::StripeObject.construct_from(foo: ["0-index", "1-index", "2-index"])
+      obj = Stripe::StripeObject.construct_from(foo: %w[0-index 1-index 2-index])
       obj.foo = ["new-value"]
       assert_equal({ foo: ["new-value"] },
                    obj.serialize_params)
     end
 
     should "#serialize_params on an array that lengthens" do
-      obj = Stripe::StripeObject.construct_from(foo: ["0-index", "1-index", "2-index"])
+      obj = Stripe::StripeObject.construct_from(foo: %w[0-index 1-index 2-index])
       obj.foo = ["new-value"] * 4
       assert_equal({ foo: ["new-value"] * 4 },
                    obj.serialize_params)
@@ -331,8 +317,8 @@ module Stripe
     end
 
     should "#serialize_params on an array that is unchanged" do
-      obj = Stripe::StripeObject.construct_from(foo: ["0-index", "1-index", "2-index"])
-      obj.foo = ["0-index", "1-index", "2-index"]
+      obj = Stripe::StripeObject.construct_from(foo: %w[0-index 1-index 2-index])
+      obj.foo = %w[0-index 1-index 2-index]
       assert_equal({}, obj.serialize_params)
     end
 
@@ -475,20 +461,6 @@ module Stripe
       assert_equal(expected, obj.to_s)
     end
 
-    should "warn that .serialize_params is deprecated" do
-      old_stderr = $stderr
-      $stderr = StringIO.new
-      begin
-        obj = Stripe::StripeObject.construct_from({})
-        Stripe::StripeObject.serialize_params(obj)
-        message = "NOTE: Stripe::StripeObject.serialize_params is " \
-                  "deprecated; use #serialize_params instead"
-        assert_match Regexp.new(message), $stderr.string
-      ensure
-        $stderr = old_stderr
-      end
-    end
-
     should "error on setting a property to an empty string" do
       obj = Stripe::StripeObject.construct_from(foo: "bar")
       e = assert_raises ArgumentError do

data/test/stripe/stripe_response_test.rb

@@ -1,49 +1,95 @@
 # frozen_string_literal: true
 
-require ::File.expand_path("../../test_helper", __FILE__)
+require ::File.expand_path("../test_helper", __dir__)
 
 module Stripe
   class StripeResponseTest < Test::Unit::TestCase
-    context ".from_faraday_hash" do
-      should "converts to StripeResponse" do
-        body = '{"foo": "bar"}'
+    context "Headers" do
+      should "allow case-insensitive header access" do
         headers = { "Request-Id" => "request-id" }
+        http_resp = create_net_http_resp(200, "", headers)
 
-        http_resp = {
-          body: body,
-          headers: headers,
-          status: 200,
-        }
+        headers = StripeResponse::Headers.from_net_http(http_resp)
 
-        resp = StripeResponse.from_faraday_hash(http_resp)
+        assert_equal "request-id", headers["request-id"]
+        assert_equal "request-id", headers["Request-Id"]
+        assert_equal "request-id", headers["Request-ID"]
+      end
 
-        assert_equal JSON.parse(body, symbolize_names: true), resp.data
-        assert_equal body, resp.http_body
-        assert_equal headers, resp.http_headers
-        assert_equal 200, resp.http_status
-        assert_equal "request-id", resp.request_id
+      should "initialize without error" do
+        StripeResponse::Headers.new({})
+        StripeResponse::Headers.new("Request-Id" => [])
+        StripeResponse::Headers.new("Request-Id" => ["request-id"])
+      end
+
+      should "initialize with error on a malformed hash" do
+        assert_raises(ArgumentError) do
+          StripeResponse::Headers.new(nil)
+        end
+
+        assert_raises(ArgumentError) do
+          StripeResponse::Headers.new(1 => [])
+        end
+
+        assert_raises(ArgumentError) do
+          StripeResponse::Headers.new("Request-Id" => 1)
+        end
+
+        assert_raises(ArgumentError) do
+          StripeResponse::Headers.new("Request-Id" => [1])
+        end
+      end
+
+      should "warn on duplicate header values" do
+        old_stderr = $stderr
+        $stderr = StringIO.new
+        begin
+          headers = StripeResponse::Headers.new("Duplicated" => %w[a b])
+          assert_equal "a", headers["Duplicated"]
+          assert_equal "Duplicate header values for `Duplicated`; returning only first",
+                       $stderr.string.rstrip
+        ensure
+          $stderr = old_stderr
+        end
       end
     end
 
-    context ".from_faraday_response" do
+    context ".from_net_http" do
       should "converts to StripeResponse" do
+        code = 200
         body = '{"foo": "bar"}'
         headers = { "Request-Id" => "request-id" }
+        http_resp = create_net_http_resp(code, body, headers)
 
-        env = Faraday::Env.from(
-          status: 200, body: body,
-          response_headers: headers
-        )
-        http_resp = Faraday::Response.new(env)
-
-        resp = StripeResponse.from_faraday_response(http_resp)
+        resp = StripeResponse.from_net_http(http_resp)
 
         assert_equal JSON.parse(body, symbolize_names: true), resp.data
         assert_equal body, resp.http_body
-        assert_equal headers, resp.http_headers
-        assert_equal 200, resp.http_status
+        assert_equal "request-id", resp.http_headers["Request-ID"]
+        assert_equal code, resp.http_status
         assert_equal "request-id", resp.request_id
       end
     end
+
+    # Synthesizes a `Net::HTTPResponse` object for testing purposes.
+    private def create_net_http_resp(code, body, headers)
+      # The "1.1" is HTTP version.
+      http_resp = Net::HTTPResponse.new("1.1", code.to_s, nil)
+      http_resp.body = body
+
+      # This is obviously super sketchy, but the Ruby team has done everything
+      # in their power to make these objects as difficult to test with as
+      # possible. Even if you specify a body, accessing `#body` the first time
+      # will attempt to read from a non-existent socket which will subsequently
+      # blow up. Setting this internal variable skips that read and allows the
+      # object to use the body that we specified above.
+      http_resp.instance_variable_set(:@read, true)
+
+      headers.each do |name, value|
+        http_resp[name] = value
+      end
+
+      http_resp
+    end
   end
 end

data/test/stripe/subscription_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 SubscriptionItemTest < Test::Unit::TestCase
@@ -43,11 +43,33 @@ module Stripe
       assert item.is_a?(Stripe::SubscriptionItem)
     end
 
-    should "be deletable" do
-      item = Stripe::SubscriptionItem.retrieve("si_123")
-      item = item.delete
-      assert_requested :delete, "#{Stripe.api_base}/v1/subscription_items/#{item.id}"
-      assert item.is_a?(Stripe::SubscriptionItem)
+    context "#delete" do
+      should "be deletable" do
+        item = Stripe::SubscriptionItem.retrieve("si_123")
+        item = item.delete
+        assert_requested :delete, "#{Stripe.api_base}/v1/subscription_items/#{item.id}"
+        assert item.is_a?(Stripe::SubscriptionItem)
+      end
+    end
+
+    context ".delete" do
+      should "be deletable" do
+        item = Stripe::SubscriptionItem.delete("si_123")
+        assert_requested :delete, "#{Stripe.api_base}/v1/subscription_items/si_123"
+        assert item.is_a?(Stripe::SubscriptionItem)
+      end
+    end
+
+    context "#create_usage_record" do
+      should "create a usage record" do
+        Stripe::SubscriptionItem.create_usage_record(
+          "si_123",
+          quantity: 5000,
+          timestamp: Time.now.to_i,
+          action: "increment"
+        )
+        assert_requested :post, "#{Stripe.api_base}/v1/subscription_items/si_123/usage_records"
+      end
     end
   end
 end

data/test/stripe/subscription_schedule_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 SubscriptionScheduleTest < Test::Unit::TestCase
@@ -51,6 +51,15 @@ module Stripe
       end
     end
 
+    context ".cancel" do
+      should "cancel a subscription schedule" do
+        schedule = Stripe::SubscriptionSchedule.cancel("sub_sched_123")
+        assert_requested :post,
+                         "#{Stripe.api_base}/v1/subscription_schedules/sub_sched_123/cancel"
+        assert schedule.is_a?(Stripe::SubscriptionSchedule)
+      end
+    end
+
     context "#release" do
       should "release a subscription schedule" do
         schedule = Stripe::SubscriptionSchedule.retrieve("sub_sched_123")
@@ -60,5 +69,14 @@ module Stripe
         assert schedule.is_a?(Stripe::SubscriptionSchedule)
       end
     end
+
+    context ".release" do
+      should "release a subscription schedule" do
+        schedule = Stripe::SubscriptionSchedule.release("sub_sched_123")
+        assert_requested :post,
+                         "#{Stripe.api_base}/v1/subscription_schedules/sub_sched_123/release"
+        assert schedule.is_a?(Stripe::SubscriptionSchedule)
+      end
+    end
   end
 end

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