stripe 1.58.0 → 2.0.0

data/test/test_helper.rb

@@ -1,45 +1,41 @@
+require 'committee'
+require 'sinatra'
 require 'stripe'
 require 'test/unit'
 require 'mocha/setup'
 require 'stringio'
 require 'shoulda/context'
-require File.expand_path('../test_data', __FILE__)
-
-# monkeypatch request methods
-module Stripe
-  @mock_rest_client = nil
-
-  def self.mock_rest_client=(mock_client)
-    @mock_rest_client = mock_client
-  end
+require 'webmock/test_unit'
 
-  class << self
-    remove_method :execute_request
-  end
+PROJECT_ROOT = File.expand_path("../../", __FILE__)
 
-  def self.execute_request(opts)
-    get_params = (opts[:headers] || {})[:params]
-    post_params = opts[:payload]
-    case opts[:method]
-    when :get then @mock_rest_client.get opts[:url], get_params, post_params
-    when :post then @mock_rest_client.post opts[:url], get_params, post_params
-    when :delete then @mock_rest_client.delete opts[:url], get_params, post_params
-    end
-  end
-end
+require File.expand_path('../api_fixtures', __FILE__)
+require File.expand_path('../api_stub_helpers', __FILE__)
+require File.expand_path('../test_data', __FILE__)
 
 class Test::Unit::TestCase
+  include APIStubHelpers
   include Stripe::TestData
   include Mocha
 
+  # Fixtures are available in tests using something like:
+  #
+  #   API_FIXTURES[:charge][:id]
+  #
+  API_FIXTURES = APIFixtures.new
+
   setup do
-    @mock = mock
-    Stripe.mock_rest_client = @mock
     Stripe.api_key = "foo"
+
+    # Stub the Stripe API with a default stub. Note that this method can be
+    # called again in test bodies in order to override responses on particular
+    # endpoints.
+    stub_api
+
+    stub_connect
   end
 
   teardown do
-    Stripe.mock_rest_client = nil
     Stripe.api_key = nil
   end
 end

metadata

@@ -1,35 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: stripe
 version: !ruby/object:Gem::Version
-  version: 1.58.0
+  version: 2.0.0
 platform: ruby
 authors:
 - Stripe
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-01-19 00:00:00.000000000 Z
+date: 2017-02-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: rest-client
+  name: faraday
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.4'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '1.4'
-    - - "<"
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '0'
 description: Stripe is the easiest way to accept payments online.  See https://stripe.com
   for details.
 email: support@stripe.com
@@ -38,6 +32,7 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
+- ".gitattributes"
 - ".github/ISSUE_TEMPLATE.md"
 - ".gitignore"
 - ".travis.yml"
@@ -89,7 +84,9 @@ files:
 - lib/stripe/singleton_api_resource.rb
 - lib/stripe/sku.rb
 - lib/stripe/source.rb
+- lib/stripe/stripe_client.rb
 - lib/stripe/stripe_object.rb
+- lib/stripe/stripe_response.rb
 - lib/stripe/subscription.rb
 - lib/stripe/subscription_item.rb
 - lib/stripe/three_d_secure.rb
@@ -97,7 +94,13 @@ files:
 - lib/stripe/transfer.rb
 - lib/stripe/util.rb
 - lib/stripe/version.rb
+- spec/fixtures.json
+- spec/fixtures.yaml
+- spec/spec.json
+- spec/spec.yaml
 - stripe.gemspec
+- test/api_fixtures.rb
+- test/api_stub_helpers.rb
 - test/stripe/account_test.rb
 - test/stripe/alipay_account_test.rb
 - test/stripe/api_operations_test.rb
@@ -109,7 +112,6 @@ files:
 - test/stripe/bank_account_test.rb
 - test/stripe/bitcoin_receiver_test.rb
 - test/stripe/bitcoin_transaction_test.rb
-- test/stripe/charge_refund_test.rb
 - test/stripe/charge_test.rb
 - test/stripe/country_spec_test.rb
 - test/stripe/coupon_test.rb
@@ -121,7 +123,6 @@ files:
 - test/stripe/invoice_item_test.rb
 - test/stripe/invoice_test.rb
 - test/stripe/list_object_test.rb
-- test/stripe/metadata_test.rb
 - test/stripe/order_return_test.rb
 - test/stripe/order_test.rb
 - test/stripe/plan_test.rb
@@ -132,7 +133,9 @@ files:
 - test/stripe/reversal_test.rb
 - test/stripe/sku_test.rb
 - test/stripe/source_test.rb
+- test/stripe/stripe_client_test.rb
 - test/stripe/stripe_object_test.rb
+- test/stripe/stripe_response_test.rb
 - test/stripe/subscription_item_test.rb
 - test/stripe/subscription_test.rb
 - test/stripe/three_d_secure_test.rb
@@ -166,6 +169,8 @@ signing_key:
 specification_version: 4
 summary: Ruby bindings for the Stripe API
 test_files:
+- test/api_fixtures.rb
+- test/api_stub_helpers.rb
 - test/stripe/account_test.rb
 - test/stripe/alipay_account_test.rb
 - test/stripe/api_operations_test.rb
@@ -177,7 +182,6 @@ test_files:
 - test/stripe/bank_account_test.rb
 - test/stripe/bitcoin_receiver_test.rb
 - test/stripe/bitcoin_transaction_test.rb
-- test/stripe/charge_refund_test.rb
 - test/stripe/charge_test.rb
 - test/stripe/country_spec_test.rb
 - test/stripe/coupon_test.rb
@@ -189,7 +193,6 @@ test_files:
 - test/stripe/invoice_item_test.rb
 - test/stripe/invoice_test.rb
 - test/stripe/list_object_test.rb
-- test/stripe/metadata_test.rb
 - test/stripe/order_return_test.rb
 - test/stripe/order_test.rb
 - test/stripe/plan_test.rb
@@ -200,7 +203,9 @@ test_files:
 - test/stripe/reversal_test.rb
 - test/stripe/sku_test.rb
 - test/stripe/source_test.rb
+- test/stripe/stripe_client_test.rb
 - test/stripe/stripe_object_test.rb
+- test/stripe/stripe_response_test.rb
 - test/stripe/subscription_item_test.rb
 - test/stripe/subscription_test.rb
 - test/stripe/three_d_secure_test.rb

data/test/stripe/charge_refund_test.rb

@@ -1,67 +0,0 @@
-require File.expand_path('../../test_helper', __FILE__)
-
-module Stripe
-  class ChargeRefundTest < Test::Unit::TestCase
-    should "refunds should be listable" do
-      @mock.expects(:get).once.returns(make_response(make_charge))
-
-      charge = Stripe::Charge.retrieve('test_charge')
-
-      assert charge.refunds.first.kind_of?(Stripe::Refund)
-    end
-
-    should "refunds should be refreshable" do
-      @mock.expects(:get).twice.returns(make_response(make_charge), make_response(make_refund(:id => 'refreshed_refund')))
-
-      charge = Stripe::Charge.retrieve('test_charge')
-      refund = charge.refunds.first
-      refund.refresh
-
-      assert_equal 'refreshed_refund', refund.id
-    end
-
-    should "refunds should be updateable" do
-      @mock.expects(:get).
-        with("#{Stripe.api_base}/v1/charges/test_charge", nil, nil).
-        once.returns(make_response(make_charge))
-      @mock.expects(:post).
-        with("#{Stripe.api_base}/v1/refunds/ref_test_refund", nil, 'metadata[key]=value').
-        once.returns(make_response(make_refund(:metadata => {'key' => 'value'})))
-
-      charge = Stripe::Charge.retrieve('test_charge')
-      refund = charge.refunds.first
-
-      assert_equal nil, refund.metadata['key']
-
-      refund.metadata['key'] = 'value'
-      refund.save
-
-      assert_equal 'value', refund.metadata['key']
-    end
-
-    should "create should return a new refund" do
-      @mock.expects(:get).
-        with("#{Stripe.api_base}/v1/charges/test_charge", nil, nil).
-        once.returns(make_response(make_charge(:id => 'test_charge')))
-      @mock.expects(:post).
-        with("#{Stripe.api_base}/v1/charges/test_charge/refunds", nil, 'amount=20').
-        once.returns(make_response(make_refund(:id => 'test_new_refund')))
-
-      charge = Stripe::Charge.retrieve('test_charge')
-      refund = charge.refunds.create(:amount => 20)
-      assert_equal 'test_new_refund', refund.id
-    end
-
-    should "should refund a charge with given amount" do
-      @mock.expects(:get).
-        with("#{Stripe.api_base}/v1/charges/test_charge", nil, nil).
-        twice.returns(make_response(make_charge(:id => 'test_charge')))
-      @mock.expects(:post).
-        with("#{Stripe.api_base}/v1/charges/test_charge/refunds", nil, 'amount=20').
-        once.returns(make_response(make_refund(:amount => 20)))
-
-      charge = Stripe::Charge.retrieve('test_charge')
-      charge.refund(:amount => 20)
-    end
-  end
-end

data/test/stripe/metadata_test.rb

@@ -1,129 +0,0 @@
-require File.expand_path('../../test_helper', __FILE__)
-
-module Stripe
-  class MetadataTest < Test::Unit::TestCase
-    setup do
-      @metadata_supported = {
-        :charge => {
-          :new => Stripe::Charge.method(:new),
-          :test => method(:make_charge),
-          :url => "/v1/charges/#{make_charge()[:id]}"
-        },
-        :customer => {
-          :new => Stripe::Customer.method(:new),
-          :test => method(:make_customer),
-          :url => "/v1/customers/#{make_customer()[:id]}"
-        },
-        :recipient => {
-          :new => Stripe::Recipient.method(:new),
-          :test => method(:make_recipient),
-          :url => "/v1/recipients/#{make_recipient()[:id]}"
-        },
-        :transfer => {
-          :new => Stripe::Transfer.method(:new),
-          :test => method(:make_transfer),
-          :url => "/v1/transfers/#{make_transfer()[:id]}"
-        },
-        :product => {
-          :new => Stripe::Product.method(:new),
-          :test => method(:make_product),
-          :url => "/v1/products/#{make_product()[:id]}"
-        },
-        :order => {
-          :new => Stripe::Order.method(:new),
-          :test => method(:make_order),
-          :url => "/v1/orders/#{make_order()[:id]}"
-        },
-        :sku => {
-          :new => Stripe::SKU.method(:new),
-          :test => method(:make_sku),
-          :url => "/v1/skus/#{make_sku()[:id]}"
-        },
-      }
-
-      @base_url = 'https://api.stripe.com'
-    end
-
-    should "not touch metadata" do
-      update_actions = lambda {|obj| obj.description = 'test'}
-      check_metadata({:metadata => {'initial' => 'true'}},
-                    'description=test',
-                    update_actions)
-    end
-
-
-    should "update metadata as a whole" do
-      update_actions = lambda {|obj| obj.metadata = {'uuid' => '6735'}}
-      check_metadata({:metadata => {}},
-                    'metadata[uuid]=6735',
-                    update_actions)
-
-      if is_greater_than_ruby_1_9?
-        check_metadata({:metadata => {:initial => 'true'}},
-                      'metadata[initial]=&metadata[uuid]=6735',
-                      update_actions)
-      end
-    end
-
-    should "update metadata keys individually" do
-      update_actions = lambda {|obj| obj.metadata['txn_id'] = '134a13'}
-      check_metadata({:metadata => {'initial' => 'true'}},
-                     'metadata[txn_id]=134a13',
-                     update_actions)
-    end
-
-    should "clear metadata as a whole" do
-      update_actions = lambda {|obj| obj.metadata = nil}
-      check_metadata({:metadata => {'initial' => 'true'}},
-                     'metadata=',
-                     update_actions)
-    end
-
-    should "clear metadata keys individually" do
-      update_actions = lambda {|obj| obj.metadata['initial'] = nil}
-      check_metadata({:metadata => {'initial' => 'true'}},
-                     'metadata[initial]=',
-                     update_actions)
-    end
-
-    should "handle combinations of whole and partial metadata updates" do
-      if is_greater_than_ruby_1_9?
-        update_actions = lambda do |obj|
-          obj.metadata = {'type' => 'summer'}
-          obj.metadata['uuid'] = '6735'
-        end
-        params = {:metadata => {'type' => 'summer', 'uuid' => '6735'}}
-        curl_args = Stripe::Util.encode_parameters(params)
-        check_metadata({:metadata => {'type' => 'christmas'}},
-                       curl_args,
-                       update_actions)
-      end
-    end
-
-    def check_metadata (initial_params, curl_args, metadata_update)
-      @metadata_supported.each do |name, methods|
-        neu = methods[:new]
-        test = methods[:test]
-        url = @base_url + methods[:url]
-
-        initial_test_obj = test.call(initial_params)
-        @mock.expects(:get).once.returns(make_response(initial_test_obj))
-
-        final_test_obj = test.call()
-        @mock.expects(:post).once.
-          returns(make_response(final_test_obj)).
-          with(url, nil, curl_args)
-
-        obj = neu.call("test")
-        obj.refresh()
-        metadata_update.call(obj)
-        obj.save
-      end
-    end
-
-    def is_greater_than_ruby_1_9?
-      version = RUBY_VERSION.dup  # clone preserves frozen state
-      Gem::Version.new(version) >= Gem::Version.new('1.9')
-    end
-  end
-end