stripe-ruby-mock 2.1.1 → 2.2.0

data/lib/stripe_mock/request_handlers/helpers/card_helpers.rb

@@ -23,14 +23,14 @@ module StripeMock
 
         is_customer = object.has_key?(:sources)
 
-        if replace_current
+        if replace_current && cards_or_sources[:data]
           cards_or_sources[:data].delete_if {|card| card[:id] == object[:default_card]}
           object[:default_card]   = card[:id] unless is_customer
           object[:default_source] = card[:id] if is_customer
           cards_or_sources[:data] = [card]
         else
-          cards_or_sources[:total_count] += 1
-          cards_or_sources[:data] << card
+          cards_or_sources[:total_count] = cards_or_sources[:total_count] || 0 + 1
+          (cards_or_sources[:data] ||= []) << card
         end
 
         object[:default_card]   = card[:id] if !is_customer && object[:default_card].nil?

data/lib/stripe_mock/request_handlers/helpers/coupon_helpers.rb

@@ -0,0 +1,13 @@
+module StripeMock
+  module RequestHandlers
+    module Helpers
+
+      def add_coupon_to_customer(customer, coupon)
+        customer[:discount] = { coupon: coupon }
+
+        customer
+      end
+
+    end
+  end
+end

data/lib/stripe_mock/request_handlers/invoices.rb

@@ -20,6 +20,7 @@ module StripeMock
 
       def update_invoice(route, method_url, params, headers)
         route =~ method_url
+        params.delete(:lines) if params[:lines]
         assert_existence :invoice, $1, invoices[$1]
         invoices[$1].merge!(params)
       end

data/lib/stripe_mock/request_handlers/subscriptions.rb

@@ -28,8 +28,23 @@ module StripeMock
 
         subscription = Data.mock_subscription({ id: (params[:id] || new_id('su')) })
         subscription.merge!(custom_subscription_params(plan, customer, params))
+
+        if params[:coupon]
+          coupon_id = params[:coupon]
+
+          raise Stripe::InvalidRequestError.new("No such coupon: #{coupon_id}", 'coupon', 400) unless coupons[coupon_id]
+
+          # FIXME assert_existence returns 404 error code but Stripe returns 400
+          # coupon = assert_existence :coupon, coupon_id, coupons[coupon_id]
+
+          coupon = Data.mock_coupon({ id: coupon_id })
+
+          subscription[:discount] = Stripe::Util.convert_to_stripe_object({ coupon: coupon }, {})
+        end
+
         add_subscription_to_customer(customer, subscription)
 
+
         subscription
       end
 
@@ -61,9 +76,21 @@ module StripeMock
         end
 
         # expand the plan for addition to the customer object
-        plan_name = params[:plan] || subscription[:plan][:id]
+        plan_name = params[:plan] if params[:plan] && params[:plan] != {}
+        plan_name ||= subscription[:plan][:id]
         plan = plans[plan_name]
 
+        if params[:coupon]
+          coupon_id = params[:coupon]
+          raise Stripe::InvalidRequestError.new("No such coupon: #{coupon_id}", 'coupon', 400) unless coupons[coupon_id]
+
+          # FIXME assert_existence returns 404 error code but Stripe returns 400
+          # coupon = assert_existence :coupon, coupon_id, coupons[coupon_id]
+
+          coupon = Data.mock_coupon({ id: coupon_id })
+          subscription[:discount] = Stripe::Util.convert_to_stripe_object({ coupon: coupon }, {})
+        end
+
         assert_existence :plan, plan_name, plan
         params[:plan] = plan if params[:plan]
         verify_card_present(customer, plan)
@@ -111,7 +138,8 @@ module StripeMock
       private
 
       def verify_card_present(customer, plan, params={})
-        if customer[:default_source].nil? && plan[:trial_period_days].nil? && plan[:amount] != 0 && plan[:trial_end].nil? && params[:trial_end].nil?
+        if customer[:default_source].nil? && customer[:trial_end].nil? && plan[:trial_period_days].nil? &&
+           plan[:amount] != 0 && plan[:trial_end].nil? && params[:trial_end].nil?
           raise Stripe::InvalidRequestError.new('You must supply a valid card xoxo', nil, 400)
         end
       end

data/lib/stripe_mock/request_handlers/transfers.rb

@@ -34,6 +34,11 @@ module StripeMock
         if params[:bank_account]
           params[:account] = get_bank_by_token(params.delete(:bank_account))
         end
+
+        unless params[:amount].is_a?(Integer) || (params[:amount].is_a?(String) && /^\d+$/.match(params[:amount]))
+          raise Stripe::InvalidRequestError.new("Invalid integer: #{params[:amount]}", 'amount', 400)
+        end
+
         transfers[id] = Data.mock_transfer(params.merge :id => id)
       end
 

data/lib/stripe_mock/request_handlers/validators/param_validators.rb

@@ -6,7 +6,7 @@ module StripeMock
         params[:id] = params[:id].to_s
 
         @base_strategy.create_plan_params.keys.each do |name|
-          raise Stripe::InvalidRequestError.new("Missing required param: #{name}", name) if params[name].nil?
+          raise Stripe::InvalidRequestError.new("Missing required param: #{name}.", name) if params[name].nil?
         end
         if plans[ params[:id] ]
           raise Stripe::InvalidRequestError.new("Plan already exists.", :id)

data/lib/stripe_mock/test_strategies/base.rb

@@ -21,6 +21,41 @@ module StripeMock
         stripe_token.id
       end
 
+      def create_coupon_params(params = {})
+        {
+          id: '10BUCKS',
+          amount_off: 1000,
+          currency: 'usd',
+          max_redemptions: 100,
+          metadata: {
+            created_by: 'admin_acct_1'
+          },
+          duration: 'once'
+        }.merge(params)
+      end
+
+      def create_coupon_percent_of_params(params = {})
+        {
+          id: '25PERCENT',
+          percent_off: 25,
+          redeem_by: nil,
+          duration_in_months: 3,
+          duration: :repeating
+        }.merge(params)
+      end
+
+      def create_coupon(params = {})
+        Stripe::Coupon.create create_coupon_params(params)
+      end
+
+      def delete_all_coupons
+        coupons = Stripe::Coupon.all
+        coupons.data.map(&:delete) if coupons.data.count > 0
+      end
+
+      def prepare_card_error
+        StripeMock.prepare_card_error(:card_error, :new_customer) if StripeMock.state == 'local'
+      end
     end
   end
 end

data/lib/stripe_mock/test_strategies/live.rb

@@ -17,6 +17,19 @@ module StripeMock
         end
       end
 
+      def create_coupon(params={})
+        delete_coupon create_coupon_params(params)[:id]
+        super
+      end
+
+      def delete_coupon(id)
+        begin
+          coupon = Stripe::Coupon.retrieve(id)
+          coupon.delete
+        rescue Stripe::StripeError
+          # do nothing
+        end
+      end
     end
   end
 end

data/lib/stripe_mock/version.rb

@@ -1,4 +1,4 @@
 module StripeMock
   # stripe-ruby-mock version
-  VERSION = "2.1.1"
+  VERSION = "2.2.0"
 end

data/lib/stripe_mock/webhook_fixtures/{customer.card.created.json → customer.source.created.json}

@@ -2,7 +2,7 @@
   "created": 1326853478,
   "livemode": false,
   "id": "evt_00000000000000",
-  "type": "customer.card.created",
+  "type": "customer.source.created",
   "object": "event",
   "data": {
     "object": {
@@ -29,4 +29,4 @@
       "address_zip_check": null
     }
   }
-}
+}

data/lib/stripe_mock/webhook_fixtures/{customer.card.deleted.json → customer.source.deleted.json}

@@ -2,7 +2,7 @@
   "created": 1326853478,
   "livemode": false,
   "id": "evt_00000000000000",
-  "type": "customer.card.deleted",
+  "type": "customer.source.deleted",
   "object": "event",
   "data": {
     "object": {
@@ -29,4 +29,4 @@
       "address_zip_check": null
     }
   }
-}
+}

data/lib/stripe_mock/webhook_fixtures/{customer.card.updated.json → customer.source.updated.json}

@@ -2,7 +2,7 @@
   "created": 1326853478,
   "livemode": false,
   "id": "evt_00000000000000",
-  "type": "customer.card.updated",
+  "type": "customer.source.updated",
   "object": "event",
   "data": {
     "object": {
@@ -33,4 +33,4 @@
       "name": "Testy Tester Jr."
     }
   }
-}
+}

data/lib/stripe_mock/webhook_fixtures/transfer.created.json

@@ -52,6 +52,7 @@
         "object": "list",
         "count": 1,
         "url": "/v1/transfers/tr_2h8RC13PPvwDZs/transactions",
+        "has_more": false,
         "data": [
           {
             "id": "ch_2fb4RERw49oI8s",

data/lib/stripe_mock/webhook_fixtures/transfer.failed.json

@@ -52,6 +52,7 @@
         "object": "list",
         "count": 1,
         "url": "/v1/transfers/tr_2h8RC13PPvwDZs/transactions",
+        "has_more": false,
         "data": [
           {
             "id": "ch_2fb4RERw49oI8s",

data/lib/stripe_mock/webhook_fixtures/transfer.paid.json

@@ -52,6 +52,7 @@
         "object": "list",
         "count": 1,
         "url": "/v1/transfers/tr_2h8RC13PPvwDZs/transactions",
+        "has_more": false,
         "data": [
           {
             "id": "ch_2fb4RERw49oI8s",

data/lib/stripe_mock/webhook_fixtures/transfer.updated.json

@@ -52,6 +52,7 @@
         "object": "list",
         "count": 1,
         "url": "/v1/transfers/tr_2h8RC13PPvwDZs/transactions",
+        "has_more": false,
         "data": [
           {
             "id": "ch_2fb4RERw49oI8s",

data/spec/integration_examples/charge_token_examples.rb

@@ -2,45 +2,47 @@ require 'spec_helper'
 
 shared_examples 'Charging with Tokens' do
 
-  describe "With OAuth" do
+  describe "With OAuth", :oauth => true do
 
-    before do
-      @cus = Stripe::Customer.create(
+    let(:cus) do
+      Stripe::Customer.create(
         :source => stripe_helper.generate_card_token({ :number => '4242424242424242', :brand => 'Visa' })
       )
+    end
 
-      @card_token = Stripe::Token.create({
-        :customer => @cus.id,
-        :source => @cus.sources.first.id
+    let(:card_token) do
+      Stripe::Token.create({
+        :customer => cus.id,
+        :source => cus.sources.first.id
       }, ENV['STRIPE_TEST_OAUTH_ACCESS_TOKEN'])
     end
 
-    it "creates with an oauth access token", :oauth => true do
+    it "creates with an oauth access token" do
       charge = Stripe::Charge.create({
         :amount => 1099,
         :currency => 'usd',
-        :source => @card_token.id
+        :source => card_token.id
       }, ENV['STRIPE_TEST_OAUTH_ACCESS_TOKEN'])
 
-      expect(charge.source.id).to_not eq @cus.sources.first.id
-      expect(charge.source.fingerprint).to eq @cus.sources.first.fingerprint
+      expect(charge.source.id).to_not eq cus.sources.first.id
+      expect(charge.source.fingerprint).to eq cus.sources.first.fingerprint
       expect(charge.source.last4).to eq '4242'
       expect(charge.source.brand).to eq 'Visa'
 
       retrieved_charge = Stripe::Charge.retrieve(charge.id)
 
-      expect(retrieved_charge.source.id).to_not eq @cus.sources.first.id
-      expect(retrieved_charge.source.fingerprint).to eq @cus.sources.first.fingerprint
+      expect(retrieved_charge.source.id).to_not eq cus.sources.first.id
+      expect(retrieved_charge.source.fingerprint).to eq cus.sources.first.fingerprint
       expect(retrieved_charge.source.last4).to eq '4242'
       expect(retrieved_charge.source.brand).to eq 'Visa'
     end
 
-    it "throws an error when the card is not an id", :oauth => true do
+    it "throws an error when the card is not an id" do
       expect {
         charge = Stripe::Charge.create({
           :amount => 1099,
           :currency => 'usd',
-          :source => @card_token
+          :source => card_token
         }, ENV['STRIPE_TEST_OAUTH_ACCESS_TOKEN'])
       }.to raise_error(Stripe::InvalidRequestError, /Invalid token id/)
     end

data/spec/integration_examples/prepare_error_examples.rb

@@ -15,4 +15,24 @@ shared_examples 'Card Error Prep' do
   #     })
   #   }.to raise_error Stripe::CardError
   # end
+
+  it 'is a valid card error', live: true do
+    stripe_helper.prepare_card_error
+
+    begin
+      Stripe::Customer.create(
+        email: 'alice@bob.com',
+        source: stripe_helper.generate_card_token(number: '123')
+      )
+    rescue Stripe::CardError => e
+      body = e.json_body
+      err  = body[:error]
+
+      expect(body).to be_a(Hash)
+      expect(err[:type]).to eq 'card_error'
+      expect(err[:param]).to eq 'number'
+      expect(err[:code]).to eq 'invalid_number'
+      expect(err[:message]).to eq 'This card number looks invalid.'
+    end
+  end
 end

data/spec/shared_stripe_examples/account_examples.rb

@@ -0,0 +1,17 @@
+require 'spec_helper'
+
+shared_examples 'Account API' do
+  it 'retrieves a stripe account', live: true do
+    account = Stripe::Account.retrieve
+
+    expect(account).to be_a Stripe::Account
+    expect(account.id).to match /acct\_/
+  end
+
+  it 'all', live: true do
+    accounts = Stripe::Account.all
+
+    expect(accounts).to be_a Stripe::ListObject
+    expect(accounts.data).to eq []
+  end
+end

data/spec/shared_stripe_examples/card_examples.rb

@@ -221,7 +221,8 @@ shared_examples 'Card API' do
       card_id = "card_123"
       expect { customer.sources.retrieve(card_id) }.to raise_error {|e|
         expect(e).to be_a Stripe::InvalidRequestError
-        expect(e.message).to include "no source", card_id
+        expect(e.message).to match /no.*source/i
+        expect(e.message).to include card_id
         expect(e.param).to eq 'id'
         expect(e.http_status).to eq 404
       }

data/spec/shared_stripe_examples/charge_examples.rb

@@ -133,6 +133,46 @@ shared_examples 'Charge API' do
     }
   end
 
+  it "updates a stripe charge" do
+    original = Stripe::Charge.create({
+      amount: 777,
+      currency: 'USD',
+      source: stripe_helper.generate_card_token,
+      description: 'Original description',
+    })
+    charge = Stripe::Charge.retrieve(original.id)
+
+    charge.description = "Updated description"
+    charge.metadata[:receipt_id] = 1234
+    charge.receipt_email = "newemail@email.com"
+    charge.fraud_details = {"user_report" => "safe"}
+    charge.save
+
+    updated = Stripe::Charge.retrieve(original.id)
+
+    expect(updated.description).to eq(charge.description)
+    expect(updated.metadata.to_hash).to eq(charge.metadata.to_hash)
+    expect(updated.receipt_email).to eq(charge.receipt_email)
+    expect(updated.fraud_details.to_hash).to eq(charge.fraud_details.to_hash)
+  end
+
+  it "disallows most parameters on updating a stripe charge" do
+    original = Stripe::Charge.create({
+      amount: 777,
+      currency: 'USD',
+      source: stripe_helper.generate_card_token,
+      description: 'Original description',
+    })
+
+    charge = Stripe::Charge.retrieve(original.id)
+    charge.currency = "CAD"
+    charge.amount = 777
+    charge.source = {any: "source"}
+
+    expect { charge.save }.to raise_error(Stripe::InvalidRequestError, /Received unknown parameters: currency, amount, source/i)
+  end
+
+
   it "creates a unique balance transaction" do
     charge1 = Stripe::Charge.create(
       amount: 999,
@@ -184,6 +224,30 @@ shared_examples 'Charge API' do
     end
   end
 
+  it 'when use starting_after param', live: true do
+    cus = Stripe::Customer.create(
+        description: 'Customer for test@example.com',
+        source: {
+            object: 'card',
+            number: '4242424242424242',
+            exp_month: 12,
+            exp_year: 2024
+        }
+    )
+    12.times do
+      Stripe::Charge.create(customer: cus.id, amount: 100, currency: "usd")
+    end
+
+    all = Stripe::Charge.all
+    default_limit = 10
+    half = Stripe::Charge.all(starting_after: all.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)
+  end
+
+
   describe 'captured status value' do
     it "reports captured by default" do
       charge = Stripe::Charge.create({