stripe 1.22.0 → 1.23.0

data/test/stripe/application_fee_refund_test.rb

@@ -3,7 +3,7 @@ require File.expand_path('../../test_helper', __FILE__)
 module Stripe
   class ApplicationFeeRefundTest < Test::Unit::TestCase
     should "refunds should be listable" do
-      @mock.expects(:get).once.returns(test_response(test_application_fee))
+      @mock.expects(:get).once.returns(make_response(make_application_fee))
 
       application_fee = Stripe::ApplicationFee.retrieve('test_application_fee')
 
@@ -11,7 +11,7 @@ module Stripe
     end
 
     should "refunds should be refreshable" do
-      @mock.expects(:get).twice.returns(test_response(test_application_fee), test_response(test_application_fee_refund(:id => 'refreshed_refund')))
+      @mock.expects(:get).twice.returns(make_response(make_application_fee), make_response(make_application_fee_refund(:id => 'refreshed_refund')))
 
       application_fee = Stripe::ApplicationFee.retrieve('test_application_fee')
       refund = application_fee.refunds.first
@@ -21,8 +21,8 @@ module Stripe
     end
 
     should "refunds should be updateable" do
-      @mock.expects(:get).once.returns(test_response(test_application_fee))
-      @mock.expects(:post).once.returns(test_response(test_application_fee_refund(:metadata => {'key' => 'value'})))
+      @mock.expects(:get).once.returns(make_response(make_application_fee))
+      @mock.expects(:post).once.returns(make_response(make_application_fee_refund(:metadata => {'key' => 'value'})))
 
       application_fee = Stripe::ApplicationFee.retrieve('test_application_fee')
       refund = application_fee.refunds.first
@@ -36,8 +36,8 @@ module Stripe
     end
 
     should "create should return a new refund" do
-      @mock.expects(:get).once.returns(test_response(test_application_fee))
-      @mock.expects(:post).once.returns(test_response(test_application_fee_refund(:id => 'test_new_refund')))
+      @mock.expects(:get).once.returns(make_response(make_application_fee))
+      @mock.expects(:post).once.returns(make_response(make_application_fee_refund(:id => 'test_new_refund')))
 
       application_fee = Stripe::ApplicationFee.retrieve('test_application_fee')
       refund = application_fee.refunds.create(:amount => 20)

data/test/stripe/application_fee_test.rb

@@ -3,7 +3,7 @@ require File.expand_path('../../test_helper', __FILE__)
 module Stripe
   class ApplicationFeeTest < Test::Unit::TestCase
     should "application fees should be listable" do
-      @mock.expects(:get).once.returns(test_response(test_application_fee_array))
+      @mock.expects(:get).once.returns(make_response(make_application_fee_array))
       fees = Stripe::ApplicationFee.all
       assert fees.data.kind_of? Array
       fees.each do |fee|
@@ -13,7 +13,7 @@ module Stripe
 
     should "application fees should be refundable" do
       @mock.expects(:get).never
-      @mock.expects(:post).once.returns(test_response({:id => "fee_test_fee", :refunded => true}))
+      @mock.expects(:post).once.returns(make_response({:id => "fee_test_fee", :refunded => true}))
       fee = Stripe::ApplicationFee.new("test_application_fee")
       fee.refund
       assert fee.refunded

data/test/stripe/balance_test.rb

@@ -3,7 +3,7 @@ require File.expand_path('../../test_helper', __FILE__)
 module Stripe
   class BalanceTest < Test::Unit::TestCase
     should "balance should be retrievable" do
-      @mock.expects(:get).once.returns(test_response(test_balance))
+      @mock.expects(:get).once.returns(make_response(make_balance))
       balance = Stripe::Balance.retrieve
       assert_equal('balance', balance['object'])
     end

data/test/stripe/bitcoin_receiver_test.rb

@@ -3,19 +3,19 @@ require File.expand_path('../../test_helper', __FILE__)
 module Stripe
   class BitcoinReceiverTest < Test::Unit::TestCase
     should "retrieve should retrieve bitcoin receiver" do
-      @mock.expects(:get).once.returns(test_response(test_bitcoin_receiver))
+      @mock.expects(:get).once.returns(make_response(make_bitcoin_receiver))
       receiver = Stripe::BitcoinReceiver.retrieve('btcrcv_test_receiver')
       assert_equal 'btcrcv_test_receiver', receiver.id
     end
 
     should "create should create a bitcoin receiver" do
-      @mock.expects(:post).once.returns(test_response(test_bitcoin_receiver))
+      @mock.expects(:post).once.returns(make_response(make_bitcoin_receiver))
       receiver = Stripe::BitcoinReceiver.create
       assert_equal "btcrcv_test_receiver", receiver.id
     end
 
     should "all should list bitcoin receivers" do
-      @mock.expects(:get).once.returns(test_response(test_bitcoin_receiver_array))
+      @mock.expects(:get).once.returns(make_response(make_bitcoin_receiver_array))
       receivers = Stripe::BitcoinReceiver.all
       assert_equal 3, receivers.data.length
       assert receivers.data.kind_of? Array
@@ -28,32 +28,32 @@ module Stripe
     end
 
     should "maintain bitcoin transaction sublist" do
-      @mock.expects(:get).with("#{Stripe.api_base}/v1/bitcoin/receivers/btcrcv_test_receiver", nil, nil).once.returns(test_response(test_bitcoin_receiver))
+      @mock.expects(:get).with("#{Stripe.api_base}/v1/bitcoin/receivers/btcrcv_test_receiver", nil, nil).once.returns(make_response(make_bitcoin_receiver))
       receiver = Stripe::BitcoinReceiver.retrieve('btcrcv_test_receiver')
-      @mock.expects(:get).with("#{Stripe.api_base}/v1/bitcoin/receivers/btcrcv_test_receiver/transactions", nil, nil).once.returns(test_response(test_bitcoin_transaction_array))
+      @mock.expects(:get).with("#{Stripe.api_base}/v1/bitcoin/receivers/btcrcv_test_receiver/transactions", nil, nil).once.returns(make_response(make_bitcoin_transaction_array))
       transactions = receiver.transactions.all
       assert_equal(3, transactions.data.length)
     end
 
     should "update should update a bitcoin receiver" do
-      @mock.expects(:get).once.returns(test_response(test_bitcoin_receiver))
-      @mock.expects(:post).with("#{Stripe.api_base}/v1/bitcoin/receivers/btcrcv_test_receiver", nil, "description=details").once.returns(test_response(test_bitcoin_receiver))
-      receiver = Stripe::BitcoinReceiver.construct_from(test_bitcoin_receiver)
+      @mock.expects(:get).once.returns(make_response(make_bitcoin_receiver))
+      @mock.expects(:post).with("#{Stripe.api_base}/v1/bitcoin/receivers/btcrcv_test_receiver", nil, "description=details").once.returns(make_response(make_bitcoin_receiver))
+      receiver = Stripe::BitcoinReceiver.construct_from(make_bitcoin_receiver)
       receiver.refresh
       receiver.description = "details"
       receiver.save
     end
 
     should "delete a bitcoin receiver with no customer through top-level API" do
-      @mock.expects(:delete).with("#{Stripe.api_base}/v1/bitcoin/receivers/btcrcv_test_receiver", nil, nil).once.returns(test_response({:deleted => true, :id => "btcrcv_test_receiver"}))
-      receiver = Stripe::BitcoinReceiver.construct_from(test_bitcoin_receiver)
+      @mock.expects(:delete).with("#{Stripe.api_base}/v1/bitcoin/receivers/btcrcv_test_receiver", nil, nil).once.returns(make_response({:deleted => true, :id => "btcrcv_test_receiver"}))
+      receiver = Stripe::BitcoinReceiver.construct_from(make_bitcoin_receiver)
       response = receiver.delete
       assert(receiver.deleted)
     end
 
     should "delete a bitcoin receiver with a customer through customer's subresource API" do
-      @mock.expects(:delete).with("#{Stripe.api_base}/v1/customers/customer_foo/sources/btcrcv_test_receiver", nil, nil).once.returns(test_response({:deleted => true, :id => "btcrcv_test_receiver"}))
-      receiver = Stripe::BitcoinReceiver.construct_from(test_bitcoin_receiver(:customer => 'customer_foo'))
+      @mock.expects(:delete).with("#{Stripe.api_base}/v1/customers/customer_foo/sources/btcrcv_test_receiver", nil, nil).once.returns(make_response({:deleted => true, :id => "btcrcv_test_receiver"}))
+      receiver = Stripe::BitcoinReceiver.construct_from(make_bitcoin_receiver(:customer => 'customer_foo'))
       response = receiver.delete
       assert(receiver.deleted)
     end

data/test/stripe/charge_test.rb

@@ -3,7 +3,7 @@ require File.expand_path('../../test_helper', __FILE__)
 module Stripe
   class ChargeTest < Test::Unit::TestCase
     should "charges should be listable" do
-      @mock.expects(:get).once.returns(test_response(test_charge_array))
+      @mock.expects(:get).once.returns(make_response(make_charge_array))
       c = Stripe::Charge.all
       assert c.data.kind_of? Array
       c.each do |charge|
@@ -13,7 +13,7 @@ module Stripe
 
     should "charges should be refundable" do
       @mock.expects(:get).never
-      @mock.expects(:post).once.returns(test_response({:id => "ch_test_charge", :refunded => true}))
+      @mock.expects(:post).once.returns(make_response({:id => "ch_test_charge", :refunded => true}))
       c = Stripe::Charge.new("test_charge")
       c.refund
       assert c.refunded
@@ -21,15 +21,15 @@ module Stripe
 
     should "charges should not be deletable" do
       assert_raises NoMethodError do
-        @mock.expects(:get).once.returns(test_response(test_charge))
+        @mock.expects(:get).once.returns(make_response(make_charge))
         c = Stripe::Charge.retrieve("test_charge")
         c.delete
       end
     end
 
     should "charges should be updateable" do
-      @mock.expects(:get).once.returns(test_response(test_charge))
-      @mock.expects(:post).once.returns(test_response(test_charge))
+      @mock.expects(:get).once.returns(make_response(make_charge))
+      @mock.expects(:post).once.returns(make_response(make_charge))
       c = Stripe::Charge.new("test_charge")
       c.refresh
       c.mnemonic = "New charge description"
@@ -37,23 +37,23 @@ module Stripe
     end
 
     should "charges should be able to be marked as fraudulent" do
-      @mock.expects(:get).once.returns(test_response(test_charge))
-      @mock.expects(:post).once.returns(test_response(test_charge))
+      @mock.expects(:get).once.returns(make_response(make_charge))
+      @mock.expects(:post).once.returns(make_response(make_charge))
       c = Stripe::Charge.new("test_charge")
       c.refresh
       c.mark_as_fraudulent
     end
 
     should "charges should be able to be marked as safe" do
-      @mock.expects(:get).once.returns(test_response(test_charge))
-      @mock.expects(:post).once.returns(test_response(test_charge))
+      @mock.expects(:get).once.returns(make_response(make_charge))
+      @mock.expects(:post).once.returns(make_response(make_charge))
       c = Stripe::Charge.new("test_charge")
       c.refresh
       c.mark_as_safe
     end
 
     should "charges should have Card objects associated with their Card property" do
-      @mock.expects(:get).once.returns(test_response(test_charge))
+      @mock.expects(:get).once.returns(make_response(make_charge))
       c = Stripe::Charge.retrieve("test_charge")
       assert c.card.kind_of?(Stripe::StripeObject) && c.card.object == 'card'
     end
@@ -66,7 +66,7 @@ module Stripe
           'card[number]' => ['4242424242424242'],
           'card[exp_month]' => ['11']
         }
-      end.once.returns(test_response(test_charge))
+      end.once.returns(make_response(make_charge))
 
       c = Stripe::Charge.create({
         :amount => 100,
@@ -86,7 +86,7 @@ module Stripe
           'currency' => ['usd'], 'amount' => ['100'],
           'source' => ['btcrcv_test_receiver']
         }
-      end.once.returns(test_response(test_charge))
+      end.once.returns(make_response(make_charge))
 
       c = Stripe::Charge.create({
         :amount => 100,

data/test/stripe/coupon_test.rb

@@ -3,14 +3,14 @@ require File.expand_path('../../test_helper', __FILE__)
 module Stripe
   class CouponTest < Test::Unit::TestCase
     should "create should return a new coupon" do
-      @mock.expects(:post).once.returns(test_response(test_coupon))
+      @mock.expects(:post).once.returns(make_response(make_coupon))
       c = Stripe::Coupon.create
       assert_equal "co_test_coupon", c.id
     end
 
     should "coupons should be updateable" do
-      @mock.expects(:get).once.returns(test_response(test_coupon))
-      @mock.expects(:post).once.returns(test_response(test_coupon))
+      @mock.expects(:get).once.returns(make_response(make_coupon))
+      @mock.expects(:post).once.returns(make_response(make_coupon))
       c = Stripe::Coupon.new("test_coupon")
       c.refresh
       c.metadata['foo'] = 'bar'

data/test/stripe/customer_card_test.rb

@@ -5,13 +5,13 @@ module Stripe
     CUSTOMER_CARD_URL = '/v1/customers/test_customer/sources/test_card'
 
     def customer
-      @mock.expects(:get).once.returns(test_response(test_customer))
+      @mock.expects(:get).once.returns(make_response(make_customer))
       Stripe::Customer.retrieve('test_customer')
     end
 
     should "customer cards should be listable" do
       c = customer
-      @mock.expects(:get).once.returns(test_response(test_customer_card_array(customer.id)))
+      @mock.expects(:get).once.returns(make_response(make_customer_card_array(customer.id)))
       cards = c.sources.all(:object => "card").data
       assert cards.kind_of? Array
       assert cards[0].kind_of? Stripe::Card
@@ -19,7 +19,7 @@ module Stripe
 
     should "customer cards should have the correct url" do
       c = customer
-      @mock.expects(:get).once.returns(test_response(test_card(
+      @mock.expects(:get).once.returns(make_response(make_card(
         :id => 'test_card',
         :customer => 'test_customer'
       )))
@@ -29,8 +29,8 @@ module Stripe
 
     should "customer cards should be deletable" do
       c = customer
-      @mock.expects(:get).once.returns(test_response(test_card))
-      @mock.expects(:delete).once.returns(test_response(test_card(:deleted => true)))
+      @mock.expects(:get).once.returns(make_response(make_card))
+      @mock.expects(:delete).once.returns(make_response(make_card(:deleted => true)))
       card = c.sources.retrieve('card')
       card.delete
       assert card.deleted
@@ -38,8 +38,8 @@ module Stripe
 
     should "customer cards should be updateable" do
       c = customer
-      @mock.expects(:get).once.returns(test_response(test_card(:exp_year => "2000")))
-      @mock.expects(:post).once.returns(test_response(test_card(:exp_year => "2100")))
+      @mock.expects(:get).once.returns(make_response(make_card(:exp_year => "2000")))
+      @mock.expects(:post).once.returns(make_response(make_card(:exp_year => "2100")))
       card = c.sources.retrieve('card')
       assert_equal "2000", card.exp_year
       card.exp_year = "2100"
@@ -49,7 +49,7 @@ module Stripe
 
     should "create should return a new customer card" do
       c = customer
-      @mock.expects(:post).once.returns(test_response(test_card(:id => "test_card")))
+      @mock.expects(:post).once.returns(make_response(make_card(:id => "test_card")))
       card = c.sources.create(:source => "tok_41YJ05ijAaWaFS")
       assert_equal "test_card", card.id
     end

data/test/stripe/customer_test.rb

@@ -3,22 +3,22 @@ require File.expand_path('../../test_helper', __FILE__)
 module Stripe
   class CustomerTest < Test::Unit::TestCase
     should "customers should be listable" do
-      @mock.expects(:get).once.returns(test_response(test_customer_array))
+      @mock.expects(:get).once.returns(make_response(make_customer_array))
       c = Stripe::Customer.all.data
       assert c.kind_of? Array
       assert c[0].kind_of? Stripe::Customer
     end
 
     should "customers should be deletable" do
-      @mock.expects(:delete).once.returns(test_response(test_customer({:deleted => true})))
+      @mock.expects(:delete).once.returns(make_response(make_customer({:deleted => true})))
       c = Stripe::Customer.new("test_customer")
       c.delete
       assert c.deleted
     end
 
     should "customers should be updateable" do
-      @mock.expects(:get).once.returns(test_response(test_customer({:mnemonic => "foo"})))
-      @mock.expects(:post).once.returns(test_response(test_customer({:mnemonic => "bar"})))
+      @mock.expects(:get).once.returns(make_response(make_customer({:mnemonic => "foo"})))
+      @mock.expects(:post).once.returns(make_response(make_customer({:mnemonic => "bar"})))
       c = Stripe::Customer.new("test_customer").refresh
       assert_equal "foo", c.mnemonic
       c.mnemonic = "bar"
@@ -27,24 +27,24 @@ module Stripe
     end
 
     should "create should return a new customer" do
-      @mock.expects(:post).once.returns(test_response(test_customer))
+      @mock.expects(:post).once.returns(make_response(make_customer))
       c = Stripe::Customer.create
       assert_equal "c_test_customer", c.id
     end
 
     should "create_upcoming_invoice should create a new invoice" do
-      @mock.expects(:post).once.returns(test_response(test_invoice))
+      @mock.expects(:post).once.returns(make_response(make_invoice))
       i = Stripe::Customer.new("test_customer").create_upcoming_invoice
       assert_equal "c_test_customer", i.customer
     end
 
     should "be able to update a customer's subscription" do
-      @mock.expects(:get).once.returns(test_response(test_customer))
+      @mock.expects(:get).once.returns(make_response(make_customer))
       c = Stripe::Customer.retrieve("test_customer")
 
       @mock.expects(:post).once.with do |url, api_key, params|
         url == "#{Stripe.api_base}/v1/customers/c_test_customer/subscription" && api_key.nil? && CGI.parse(params) == {'plan' => ['silver']}
-      end.returns(test_response(test_subscription(:plan => 'silver')))
+      end.returns(make_response(make_subscription(:plan => 'silver')))
       s = c.update_subscription({:plan => 'silver'})
 
       assert_equal 'subscription', s.object
@@ -52,15 +52,15 @@ module Stripe
     end
 
     should "be able to cancel a customer's subscription" do
-      @mock.expects(:get).once.returns(test_response(test_customer))
+      @mock.expects(:get).once.returns(make_response(make_customer))
       c = Stripe::Customer.retrieve("test_customer")
 
       # Not an accurate response, but whatever
 
-      @mock.expects(:delete).once.with("#{Stripe.api_base}/v1/customers/c_test_customer/subscription?at_period_end=true", nil, nil).returns(test_response(test_subscription(:plan => 'silver')))
+      @mock.expects(:delete).once.with("#{Stripe.api_base}/v1/customers/c_test_customer/subscription?at_period_end=true", nil, nil).returns(make_response(make_subscription(:plan => 'silver')))
       c.cancel_subscription({:at_period_end => 'true'})
 
-      @mock.expects(:delete).once.with("#{Stripe.api_base}/v1/customers/c_test_customer/subscription", nil, nil).returns(test_response(test_subscription(:plan => 'silver')))
+      @mock.expects(:delete).once.with("#{Stripe.api_base}/v1/customers/c_test_customer/subscription", nil, nil).returns(make_response(make_subscription(:plan => 'silver')))
       c.cancel_subscription
     end
 
@@ -69,7 +69,7 @@ module Stripe
 
       @mock.expects(:post).once.with do |url, api_key, params|
         url == "#{Stripe.api_base}/v1/customers/test_customer/subscriptions" && api_key.nil? && CGI.parse(params) == {'plan' => ['silver']}
-      end.returns(test_response(test_subscription(:plan => 'silver')))
+      end.returns(make_response(make_subscription(:plan => 'silver')))
       s = c.create_subscription({:plan => 'silver'})
 
       assert_equal 'subscription', s.object
@@ -77,10 +77,10 @@ module Stripe
     end
 
     should "be able to delete a customer's discount" do
-      @mock.expects(:get).once.returns(test_response(test_customer))
+      @mock.expects(:get).once.returns(make_response(make_customer))
       c = Stripe::Customer.retrieve("test_customer")
 
-      @mock.expects(:delete).once.with("#{Stripe.api_base}/v1/customers/c_test_customer/discount", nil, nil).returns(test_response(test_delete_discount_response))
+      @mock.expects(:delete).once.with("#{Stripe.api_base}/v1/customers/c_test_customer/discount", nil, nil).returns(make_response(make_delete_discount_response))
       c.delete_discount
       assert_equal nil, c.discount
     end

data/test/stripe/file_upload_test.rb

@@ -3,7 +3,7 @@ require File.expand_path('../../test_helper', __FILE__)
 module Stripe
   class FileUploadTest < Test::Unit::TestCase
     should "create should return a new file" do
-      @mock.expects(:post).once.returns(test_response(test_file))
+      @mock.expects(:post).once.returns(make_response(make_file))
       f = Stripe::FileUpload.create({
         :purpose => "dispute_evidence",
         :file => File.new(__FILE__),
@@ -12,14 +12,14 @@ module Stripe
     end
 
     should "files should be retrievable" do
-      @mock.expects(:get).once.returns(test_response(test_file))
+      @mock.expects(:get).once.returns(make_response(make_file))
       c = Stripe::FileUpload.new("fil_test_file")
       c.refresh
       assert_equal 1403047735, c.created
     end
 
     should "files should be listable" do
-      @mock.expects(:get).once.returns(test_response(test_file_array))
+      @mock.expects(:get).once.returns(make_response(make_file_array))
       c = Stripe::FileUpload.all.data
       assert c.kind_of? Array
       assert c[0].kind_of? Stripe::FileUpload

data/test/stripe/invoice_test.rb

@@ -3,35 +3,35 @@ require File.expand_path('../../test_helper', __FILE__)
 module Stripe
   class InvoiceTest < Test::Unit::TestCase
     should "retrieve should retrieve invoices" do
-      @mock.expects(:get).once.returns(test_response(test_invoice))
+      @mock.expects(:get).once.returns(make_response(make_invoice))
       i = Stripe::Invoice.retrieve('in_test_invoice')
       assert_equal 'in_test_invoice', i.id
     end
 
     should "create should create a new invoice" do
-      @mock.expects(:post).once.returns(test_response(test_invoice))
+      @mock.expects(:post).once.returns(make_response(make_invoice))
       i = Stripe::Invoice.create
       assert_equal "in_test_invoice", i.id
     end
 
     should "pay should pay an invoice" do
-      @mock.expects(:get).once.returns(test_response(test_invoice))
+      @mock.expects(:get).once.returns(make_response(make_invoice))
       i = Stripe::Invoice.retrieve('in_test_invoice')
 
-      @mock.expects(:post).once.with('https://api.stripe.com/v1/invoices/in_test_invoice/pay', nil, '').returns(test_response(test_paid_invoice))
+      @mock.expects(:post).once.with('https://api.stripe.com/v1/invoices/in_test_invoice/pay', nil, '').returns(make_response(make_paid_invoice))
       i.pay
       assert_equal nil, i.next_payment_attempt
     end
 
     should "pay with extra opts should pay an invoice" do
-      @mock.expects(:get).once.returns(test_response(test_invoice))
+      @mock.expects(:get).once.returns(make_response(make_invoice))
       i = Stripe::Invoice.retrieve('in_test_invoice', {:api_key => 'foobar'})
 
       Stripe.expects(:execute_request).with do |opts|
         opts[:url] == "#{Stripe.api_base}/v1/invoices/in_test_invoice/pay" &&
           opts[:method] == :post &&
           opts[:headers][:authorization] == 'Bearer foobar'
-      end.returns(test_response(test_paid_invoice))
+      end.returns(make_response(make_paid_invoice))
 
       i.pay
       assert_equal nil, i.next_payment_attempt