stripe-ruby-mock 1.8.3.14 → 1.8.4.1

data/README.md

@@ -231,6 +231,16 @@ If you always want debug to be on (it's quite verbose), you should put this in a
 * Throw useful errors that emulate Stripe's
   * For example: "You must supply either a card or a customer id" for `Stripe::Charge`
 
+## Developing stripe-ruby-mock
+
+To run the tests:
+
+    $ rspec
+
+Patches are welcome and greatly appreciated! If you're contributing to fix a problem,
+be sure to write tests that illustrate the problem being fixed.
+This will help ensure that the problem remains fixed in future updates.
+
 ## Copyright
 
 Copyright (c) 2013 Gilbert

data/lib/stripe_mock/data.rb

@@ -2,36 +2,28 @@ module StripeMock
   module Data
 
     def self.test_customer(params)
+      cus_id = params[:id] || "test_cus_default"
+      card_id = params.delete(:card_id)
+      cards = []
+      cards << test_card(id: card_id, customer: cus_id) if card_id
       {
         email: 'stripe_mock@example.com',
         description: 'an auto-generated stripe customer data mock',
         object: "customer",
         created: 1372126710,
-        id: "cus_24vuDSooAnB087",
+        id: cus_id,
         livemode: false,
-        active_card: {
-          object: "card",
-          last4: "4242",
-          type: "Visa",
-          exp_month: 12,
-          exp_year: 2013,
-          fingerprint: "wXWJT135mEK107G8",
-          country: "US",
-          name: "Vetty Vet",
-          address_line1: nil,
-          address_line2: nil,
-          address_city: nil,
-          address_state: nil,
-          address_zip: nil,
-          address_country: nil,
-          cvc_check: "pass",
-          address_line1_check: nil,
-          address_zip_check: nil
-        },
         delinquent: false,
         subscription: nil,
         discount: nil,
-        account_balance: 0
+        account_balance: 0,
+        cards: {
+          object: "list",
+          count: cards.count,
+          url: "/v1/customers/#{cus_id}/cards",
+          data: cards
+        },
+        default_card: card_id
       }.merge(params)
     end
 
@@ -87,13 +79,25 @@ module StripeMock
 
     def self.test_card(params={})
       {
-        :type => "Visa",
-        :last4 => "4242",
-        :exp_month => 11,
-        :country => "US",
-        :exp_year => 2012,
-        :id => "cc_test_card",
-        :object => "card"
+        id: "test_cc_default",
+        object: "card",
+        last4: "4242",
+        type: "Visa",
+        exp_month: 4,
+        exp_year: 2013,
+        fingerprint: "wXWJT135mEK107G8",
+        customer: "test_cus_default",
+        country: "US",
+        name: "Johnny App",
+        address_line1: nil,
+        address_line2: nil,
+        address_city: nil,
+        address_state: nil,
+        address_zip: nil,
+        address_country: nil,
+        cvc_check: nil,
+        address_line1_check: nil,
+        address_zip_check: nil
       }.merge(params)
     end
 

data/lib/stripe_mock/instance.rb

@@ -47,16 +47,16 @@ module StripeMock
       end
 
       method_url = "#{method} #{url}"
-      handler = Instance.handler_for_method_url(method_url)
-      mock_error = @error_queue.error_for_handler_name(handler[:name])
-
-      if handler && mock_error
-        @error_queue.dequeue
-        raise mock_error
-      elsif handler
-        res = self.send(handler[:name], handler[:route], method_url, params, headers)
-        puts "[StripeMock res] #{res}" if @debug == true
-        [res, api_key]
+
+      if handler = Instance.handler_for_method_url(method_url)
+        if mock_error = @error_queue.error_for_handler_name(handler[:name])
+          @error_queue.dequeue
+          raise mock_error
+        else
+          res = self.send(handler[:name], handler[:route], method_url, params, headers)
+          puts "[StripeMock res] #{res}" if @debug == true
+          [res, api_key]
+        end
       else
         puts "WARNING: Unrecognized method + url: [#{method} #{url}]"
         puts " params: #{params}"

data/lib/stripe_mock/request_handlers/customers.rb

@@ -13,6 +13,7 @@ module StripeMock
 
       def new_customer(route, method_url, params, headers)
         params[:id] ||= new_id('cus')
+        params[:card_id] = new_id('cc') if params.delete(:card)
         customers[ params[:id] ] = Data.test_customer(params)
       end
 
@@ -46,8 +47,24 @@ module StripeMock
       def update_customer(route, method_url, params, headers)
         route =~ method_url
         assert_existance :customer, $1, customers[$1]
-        customers[$1] ||= Data.test_customer(:id => $1)
-        customers[$1].merge!(params)
+
+        card_id = new_id('cc') if params.delete(:card)
+        cus = customers[$1] ||= Data.test_customer(:id => $1)
+        cus.merge!(params)
+
+        if card_id
+          new_card = Data.test_card(id: card_id, customer: cus[:id])
+
+          if cus[:cards][:count] == 0
+            cus[:cards][:count] += 1
+          else
+            cus[:cards][:data].delete_if {|card| card[:id] == cus[:default_card]}
+          end
+          cus[:cards][:data] << new_card
+          cus[:default_card] = new_card[:id]
+        end
+
+        cus
       end
 
       def get_customer(route, method_url, params, headers)

data/lib/stripe_mock/version.rb

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

data/spec/instance_spec.rb

@@ -20,6 +20,15 @@ describe StripeMock::Instance do
     expect(res[:name]).to eq('String Plan')
   end
 
+  it "exits gracefully on an unrecognized handler url" do
+    dummy_params = {
+      "id" => "str_12345",
+      "name" => "PLAN"
+    }
+
+    expect { res, api_key = StripeMock.instance.mock_request('post', '/v1/unrecongnized_method', 'api_key', dummy_params) }.to_not raise_error
+  end
+
   it "can toggle debug" do
     StripeMock.toggle_debug(true)
     expect(StripeMock.instance.debug).to eq(true)

data/spec/shared_stripe_examples/customer_examples.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 shared_examples 'Customer API' do
 
-  it "creates a stripe customer" do
+  it "creates a stripe customer with a default card" do
     customer = Stripe::Customer.create({
       email: 'johnny@appleseed.com',
       card: 'some_card_token',
@@ -11,6 +11,27 @@ shared_examples 'Customer API' do
     expect(customer.id).to match(/^test_cus/)
     expect(customer.email).to eq('johnny@appleseed.com')
     expect(customer.description).to eq('a description')
+
+    expect(customer.cards.count).to eq(1)
+    expect(customer.cards.data.length).to eq(1)
+    expect(customer.default_card).to_not be_nil
+    expect(customer.default_card).to eq customer.cards.data.first.id
+
+    expect { customer.card }.to raise_error
+  end
+
+  it "creates a stripe customer without a card" do
+    customer = Stripe::Customer.create({
+      email: 'cardless@appleseed.com',
+      description: "no card"
+    })
+    expect(customer.id).to match(/^test_cus/)
+    expect(customer.email).to eq('cardless@appleseed.com')
+    expect(customer.description).to eq('no card')
+
+    expect(customer.cards.count).to eq(0)
+    expect(customer.cards.data.length).to eq(0)
+    expect(customer.default_card).to be_nil
   end
 
   it "stores a created stripe customer in memory" do
@@ -39,6 +60,7 @@ shared_examples 'Customer API' do
 
     expect(customer.id).to eq(original.id)
     expect(customer.email).to eq(original.email)
+    expect(customer.default_card).to eq(original.default_card)
     expect(customer.subscription).to be_nil
   end
 
@@ -74,6 +96,22 @@ shared_examples 'Customer API' do
     expect(customer.description).to eq('new desc')
   end
 
+  it "updates a stripe customer's card" do
+    original = Stripe::Customer.create(id: 'test_customer_update', card: 'token')
+    card = original.cards.data.first
+    expect(original.default_card).to eq(card.id)
+    expect(original.cards.count).to eq(1)
+
+    original.card = 'new_token'
+    original.save
+
+    new_card = original.cards.data.first
+    expect(original.cards.count).to eq(1)
+    expect(original.default_card).to eq(new_card.id)
+
+    expect(new_card.id).to_not eq(card.id)
+  end
+
   it "updates a stripe customer's subscription" do
     plan = Stripe::Plan.create(id: 'silver')
     customer = Stripe::Customer.create(id: 'test_customer_sub')

data/stripe-ruby-mock.gemspec

@@ -17,7 +17,7 @@ Gem::Specification.new do |gem|
   gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
   gem.require_paths = ['lib']
 
-  gem.add_dependency 'stripe', '~> 1.8.3'
+  gem.add_dependency 'stripe', '~> 1.8.4'
   gem.add_dependency 'jimson-temp'
   gem.add_dependency 'dante', '>= 0.1.5'
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: stripe-ruby-mock
 version: !ruby/object:Gem::Version
-  version: 1.8.3.14
+  version: 1.8.4.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-07-18 00:00:00.000000000 Z
+date: 2013-07-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: stripe
@@ -18,7 +18,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 1.8.3
+        version: 1.8.4
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -26,7 +26,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 1.8.3
+        version: 1.8.4
 - !ruby/object:Gem::Dependency
   name: jimson-temp
   requirement: !ruby/object:Gem::Requirement