mirror of
https://github.com/stripe/stripe-ruby.git
synced 2025-10-04 00:00:47 -04:00
Cleanup unused test variables
This commit is contained in:
parent
294d6c63b3
commit
3e019adb6a
@ -5,12 +5,12 @@ module Stripe
|
|||||||
class ApiResourceTest < Test::Unit::TestCase
|
class ApiResourceTest < Test::Unit::TestCase
|
||||||
should "creating a new APIResource should not fetch over the network" do
|
should "creating a new APIResource should not fetch over the network" do
|
||||||
@mock.expects(:get).never
|
@mock.expects(:get).never
|
||||||
c = Stripe::Customer.new("someid")
|
Stripe::Customer.new("someid")
|
||||||
end
|
end
|
||||||
|
|
||||||
should "creating a new APIResource from a hash should not fetch over the network" do
|
should "creating a new APIResource from a hash should not fetch over the network" do
|
||||||
@mock.expects(:get).never
|
@mock.expects(:get).never
|
||||||
c = Stripe::Customer.construct_from({
|
Stripe::Customer.construct_from({
|
||||||
:id => "somecustomer",
|
:id => "somecustomer",
|
||||||
:card => {:id => "somecard", :object => "card"},
|
:card => {:id => "somecard", :object => "card"},
|
||||||
:object => "customer"
|
:object => "customer"
|
||||||
@ -185,14 +185,14 @@ module Stripe
|
|||||||
(url =~ %r{^#{Stripe.api_base}/v1/charges?} &&
|
(url =~ %r{^#{Stripe.api_base}/v1/charges?} &&
|
||||||
query.keys.sort == ['offset', 'sad'])
|
query.keys.sort == ['offset', 'sad'])
|
||||||
end.returns(test_response({ :count => 1, :data => [test_charge] }))
|
end.returns(test_response({ :count => 1, :data => [test_charge] }))
|
||||||
c = Stripe::Charge.all(:count => nil, :offset => 5, :sad => false)
|
Stripe::Charge.all(:count => nil, :offset => 5, :sad => false)
|
||||||
|
|
||||||
@mock.expects(:post).with do |url, api_key, params|
|
@mock.expects(:post).with do |url, api_key, params|
|
||||||
url == "#{Stripe.api_base}/v1/charges" &&
|
url == "#{Stripe.api_base}/v1/charges" &&
|
||||||
api_key.nil? &&
|
api_key.nil? &&
|
||||||
CGI.parse(params) == { 'amount' => ['50'], 'currency' => ['usd'] }
|
CGI.parse(params) == { 'amount' => ['50'], 'currency' => ['usd'] }
|
||||||
end.returns(test_response({ :count => 1, :data => [test_charge] }))
|
end.returns(test_response({ :count => 1, :data => [test_charge] }))
|
||||||
c = Stripe::Charge.create(:amount => 50, :currency => 'usd', :card => { :number => nil })
|
Stripe::Charge.create(:amount => 50, :currency => 'usd', :card => { :number => nil })
|
||||||
end
|
end
|
||||||
|
|
||||||
should "requesting with a unicode ID should result in a request" do
|
should "requesting with a unicode ID should result in a request" do
|
||||||
@ -210,7 +210,7 @@ module Stripe
|
|||||||
should "making a GET request with parameters should have a query string and no body" do
|
should "making a GET request with parameters should have a query string and no body" do
|
||||||
params = { :limit => 1 }
|
params = { :limit => 1 }
|
||||||
@mock.expects(:get).once.with("#{Stripe.api_base}/v1/charges?limit=1", nil, nil).returns(test_response([test_charge]))
|
@mock.expects(:get).once.with("#{Stripe.api_base}/v1/charges?limit=1", nil, nil).returns(test_response([test_charge]))
|
||||||
c = Stripe::Charge.all(params)
|
Stripe::Charge.all(params)
|
||||||
end
|
end
|
||||||
|
|
||||||
should "making a POST request with parameters should have a body and no query string" do
|
should "making a POST request with parameters should have a body and no query string" do
|
||||||
@ -218,7 +218,7 @@ module Stripe
|
|||||||
@mock.expects(:post).once.with do |url, get, post|
|
@mock.expects(:post).once.with do |url, get, post|
|
||||||
get.nil? && CGI.parse(post) == {'amount' => ['100'], 'currency' => ['usd'], 'card' => ['sc_token']}
|
get.nil? && CGI.parse(post) == {'amount' => ['100'], 'currency' => ['usd'], 'card' => ['sc_token']}
|
||||||
end.returns(test_response(test_charge))
|
end.returns(test_response(test_charge))
|
||||||
c = Stripe::Charge.create(params)
|
Stripe::Charge.create(params)
|
||||||
end
|
end
|
||||||
|
|
||||||
should "loading an object should issue a GET request" do
|
should "loading an object should issue a GET request" do
|
||||||
|
@ -52,10 +52,10 @@ module Stripe
|
|||||||
# Not an accurate response, but whatever
|
# 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('silver')))
|
@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('silver')))
|
||||||
s = c.cancel_subscription({:at_period_end => 'true'})
|
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('silver')))
|
@mock.expects(:delete).once.with("#{Stripe.api_base}/v1/customers/c_test_customer/subscription", nil, nil).returns(test_response(test_subscription('silver')))
|
||||||
s = c.cancel_subscription
|
c.cancel_subscription
|
||||||
end
|
end
|
||||||
|
|
||||||
should "be able to delete a customer's discount" do
|
should "be able to delete a customer's discount" do
|
||||||
@ -63,7 +63,7 @@ module Stripe
|
|||||||
c = Stripe::Customer.retrieve("test_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(test_response(test_delete_discount_response))
|
||||||
s = c.delete_discount
|
c.delete_discount
|
||||||
assert_equal nil, c.discount
|
assert_equal nil, c.discount
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
Loading…
x
Reference in New Issue
Block a user