stripe-ruby-mock 2.5.6 → 3.1.0.rc2

data/spec/instance_spec.rb

@@ -13,13 +13,13 @@ describe StripeMock::Instance do
   after { StripeMock.stop }
 
   it "handles both string and symbol hash keys" do
-    string_params = stripe_helper.create_plan_params(
-      "id" => "str_abcde",
-      :name => "String Plan"
+    symbol_params = stripe_helper.create_product_params(
+      :name => "Symbol Product",
+      "type" => "service"
     )
-    res, api_key = StripeMock.instance.mock_request('post', '/v1/plans', api_key: 'api_key', params: string_params)
-    expect(res.data[:id]).to eq('str_abcde')
-    expect(res.data[:name]).to eq('String Plan')
+    res, api_key = StripeMock.instance.mock_request('post', '/v1/products', api_key: 'api_key', params: symbol_params)
+    expect(res.data[:name]).to eq('Symbol Product')
+    expect(res.data[:type]).to eq('service')
   end
 
   it "exits gracefully on an unrecognized handler url" do
@@ -54,17 +54,15 @@ describe StripeMock::Instance do
   end
 
   it "allows non-usd default currency" do
+    pending("Stripe::Plan requires currency param - how can we test this?")
     old_default_currency = StripeMock.default_currency
-    customer = begin
+    plan = begin
       StripeMock.default_currency = "jpy"
-      Stripe::Customer.create({
-        email: 'johnny@appleseed.com',
-        source: stripe_helper.generate_card_token
-      })
+      Stripe::Plan.create(interval: 'month')
     ensure
       StripeMock.default_currency = old_default_currency
     end
-    expect(customer.currency).to eq("jpy")
+    expect(plan.currency).to eq("jpy")
   end
 
   context 'when creating sources with metadata' do

data/spec/list_spec.rb

@@ -109,6 +109,21 @@ describe StripeMock::Data::List do
     end
   end
 
+  context "active filter" do
+    it "accepts an active param which filters out data accordingly" do
+      product = Stripe::Product.create(id: "prod_123", name: "My Beautiful Product", type: "service")
+
+      plan_attributes = { product: product.id, interval: "month", currency: "usd", amount: 500 }
+      plan_a = Stripe::Plan.create(plan_attributes)
+      plan_b = Stripe::Plan.create(**plan_attributes, active: false)
+
+      list = StripeMock::Data::List.new([plan_a, plan_b], active: true)
+
+      expect(list.active).to eq(true)
+      expect(list.to_h[:data].count).to eq(1)
+    end
+  end
+
   context "pagination" do
     it "has a has_more field when it has more" do
       list = StripeMock::Data::List.new(
@@ -137,4 +152,27 @@ describe StripeMock::Data::List do
       expect { list.to_h }.to raise_error
     end
   end
+
+  context "with data containing records marked 'deleted'" do
+    let(:customer_data) { StripeMock.instance.customers.values }
+    let(:customers) do
+      customer_data.map { |datum| Stripe::Util.convert_to_stripe_object(datum) }
+    end
+
+    before do
+      StripeMock.instance.customers.clear
+      Stripe::Customer.create
+      Stripe::Customer.delete(Stripe::Customer.create.id)
+    end
+
+    it "does not raise error on initialization" do
+      expect { StripeMock::Data::List.new(customer_data) }.to_not raise_error
+      expect { StripeMock::Data::List.new(customers) }.to_not raise_error
+    end
+
+    it "omits records marked 'deleted'" do
+      expect(StripeMock::Data::List.new(customer_data).data.size).to eq(1)
+      expect(StripeMock::Data::List.new(customers).data.size).to eq(1)
+    end
+  end
 end

data/spec/server_spec.rb

@@ -19,6 +19,7 @@ describe 'StripeMock Server', :mock_server => true do
 
   after { StripeMock.stop_client(:clear_server_data => true) }
 
+  let(:product) { stripe_helper.create_product }
 
   it "uses an RPC client for mock requests" do
     charge = Stripe::Charge.create(
@@ -37,21 +38,23 @@ describe 'StripeMock Server', :mock_server => true do
     customer = Stripe::Customer.create(email: 'johnny@appleseed.com')
     expect(customer.email).to eq('johnny@appleseed.com')
 
-    server_customer_data = StripeMock.client.get_server_data(:customers)[customer.id]
+    server_customer_data = StripeMock.client.get_server_data(:customers)
+    server_customer_data = server_customer_data[server_customer_data.keys.first][customer.id.to_sym]
     expect(server_customer_data).to_not be_nil
     expect(server_customer_data[:email]).to eq('johnny@appleseed.com')
 
     StripeMock.stop_client
     StripeMock.start_client
 
-    server_customer_data = StripeMock.client.get_server_data(:customers)[customer.id]
+    server_customer_data = StripeMock.client.get_server_data(:customers)
+    server_customer_data = server_customer_data[server_customer_data.keys.first][customer.id.to_sym]
     expect(server_customer_data).to_not be_nil
     expect(server_customer_data[:email]).to eq('johnny@appleseed.com')
   end
 
 
   it "returns a response with symbolized hash keys" do
-    stripe_helper.create_plan(id: 'x')
+    stripe_helper.create_plan(id: 'x', product: product.id)
     response, api_key = StripeMock.redirect_to_mock_server('get', '/v1/plans/x', api_key: 'xxx')
     response.data.keys.each {|k| expect(k).to be_a(Symbol) }
   end

data/spec/shared_stripe_examples/account_examples.rb

@@ -15,7 +15,7 @@ shared_examples 'Account API' do
       expect(account.id).to match /acct\_/
     end
     it 'retrieves all' do
-      accounts = Stripe::Account.all
+      accounts = Stripe::Account.list
 
       expect(accounts).to be_a Stripe::ListObject
       expect(accounts.data.count).to satisfy { |n| n >= 1 }

data/spec/shared_stripe_examples/account_link_examples.rb

@@ -0,0 +1,16 @@
+require 'spec_helper'
+
+shared_examples 'Account Link API' do
+  describe 'create account link' do
+    it 'creates an account link' do
+      account_link = Stripe::AccountLink.create(
+        type: 'onboarding',
+        account: 'acct_103ED82ePvKYlo2C',
+        failure_url: 'https://stripe.com',
+        success_url: 'https://stripe.com'
+      )
+
+      expect(account_link).to be_a Stripe::AccountLink
+    end
+  end
+end

data/spec/shared_stripe_examples/balance_examples.rb

@@ -8,4 +8,10 @@ shared_examples 'Balance API' do
     expect(balance.available[0].amount).to eq(2000)
   end
 
+  it "retrieves a stripe instant balance" do
+    StripeMock.set_account_balance(2000)
+    balance = Stripe::Balance.retrieve()
+    expect(balance.instant_available[0].amount).to eq(2000)
+  end
+
 end

data/spec/shared_stripe_examples/balance_transaction_examples.rb

@@ -26,7 +26,7 @@ shared_examples 'Balance Transaction API' do
   describe "listing balance transactions" do
 
     it "retrieves all balance transactions" do
-      disputes = Stripe::BalanceTransaction.all
+      disputes = Stripe::BalanceTransaction.list
 
       expect(disputes.count).to eq(10)
       expect(disputes.map &:id).to include('txn_05RsQX2eZvKYlo2C0FRTGSSA','txn_15RsQX2eZvKYlo2C0ERTYUIA', 'txn_25RsQX2eZvKYlo2C0ZXCVBNM', 'txn_35RsQX2eZvKYlo2C0QAZXSWE', 'txn_45RsQX2eZvKYlo2C0EDCVFRT', 'txn_55RsQX2eZvKYlo2C0OIKLJUY', 'txn_65RsQX2eZvKYlo2C0ASDFGHJ', 'txn_75RsQX2eZvKYlo2C0EDCXSWQ', 'txn_85RsQX2eZvKYlo2C0UJMCDET', 'txn_95RsQX2eZvKYlo2C0EDFRYUI')
@@ -38,7 +38,7 @@ shared_examples 'Balance Transaction API' do
     transfer_id = Stripe::Transfer.create({ amount: 2730, currency: "usd" })
 
     # verify transfer currently has no balance transactions
-    transfer_transactions = Stripe::BalanceTransaction.all({transfer: transfer_id})
+    transfer_transactions = Stripe::BalanceTransaction.list({transfer: transfer_id})
     expect(transfer_transactions.count).to eq(0)
 
     # verify we can create a new balance transaction associated with the transfer
@@ -55,7 +55,7 @@ shared_examples 'Balance Transaction API' do
     stripe_helper.upsert_stripe_object(:balance_transaction, {id: existing_txn_id, transfer: transfer_id})
 
     # now verify that only these balance transactions are retrieved with the transfer
-    transfer_transactions = Stripe::BalanceTransaction.all({transfer: transfer_id})
+    transfer_transactions = Stripe::BalanceTransaction.list({transfer: transfer_id})
     expect(transfer_transactions.count).to eq(2)
     expect(transfer_transactions.map &:id).to include(new_txn_id, existing_txn_id)
   end

data/spec/shared_stripe_examples/bank_examples.rb

@@ -77,7 +77,7 @@ shared_examples 'Bank API' do
     let!(:bank) { customer.sources.create(source: bank_token) }
 
     it "can retrieve all customer's banks" do
-      retrieved = customer.sources.all
+      retrieved = customer.sources.list
       expect(retrieved.count).to eq(1)
     end
 
@@ -174,7 +174,7 @@ shared_examples 'Bank API' do
 
       customer = Stripe::Customer.retrieve('test_customer_bank')
 
-      list = customer.sources.all
+      list = customer.sources.list
 
       expect(list.object).to eq("list")
       expect(list.count).to eq(2)
@@ -191,7 +191,7 @@ shared_examples 'Bank API' do
       Stripe::Customer.create(id: 'no_banks')
       customer = Stripe::Customer.retrieve('no_banks')
 
-      list = customer.sources.all
+      list = customer.sources.list
 
       expect(list.object).to eq("list")
       expect(list.count).to eq(0)

data/spec/shared_stripe_examples/card_examples.rb

@@ -134,7 +134,7 @@ shared_examples 'Card API' do
     let!(:card) { customer.sources.create(source: card_token) }
 
     it "can retrieve all customer's cards" do
-      retrieved = customer.sources.all
+      retrieved = customer.sources.list
       expect(retrieved.count).to eq(1)
     end
 
@@ -198,7 +198,7 @@ shared_examples 'Card API' do
     let!(:card) { recipient.cards.create(card: card_token) }
 
     it "can retrieve all recipient's cards" do
-      retrieved = recipient.cards.all
+      retrieved = recipient.cards.list
       expect(retrieved.count).to eq(1)
     end
 
@@ -279,7 +279,7 @@ shared_examples 'Card API' do
 
       customer = Stripe::Customer.retrieve('test_customer_card')
 
-      list = customer.sources.all
+      list = customer.sources.list
 
       expect(list.object).to eq("list")
       expect(list.count).to eq(2)
@@ -296,7 +296,7 @@ shared_examples 'Card API' do
       Stripe::Customer.create(id: 'no_cards')
       customer = Stripe::Customer.retrieve('no_cards')
 
-      list = customer.sources.all
+      list = customer.sources.list
 
       expect(list.object).to eq("list")
       expect(list.count).to eq(0)

data/spec/shared_stripe_examples/card_token_examples.rb

@@ -145,34 +145,30 @@ shared_examples 'Card Token Mocking' do
     end
 
     it 'generates a card token from another card', oauth: true do
-      token = Stripe::Token.create(
-          card: {
-              exp_month: 10,
-              exp_year: 2016,
-              number: '4242424242424242'
-          }
-      )
+      token = StripeMock.generate_card_token(last4: "2244", exp_month: 33, exp_year: 2255)
 
-      cus1 = Stripe::Customer.create(source: token.id)
+      cus1 = Stripe::Customer.create()
+      cus1.source = token
+      cus1.save
 
       card1 = cus1.sources.data.first
-      expect(card1.last4).to eq('4242')
-      expect(card1.exp_month).to eq(10)
-      expect(card1.exp_year).to eq(2016)
+      expect(card1.last4).to eq("2244")
+      expect(card1.exp_month).to eq(33)
+      expect(card1.exp_year).to eq(2255)
 
-      card_token = Stripe::Token.create(
-          {
-              customer: cus1.id,
-              card: card1.id
-          },
-          ENV['STRIPE_TEST_OAUTH_ACCESS_TOKEN'])
+      card_token = Stripe::Token.create({
+        customer: cus1.id,
+        card: card1.id
+      })
 
-      cus2 = Stripe::Customer.create({ source: card_token.id }, ENV['STRIPE_TEST_OAUTH_ACCESS_TOKEN'])
+      cus2 = Stripe::Customer.create({}, ENV['STRIPE_TEST_OAUTH_ACCESS_TOKEN'])
+      cus2.source = card_token.id
+      cus2.save
 
       card2 = cus2.sources.data.first
-      expect(card2.last4).to eq('4242')
-      expect(card2.exp_month).to eq(10)
-      expect(card2.exp_year).to eq(2016)
+      expect(card2.last4).to eq("2244")
+      expect(card2.exp_month).to eq(33)
+      expect(card2.exp_year).to eq(2255)
     end
 
     it "throws an error if neither card nor customer are provided", :live => true do

data/spec/shared_stripe_examples/charge_examples.rb

@@ -163,15 +163,19 @@ shared_examples 'Charge API' do
   end
 
   it "creates a balance transaction" do
+    amount = 300
+    fee = 10
     charge = Stripe::Charge.create({
-      amount: 300,
+      amount: amount,
       currency: 'USD',
-      source: stripe_helper.generate_card_token
+      source: stripe_helper.generate_card_token,
+      application_fee: fee,
     })
     bal_trans = Stripe::BalanceTransaction.retrieve(charge.balance_transaction)
-    expect(bal_trans.amount).to eq(charge.amount)
-    expect(bal_trans.fee).to eq(39)
+    expect(bal_trans.amount).to eq(amount)
+    expect(bal_trans.fee).to eq(39 + fee)
     expect(bal_trans.source).to eq(charge.id)
+    expect(bal_trans.net).to eq(amount - bal_trans.fee)
   end
 
   context 'when conversion rate is set' do
@@ -274,20 +278,6 @@ shared_examples 'Charge API' do
     }.not_to raise_error
   end
 
-  it "marks a charge as safe" do
-    original = Stripe::Charge.create({
-      amount: 777,
-      currency: 'USD',
-      source: stripe_helper.generate_card_token
-    })
-    charge = Stripe::Charge.retrieve(original.id)
-
-    charge.mark_as_safe
-
-    updated = Stripe::Charge.retrieve(original.id)
-    expect(updated.fraud_details[:user_report]).to eq "safe"
-  end
-
   it "does not lose data when updating a charge" do
     original = Stripe::Charge.create({
       amount: 777,
@@ -353,28 +343,29 @@ shared_examples 'Charge API' do
     end
 
     it "stores charges for a customer in memory" do
-      expect(@customer.charges.data.map(&:id)).to eq([@charge.id])
+      charges = Stripe::Charge.list(customer: @customer.id)
+      expect(charges.map(&:id)).to eq([@charge.id])
     end
 
     it "stores all charges in memory" do
-      expect(Stripe::Charge.all.data.map(&:id).reverse).to eq([@charge.id, @charge2.id])
+      expect(Stripe::Charge.list.data.map(&:id).reverse).to eq([@charge.id, @charge2.id])
     end
 
     it "defaults count to 10 charges" do
       11.times { Stripe::Charge.create(amount: 1, currency: 'usd', source: stripe_helper.generate_card_token) }
 
-      expect(Stripe::Charge.all.data.count).to eq(10)
+      expect(Stripe::Charge.list.data.count).to eq(10)
     end
 
     it "is marked as having more when more objects exist" do
       11.times { Stripe::Charge.create(amount: 1, currency: 'usd', source: stripe_helper.generate_card_token) }
 
-      expect(Stripe::Charge.all.has_more).to eq(true)
+      expect(Stripe::Charge.list.has_more).to eq(true)
     end
 
     context "when passing limit" do
       it "gets that many charges" do
-        expect(Stripe::Charge.all(limit: 1).count).to eq(1)
+        expect(Stripe::Charge.list(limit: 1).count).to eq(1)
       end
     end
   end
@@ -394,13 +385,13 @@ shared_examples 'Charge API' do
       Stripe::Charge.create(customer: cus.id, amount: 100, currency: "usd")
     end
 
-    all = Stripe::Charge.all
+    all_charges = Stripe::Charge.list
     default_limit = 10
-    half = Stripe::Charge.all(starting_after: all.data.at(1).id)
+    half = Stripe::Charge.list(starting_after: all_charges.data.at(1).id)
 
     expect(half).to be_a(Stripe::ListObject)
     expect(half.data.count).to eq(default_limit)
-    expect(half.data.first.id).to eq(all.data.at(2).id)
+    expect(half.data.first.id).to eq(all_charges.data.at(2).id)
   end
 
 
@@ -472,29 +463,34 @@ shared_examples 'Charge API' do
 
   describe "idempotency" do
     let(:customer) { Stripe::Customer.create(email: 'johnny@appleseed.com') }
-    let(:idempotent_charge_params) {{
+    let(:charge_params) {{
       amount: 777,
       currency: 'USD',
       customer: customer.id,
-      capture: true,
+      capture: true
+    }}
+    let(:charge_headers) {{
       idempotency_key: 'onceisenough'
     }}
 
     it "returns the original charge if the same idempotency_key is passed in" do
-      charge1 = Stripe::Charge.create(idempotent_charge_params)
-      charge2 = Stripe::Charge.create(idempotent_charge_params)
+      charge1 = Stripe::Charge.create(charge_params, charge_headers)
+      charge2 = Stripe::Charge.create(charge_params, charge_headers)
 
       expect(charge1).to eq(charge2)
     end
 
-    it "returns different charges if different idempotency_keys are used for each charge" do
-      idempotent_charge_params2 = idempotent_charge_params.clone
-      idempotent_charge_params2[:idempotency_key] = 'thisoneisdifferent'
+    context 'different key' do
+      let(:different_charge_headers) {{
+        idempotency_key: 'thisoneisdifferent'
+      }}
 
-      charge1 = Stripe::Charge.create(idempotent_charge_params)
-      charge2 = Stripe::Charge.create(idempotent_charge_params2)
+      it "returns different charges if different idempotency_keys are used for each charge" do
+        charge1 = Stripe::Charge.create(charge_params, charge_headers)
+        charge2 = Stripe::Charge.create(charge_params, different_charge_headers)
 
-      expect(charge1).not_to eq(charge2)
+        expect(charge1).not_to eq(charge2)
+      end
     end
   end