stripe 3.26.1 → 3.27.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4ac82de8d5c93956f56ab752a21fbd5470aa3f2452c7360c8bad21b62437f1de
-  data.tar.gz: 4c03b8ed0e4d03a24a4b18e6668147b7c88e4ae1977cbb5d7d6319ff4c0535f0
+  metadata.gz: c1bbd688d73104dce2fc75c1363172aed7f44f6f6e3d40e634588adc1efd9cb3
+  data.tar.gz: b05dd5cec9bdaabdd57f305fd1cd08058158d71cea7f2377e98949f12d856c57
 SHA512:
-  metadata.gz: 3094e2f430018ebba15fe8f78c6b9e28480c8f47ec5aeb3185296a5c7f27357673b3436f1631c76bdc5b09535bdd6885794de7a5b24c6cfe4d6d2fb16e1c68b5
-  data.tar.gz: ef354896a762a5cb3fd34028f9f04413bf1b576b44d040167b2e51fd6724164cb72821a2f4be02679c16ea4663738409b6a83ffd8426a4de2191832f397ed250
+  metadata.gz: ccb476039053498ddd0b3e67971acdf6b35d81fc7c86b5871523d742eb8534d72f0d1094d93d5ae778125528c33fbfba6552ce3edde0e5b88114aba1b5f3c6c1
+  data.tar.gz: a347c29e5cf52a0df6d12232cb3cb82a371b536858ce254e296dd1292e3d1422606eb87577307480a6abd5baa706cafb8d444c56c45eb6c7f156d9f048ed9af3

data/.travis.yml

@@ -18,7 +18,7 @@ sudo: false
 env:
   global:
     # If changing this number, please also change it in `test/test_helper.rb`.
-    - STRIPE_MOCK_VERSION=0.30.0
+    - STRIPE_MOCK_VERSION=0.32.0
 
 cache:
   directories:

data/CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## 3.27.0 - 2018-09-24
+* [#689](https://github.com/stripe/stripe-ruby/pull/689) Rename `FileUpload` to `File`
+
 ## 3.26.1 - 2018-09-14
 * [#688](https://github.com/stripe/stripe-ruby/pull/688) Fix hash equality on `StripeObject`
 

data/VERSION

@@ -1 +1 @@
-3.26.1
+3.27.0

data/lib/stripe.rb

@@ -55,8 +55,8 @@ require "stripe/dispute"
 require "stripe/ephemeral_key"
 require "stripe/event"
 require "stripe/exchange_rate"
+require "stripe/file"
 require "stripe/file_link"
-require "stripe/file_upload"
 require "stripe/invoice"
 require "stripe/invoice_item"
 require "stripe/invoice_line_item"
@@ -103,7 +103,7 @@ module Stripe
 
   @api_base = "https://api.stripe.com"
   @connect_base = "https://connect.stripe.com"
-  @uploads_base = "https://uploads.stripe.com"
+  @uploads_base = "https://files.stripe.com"
 
   @log_level = nil
   @logger = nil

data/lib/stripe/{file_upload.rb → file.rb}

@@ -1,23 +1,21 @@
 # frozen_string_literal: true
 
 module Stripe
-  class FileUpload < APIResource
+  class File < APIResource
     extend Stripe::APIOperations::Create
     extend Stripe::APIOperations::List
 
-    OBJECT_NAME = "file_upload".freeze
+    # This resource can have two different object names. In latter API
+    # versions, only `file` is used, but since stripe-ruby may be used with
+    # any API version, we need to support deserializing the older
+    # `file_upload` object into the same class.
+    OBJECT_NAME = "file".freeze
+    OBJECT_NAME_ALT = "file_upload".freeze
 
     def self.resource_url
       "/v1/files"
     end
 
-    def self.request(method, url, params = {}, opts = {})
-      opts = {
-        api_base: Stripe.uploads_base,
-      }.merge(Util.normalize_opts(opts))
-      super
-    end
-
     def self.create(params = {}, opts = {})
       # rest-client would accept a vanilla `File` for upload, but Faraday does
       # not. Support the old API by wrapping a `File`-like object with an
@@ -27,9 +25,13 @@ module Stripe
       end
 
       opts = {
+        api_base: Stripe.uploads_base,
         content_type: "multipart/form-data",
       }.merge(Util.normalize_opts(opts))
       super
     end
   end
+
+  # For backwards compatibility, the `File` class is aliased to `FileUpload`.
+  FileUpload = File
 end

data/lib/stripe/stripe_object.rb

@@ -96,13 +96,14 @@ module Stripe
       other.is_a?(StripeObject) && @values == other.instance_variable_get(:@values)
     end
 
-    # Hash equality. As with ==, we consider two equivalent Stripe objects equal.
+    # Hash equality. As with `#==`, we consider two equivalent Stripe objects equal.
     def eql?(other)
+      # Defer to the implementation on `#==`.
       self == other
     end
 
-    # As for equality, we hash to Stripe objects to the same value if they're
-    # equivalent objects.
+    # As with equality in `#==` and `#eql?`, we hash two Stripe objects to the
+    # same value if they're equivalent objects.
     def hash
       @values.hash
     end

data/lib/stripe/util.rb

@@ -64,8 +64,9 @@ module Stripe
         EphemeralKey::OBJECT_NAME             => EphemeralKey,
         Event::OBJECT_NAME                    => Event,
         ExchangeRate::OBJECT_NAME             => ExchangeRate,
+        File::OBJECT_NAME                     => File,
+        File::OBJECT_NAME_ALT                 => File,
         FileLink::OBJECT_NAME                 => FileLink,
-        FileUpload::OBJECT_NAME               => FileUpload,
         Invoice::OBJECT_NAME                  => Invoice,
         InvoiceItem::OBJECT_NAME              => InvoiceItem,
         InvoiceLineItem::OBJECT_NAME          => InvoiceLineItem,

data/lib/stripe/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Stripe
-  VERSION = "3.26.1".freeze
+  VERSION = "3.27.0".freeze
 end

data/test/stripe/file_test.rb

@@ -0,0 +1,73 @@
+# frozen_string_literal: true
+
+require ::File.expand_path("../../test_helper", __FILE__)
+
+module Stripe
+  class FileTest < Test::Unit::TestCase
+    should "be listable" do
+      files = Stripe::File.list
+      assert_requested :get, "#{Stripe.api_base}/v1/files"
+      assert files.data.is_a?(Array)
+      assert files.data[0].is_a?(Stripe::File)
+    end
+
+    should "be retrievable" do
+      file = Stripe::File.retrieve("file_123")
+      assert_requested :get, "#{Stripe.api_base}/v1/files/file_123"
+      assert file.is_a?(Stripe::File)
+    end
+
+    context ".create" do
+      setup do
+        # We don't point to the same host for the API and uploads in
+        # production, but `stripe-mock` supports both APIs.
+        Stripe.uploads_base = Stripe.api_base
+
+        # Set `api_base` to `nil` to ensure that these requests are _not_ sent
+        # to the default API hostname.
+        Stripe.api_base = nil
+      end
+
+      should "be creatable with a File" do
+        file = Stripe::File.create(
+          purpose: "dispute_evidence",
+          file: ::File.new(__FILE__)
+        )
+        assert_requested :post, "#{Stripe.uploads_base}/v1/files"
+        assert file.is_a?(Stripe::File)
+      end
+
+      should "be creatable with a Tempfile" do
+        tempfile = Tempfile.new("foo")
+        tempfile.write("Hello world")
+        tempfile.rewind
+
+        file = Stripe::File.create(
+          purpose: "dispute_evidence",
+          file: tempfile
+        )
+        assert_requested :post, "#{Stripe.uploads_base}/v1/files"
+        assert file.is_a?(Stripe::File)
+      end
+
+      should "be creatable with Faraday::UploadIO" do
+        file = Stripe::File.create(
+          purpose: "dispute_evidence",
+          file: Faraday::UploadIO.new(::File.new(__FILE__), nil)
+        )
+        assert_requested :post, "#{Stripe.uploads_base}/v1/files"
+        assert file.is_a?(Stripe::File)
+      end
+    end
+
+    should "be deserializable when `object=file`" do
+      file = Stripe::Util.convert_to_stripe_object({ object: "file" }, {})
+      assert file.is_a?(Stripe::File)
+    end
+
+    should "be deserializable when `object=file_upload`" do
+      file = Stripe::Util.convert_to_stripe_object({ object: "file_upload" }, {})
+      assert file.is_a?(Stripe::File)
+    end
+  end
+end

data/test/stripe/file_upload_test.rb

@@ -3,43 +3,73 @@
 require ::File.expand_path("../../test_helper", __FILE__)
 
 module Stripe
+  # This is a strict copy of `FileTest`, except that it uses
+  # `Stripe::FileUpload` instead of `Stripe::File`.
   class FileUploadTest < Test::Unit::TestCase
     should "be listable" do
       files = Stripe::FileUpload.list
+      assert_requested :get, "#{Stripe.api_base}/v1/files"
       assert files.data.is_a?(Array)
       assert files.data[0].is_a?(Stripe::FileUpload)
     end
 
     should "be retrievable" do
       file = Stripe::FileUpload.retrieve("file_123")
+      assert_requested :get, "#{Stripe.api_base}/v1/files/file_123"
       assert file.is_a?(Stripe::FileUpload)
     end
 
-    should "be creatable with a File" do
-      file = Stripe::FileUpload.create(
-        purpose: "dispute_evidence",
-        file: ::File.new(__FILE__)
-      )
-      assert file.is_a?(Stripe::FileUpload)
-    end
+    context ".create" do
+      setup do
+        # We don't point to the same host for the API and uploads in
+        # production, but `stripe-mock` supports both APIs.
+        Stripe.uploads_base = Stripe.api_base
+
+        # Set `api_base` to `nil` to ensure that these requests are _not_ sent
+        # to the default API hostname. `api_base` is reset when each test
+        # starts so this won't affect the global state.
+        Stripe.api_base = nil
+      end
+
+      should "be creatable with a File" do
+        file = Stripe::FileUpload.create(
+          purpose: "dispute_evidence",
+          file: ::File.new(__FILE__)
+        )
+        assert_requested :post, "#{Stripe.uploads_base}/v1/files"
+        assert file.is_a?(Stripe::FileUpload)
+      end
 
-    should "be creatable with a Tempfile" do
-      tempfile = Tempfile.new("foo")
-      tempfile.write("Hello world")
-      tempfile.rewind
+      should "be creatable with a Tempfile" do
+        tempfile = Tempfile.new("foo")
+        tempfile.write("Hello world")
+        tempfile.rewind
+
+        file = Stripe::FileUpload.create(
+          purpose: "dispute_evidence",
+          file: tempfile
+        )
+        assert_requested :post, "#{Stripe.uploads_base}/v1/files"
+        assert file.is_a?(Stripe::FileUpload)
+      end
+
+      should "be creatable with Faraday::UploadIO" do
+        file = Stripe::FileUpload.create(
+          purpose: "dispute_evidence",
+          file: Faraday::UploadIO.new(::File.new(__FILE__), nil)
+        )
+        assert_requested :post, "#{Stripe.uploads_base}/v1/files"
+        assert file.is_a?(Stripe::FileUpload)
+      end
+    end
 
-      file = Stripe::FileUpload.create(
-        purpose: "dispute_evidence",
-        file: tempfile
-      )
+    should "be deserializable when `object=file`" do
+      file = Stripe::Util.convert_to_stripe_object({ object: "file" }, {})
       assert file.is_a?(Stripe::FileUpload)
     end
 
-    should "be creatable with Faraday::UploadIO" do
-      file = Stripe::FileUpload.create(
-        purpose: "dispute_evidence",
-        file: Faraday::UploadIO.new(::File.new(__FILE__), nil)
-      )
+    should "be deserializable when `object=file_upload`" do
+      file = Stripe::Util.convert_to_stripe_object({ object: "file_upload" }, {})
       assert file.is_a?(Stripe::FileUpload)
     end
   end

data/test/stripe/issuing/dispute_test.rb

@@ -8,7 +8,7 @@ module Stripe
       should "be creatable" do
         dispute = Stripe::Issuing::Dispute.create(
           reason: "fraudulent",
-          transaction: "ipi_123"
+          disputed_transaction: "ipi_123"
         )
         assert_requested :post, "#{Stripe.api_base}/v1/issuing/disputes"
         assert dispute.is_a?(Stripe::Issuing::Dispute)

data/test/stripe/stripe_object_test.rb

@@ -118,7 +118,7 @@ module Stripe
     end
 
     context "#eql?" do
-      should "produce the true for two equivalent Stripe objects" do
+      should "produce true for two equivalent Stripe objects" do
         obj1 = Stripe::StripeObject.construct_from(id: 1, name: "Stripe")
         obj2 = Stripe::StripeObject.construct_from(id: 1, name: "Stripe")
         assert obj1.eql?(obj2)

data/test/test_helper.rb

@@ -16,7 +16,7 @@ PROJECT_ROOT = ::File.expand_path("../../", __FILE__)
 require ::File.expand_path("../test_data", __FILE__)
 
 # If changing this number, please also change it in `.travis.yml`.
-MOCK_MINIMUM_VERSION = "0.30.0".freeze
+MOCK_MINIMUM_VERSION = "0.32.0".freeze
 MOCK_PORT = ENV["STRIPE_MOCK_PORT"] || 12_111
 
 # Disable all real network connections except those that are outgoing to
@@ -50,10 +50,6 @@ module Test
         Stripe.api_key = "sk_test_123"
         Stripe.api_base = "http://localhost:#{MOCK_PORT}"
 
-        # We don't point to the same host for the API and uploads in
-        # production, but `stripe-mock` supports both APIs.
-        Stripe.uploads_base = Stripe.api_base
-
         stub_connect
       end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: stripe
 version: !ruby/object:Gem::Version
-  version: 3.26.1
+  version: 3.27.0
 platform: ruby
 authors:
 - Stripe
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-09-14 00:00:00.000000000 Z
+date: 2018-09-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -76,8 +76,8 @@ files:
 - lib/stripe/errors.rb
 - lib/stripe/event.rb
 - lib/stripe/exchange_rate.rb
+- lib/stripe/file.rb
 - lib/stripe/file_link.rb
-- lib/stripe/file_upload.rb
 - lib/stripe/invoice.rb
 - lib/stripe/invoice_item.rb
 - lib/stripe/invoice_line_item.rb
@@ -147,6 +147,7 @@ files:
 - test/stripe/errors_test.rb
 - test/stripe/exchange_rate_test.rb
 - test/stripe/file_link_test.rb
+- test/stripe/file_test.rb
 - test/stripe/file_upload_test.rb
 - test/stripe/invoice_item_test.rb
 - test/stripe/invoice_line_item_test.rb
@@ -240,6 +241,7 @@ test_files:
 - test/stripe/errors_test.rb
 - test/stripe/exchange_rate_test.rb
 - test/stripe/file_link_test.rb
+- test/stripe/file_test.rb
 - test/stripe/file_upload_test.rb
 - test/stripe/invoice_item_test.rb
 - test/stripe/invoice_line_item_test.rb